diff options
| author |   Goldwyn Rodrigues <rgoldwyn@suse.com> | 2015-12-21 10:50:59 +1100 |
|---|---|---|
| committer |   NeilBrown <neilb@suse.com> | 2016-01-06 11:38:27 +1100 |
| commit | ac277c6a8a39bc50f891a3477625330c276bd7f5 (patch) | |
| tree | 767ee803cc2e14dddf8607554b14b6b6e60748c4 /drivers/md/md-cluster.c | |
| parent | md/raid5: remove redundant check in stripe_add_to_batch_list() (diff) | |
| download | linux-dev-ac277c6a8a39bc50f891a3477625330c276bd7f5.tar.xz linux-dev-ac277c6a8a39bc50f891a3477625330c276bd7f5.zip | |
md-cluster: Avoid the resync ping-pong
If a RESYNCING message with (0,0) has been sent before, do not send it
again. This avoids a resync ping pong between the nodes. We read
the bitmap lockresource's LVB to figure out the previous value
of the RESYNCING message.
Signed-off-by: Goldwyn Rodrigues <rgoldwyn@suse.com>
Signed-off-by: NeilBrown <neilb@suse.com>
Diffstat (limited to 'drivers/md/md-cluster.c')
| -rw-r--r-- | drivers/md/md-cluster.c | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/drivers/md/md-cluster.c b/drivers/md/md-cluster.c index d6a1126d85ce..e57bbfed1638 100644 --- a/drivers/md/md-cluster.c +++ b/drivers/md/md-cluster.c @@ -882,8 +882,16 @@ static int resync_start(struct mddev *mddev) static int resync_info_update(struct mddev *mddev, sector_t lo, sector_t hi) { struct md_cluster_info *cinfo = mddev->cluster_info; + struct resync_info ri; struct cluster_msg cmsg = {0}; + /* do not send zero again, if we have sent before */ + if (hi == 0) { + memcpy(&ri, cinfo->bitmap_lockres->lksb.sb_lvbptr, sizeof(struct resync_info)); + if (le64_to_cpu(ri.hi) == 0) + return 0; + } + add_resync_info(cinfo->bitmap_lockres, lo, hi); /* Re-acquire the lock to refresh LVB */ dlm_lock_sync(cinfo->bitmap_lockres, DLM_LOCK_PW); |