Bug 240453
Summary: | DLM locking assertion failure line 1390 | ||
---|---|---|---|
Product: | [Retired] Red Hat Cluster Suite | Reporter: | Bryn M. Reeves <bmr> |
Component: | dlm-kernel | Assignee: | David Teigland <teigland> |
Status: | CLOSED ERRATA | QA Contact: | Cluster QE <mspqa-list> |
Severity: | high | Docs Contact: | |
Priority: | medium | ||
Version: | 4 | CC: | ccaulfie, cfeist, cluster-maint, jplans |
Target Milestone: | --- | Keywords: | ZStream |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | RHBA-2007-0995 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-11-21 21:55:48 UTC | Type: | --- |
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: | |||
Bug Blocks: | 301511 |
Description
Bryn M. Reeves
2007-05-17 17:04:15 UTC
This is code that we've never used or tested; I'm surprised it works at all! Just so expectations are set appropriately, if you use the rhel4 dlm for anything beyond gfs/clvm/rgmanager, you're in uncharted territory and will definately find a lot of broken things. Rewriting the dlm (the result being in rhel5) was the only way to make the dlm more generally usable. In rhel5 it's definately our aim to make the dlm work in general for user's apps. (The kind of deadlock detection involved in this bug is a feature that I'm working on right now, actually, and is planned for 5.1.) Now, on to this specific bug in conversion_deadlock_check(), it should be pretty trivial to fix, I'd suggest changing DLM_ASSERT(lkb->lkb_status == GDLM_LKSTS_CONVERT,); into if (kb->lkb_status != GDLM_LKSTS_CONVERT) return NULL; If the person reporting this problem can test and confirm that the change in comment 3 works, then I'll check in that change. patch added to cvs Checking in locking.c; /cvs/cluster/cluster/dlm-kernel/src/Attic/locking.c,v <-- locking.c new revision: 1.50.2.11; previous revision: 1.50.2.10 done An advisory has been issued which should help the problem described in this bug report. This report is therefore being closed with a resolution of ERRATA. For more information on the solution and/or where to find the updated files, please follow the link below. You may reopen this bug report if the solution does not work for you. http://rhn.redhat.com/errata/RHBA-2007-0995.html (In reply to comment #6) > patch added to cvs I notice that this patch isn't included in the STABLE branch in CVS, but is in RHEL46 branch. |