[PATCH 4/8] xfs: introduce xfs_rw_lock() helpers for locking the inode

Dave Chinner david at fromorbit.com
Mon Jan 10 16:26:18 CST 2011


On Mon, Jan 10, 2011 at 02:23:05PM -0500, Christoph Hellwig wrote:
> > -	xfs_ilock(ip, XFS_IOLOCK_EXCL);
> > +	xfs_rw_ilock(ip, XFS_ILOCK_EXCL|XFS_IOLOCK_EXCL);
> 
> this now takes i_mutex, which will be taken again by
> generic_file_splice_write.

Ah, oops. I'll revert that bit - I thought we had test coverage of
that interface. I guess we need a basic sanity test for this
functionality. I'll write one.

Cheers,

Dave.
-- 
Dave Chinner
david at fromorbit.com




More information about the xfs mailing list