| Summary: | Posix locks cannot handle a single unlock over multiple held locks | ||||||
|---|---|---|---|---|---|---|---|
| Product: | [Community] GlusterFS | Reporter: | Pavan Vilas Sondur <pavan> | ||||
| Component: | locks | Assignee: | Pavan Vilas Sondur <pavan> | ||||
| Status: | CLOSED CURRENTRELEASE | QA Contact: | |||||
| Severity: | medium | Docs Contact: | |||||
| Priority: | high | ||||||
| Version: | mainline | CC: | corentin.chary, gluster-bugs, vbellur, vijay, vikas | ||||
| Target Milestone: | --- | ||||||
| Target Release: | --- | ||||||
| Hardware: | All | ||||||
| OS: | Linux | ||||||
| Whiteboard: | |||||||
| Fixed In Version: | Doc Type: | Bug Fix | |||||
| Doc Text: | Story Points: | --- | |||||
| Clone Of: | Environment: | ||||||
| Last Closed: | Type: | --- | |||||
| Regression: | RTP | Mount Type: | --- | ||||
| Documentation: | --- | CRM: | |||||
| Verified Versions: | Category: | --- | |||||
| oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |||||
| Cloudforms Team: | --- | Target Upstream Version: | |||||
| Attachments: |
|
||||||
|
Description
Pavan Vilas Sondur
2009-10-05 11:41:36 UTC
Created attachment 77 [details]
new version of pnmtotiff patch to use packbits by default
Here is a patch that seems to fix the problem (for my test case anyway).
* Init the list structure, because it will be used later, and subtract_lock does not do it
* Remove the special handling for unlocks, I see no reason to put it here
This should also fix some bugs not related to unlocks.
a single unlock over a region where more than 1 lock was previoudly held, fails to unlock all held locks. PATCH: http://patches.gluster.com/patch/1840 in master (features/locks: Fix insert_and_merge) PATCH: http://patches.gluster.com/patch/1839 in release-2.0 (features/locks: Fix insert_and_merge) PATCH: http://patches.gluster.com/patch/1847 in master (Add INIT_LIST_HEAD which was left out in the previous commit) PATCH: http://patches.gluster.com/patch/1846 in release-2.0 (Add INIT_LIST_HEAD which was left out in the previous commit) |