[ ssic-linux-Bugs-1864069 ] non-paired lock/unlock in cfs_commit_inode

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

-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/