diff options
| author | Krishnan Parthasarathi <kparthas@redhat.com> | 2012-11-26 13:09:44 +0530 |
|---|---|---|
| committer | Anand Avati <avati@redhat.com> | 2012-11-28 16:27:01 -0800 |
| commit | fadc34e7ce82f9e7f98f20e995cb2bbf71a00b20 (patch) | |
| tree | 4f78b550af17a89314314342428abf588511ad49 /tests | |
| parent | 5f88d4c1b036228ca4036e854d533f0b2fc3adc1 (diff) | |
glusterd: Protected conf->xprt_list racy access.
- epoll on RPCSVC_EVENT_ACCEPT would add corresponding xprt
onto the xprt_list. Concurrently, synctask thread (volume op)
would call into glusterd_fetchspec_notify which iterates on
the xprt_list. Added a mutex to protect such a racy access of
the list.
Change-Id: Idc51b4bdb1c814dfab7790e1c899d6977f7640f2
BUG: 878873
Signed-off-by: Krishnan Parthasarathi <kparthas@redhat.com>
Reviewed-on: http://review.gluster.org/4241
Reviewed-by: Raghavendra G <raghavendra@gluster.com>
Reviewed-by: Jeff Darcy <jdarcy@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/bugs/bug-878873.t | 29 |
1 files changed, 29 insertions, 0 deletions
diff --git a/tests/bugs/bug-878873.t b/tests/bugs/bug-878873.t new file mode 100644 index 00000000000..efdcaca8925 --- /dev/null +++ b/tests/bugs/bug-878873.t @@ -0,0 +1,29 @@ +#!/bin/bash + +. $(dirname $0)/../include.rc + +cleanup; + +TEST glusterd +TEST pidof glusterd +TEST $CLI volume info; + +TEST $CLI volume create $V0 $H0:$B0/${V0}1 $H0:$B0/${V0}2 $H0:$B0/${V0}3; + +function brick_count() +{ + local vol=$1; + + $CLI volume info $vol | egrep "^Brick[0-9]+: " | wc -l; +} + + +TEST $CLI volume start $V0 +TEST $CLI volume remove-brick $V0 $H0:$B0/${V0}2 start; +TEST $CLI volume remove-brick $V0 $H0:$B0/${V0}2 commit; +EXPECT '2' brick_count $V0 + +TEST $CLI volume remove-brick $V0 $H0:$B0/${V0}3; +EXPECT '1' brick_count $V0 + + |