[SSI-devel] [ ssic-linux-Bugs-1864069 ] non-paired lock/unlock in cfs_commit_inode
Brought to you by:
brucewalker,
rogertsang
From: SourceForge.net <no...@so...> - 2008-01-04 17:12:51
|
Bugs item #1864069, was opened at 2008-01-04 09:12 Message generated for change (Tracker Item Submitted) made by Item Submitter You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=405834&aid=1864069&group_id=32541 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: Filesystem Group: v2.0.0pre1 Status: Open Resolution: None Priority: 5 Private: No Submitted By: Nobody/Anonymous (nobody) Assigned to: Nobody/Anonymous (nobody) Summary: non-paired lock/unlock in cfs_commit_inode Initial Comment: In openssi/kernel/cluster/ssi/cfs/write.c we have: int cfs_commit_inode(struct inode *inode, unsigned long idx_start, unsigned int npages, int how) { [...] spin_lock(&cfs_wreq_lock); res = cfs_scan_commit(inode, &head, idx_start, npages); spin_unlock(&cfs_wreq_lock); if (res) { res += cfs_scan_commit(inode, &head, 0, 0); spin_unlock(&cfs_wreq_lock); error = cfs_commit_list(&head, how); } else spin_unlock(&cfs_wreq_lock); Looks like we're unlocking cfs_wreq_lock too many times. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=405834&aid=1864069&group_id=32541 |