Bug 1176756
Summary: | glusterd: remote locking failure when multiple synctask transactions are run | ||
---|---|---|---|
Product: | [Community] GlusterFS | Reporter: | Atin Mukherjee <amukherj> |
Component: | glusterd | Assignee: | Atin Mukherjee <amukherj> |
Status: | CLOSED CURRENTRELEASE | QA Contact: | |
Severity: | unspecified | Docs Contact: | |
Priority: | unspecified | ||
Version: | 3.6.0 | CC: | bugs, gluster-bugs, jbyers, rabhat |
Target Milestone: | --- | Keywords: | Triaged |
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | 1173414 | Environment: | |
Last Closed: | 2016-01-08 09:17:59 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | 1173414, 1182458 | ||
Bug Blocks: | 1184460 |
Description
Atin Mukherjee
2014-12-23 04:23:26 UTC
REVIEW: http://review.gluster.org/9328 (glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3) posted (#1) for review on release-3.6 by Atin Mukherjee (amukherj) REVIEW: http://review.gluster.org/9328 (glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3) posted (#2) for review on release-3.6 by Atin Mukherjee (amukherj) REVIEW: http://review.gluster.org/9328 (glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3) posted (#3) for review on release-3.6 by Atin Mukherjee (amukherj) REVIEW: http://review.gluster.org/9328 (glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3) posted (#4) for review on release-3.6 by Atin Mukherjee (amukherj) REVIEW: http://review.gluster.org/9328 (glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3) posted (#5) for review on release-3.6 by Atin Mukherjee (amukherj) COMMIT: http://review.gluster.org/9328 committed in release-3.6 by Raghavendra Bhat (raghavendra) ------ commit a1d9f01b28267fc333aebc49cb81ee69dc2c24f8 Author: Atin Mukherjee <amukherj> Date: Fri Dec 12 07:21:19 2014 +0530 glusterd: Maintain per transaction xaction_peers list in syncop & mgmt_v3 In current implementation xaction_peers list is maintained in a global variable (glustrd_priv_t) for syncop/mgmt_v3. This means consistency and atomicity of peerinfo list across transactions is not guranteed when multiple syncop/mgmt_v3 transaction are going through. We had got into a problem in mgmt_v3-locks.t which was failing spuriously, the reason for that was two volume set operations (in two different volume) was going through simultaneouly and both of these transaction were manipulating the same xaction_peers structure which lead to a corrupted list. Because of which in some cases unlock request to peer was never triggered and we end up with having stale locks. Solution is to maintain a per transaction local xaction_peers list for every syncop. Please note I've identified this problem in op-sm area as well and a separate patch will be attempted to fix it. Finally thanks to Krishnan Parthasarathi and Kaushal M for your constant help to get to the root cause. Backport URL : http://review.gluster.org/#/c/9269/ http://review.gluster.org/#/c/9422/ http://review.gluster.org/#/c/9350/ Change-Id: Ib1eaac9e5c8fc319f4e7f8d2ad965bc1357a7c63 BUG: 1176756 Signed-off-by: Atin Mukherjee <amukherj> Reviewed-on: http://review.gluster.org/9269 Tested-by: Gluster Build System <jenkins.com> Reviewed-by: Kaushal M <kaushal> Reviewed-on: http://review.gluster.org/9328 Reviewed-by: Raghavendra Bhat <raghavendra> Tested-by: Raghavendra Bhat <raghavendra> |