diff options
| author | Krishnan P <kp@gluster.com> | 2011-06-20 01:01:20 +0000 |
|---|---|---|
| committer | Anand Avati <avati@gluster.com> | 2011-06-20 00:21:04 -0700 |
| commit | 608b22609793549eb8e8e2b2899af2d1ff5c193c (patch) | |
| tree | dc3248c9f2cf033846ea86ae200c92cdd7d03c04 /xlators/cluster/afr/src/pump.c | |
| parent | 96ae0db506f43ceed0d3346d0351b1bb73b0b19c (diff) | |
pump: mark pending before notify to children to avoid race in single CPU.
Signed-off-by: Krishnan Parthasarathi <kp@gluster.com>
Signed-off-by: Anand Avati <avati@gluster.com>
BUG: 3050 ('replace-brick' hangs on vm's)
URL: http://bugs.gluster.com/cgi-bin/bugzilla3/show_bug.cgi?id=3050
Diffstat (limited to 'xlators/cluster/afr/src/pump.c')
| -rw-r--r-- | xlators/cluster/afr/src/pump.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/xlators/cluster/afr/src/pump.c b/xlators/cluster/afr/src/pump.c index 24289b52505..873fc8bde8a 100644 --- a/xlators/cluster/afr/src/pump.c +++ b/xlators/cluster/afr/src/pump.c @@ -873,13 +873,13 @@ pump_cmd_start_setxattr_cbk (call_frame_t *frame, "Successfully initiated destination " "brick connect"); + pump_mark_start_pending (this); + /* send the PARENT_UP as pump is ready now */ prev = cookie; if (prev && prev->this) prev->this->notify (prev->this, GF_EVENT_PARENT_UP, this); - pump_mark_start_pending (this); - out: local->op_ret = ret; pump_command_reply (frame, this); |