Re: [PATCH 01/17] staging/lustre/llite: fix open lock matching in ll_md_blocking_ast()

2014-03-03 Thread Dan Carpenter
On Fri, Feb 28, 2014 at 09:16:30PM -0500, Oleg Drokin wrote: > > - if (och) { /* There might be a race and somebody have freed this och > - already */ > + if (och != NULL) { > + /* There might be a race and this handle may already > +be

Re: [PATCH 01/17] staging/lustre/llite: fix open lock matching in ll_md_blocking_ast()

2014-03-03 Thread Dan Carpenter
On Fri, Feb 28, 2014 at 09:16:30PM -0500, Oleg Drokin wrote: - if (och) { /* There might be a race and somebody have freed this och - already */ + if (och != NULL) { + /* There might be a race and this handle may already +be closed. */

[PATCH 01/17] staging/lustre/llite: fix open lock matching in ll_md_blocking_ast()

2014-02-28 Thread Oleg Drokin
From: "John L. Hammond" In ll_md_blocking_ast() match open locks before all others, ensuring that MDS_INODELOCK_OPEN is not cleared from bits by another open lock with a different mode. Change the int flags parameter of ll_md_real_close() to fmode_t fmode. Clean up verious style issues in both

[PATCH 01/17] staging/lustre/llite: fix open lock matching in ll_md_blocking_ast()

2014-02-28 Thread Oleg Drokin
From: John L. Hammond john.hamm...@intel.com In ll_md_blocking_ast() match open locks before all others, ensuring that MDS_INODELOCK_OPEN is not cleared from bits by another open lock with a different mode. Change the int flags parameter of ll_md_real_close() to fmode_t fmode. Clean up verious