On Thu, Dec 08, 2011 at 10:58:02AM -0500, Christoph Hellwig wrote: > Replace i_pin_wait, which is only used during synchronous inode flushing > with a bit waitqueue. This trades off a much smaller inode against > slightly slower wakeup performance, and saves 12 (32-bit) or 20 (64-bit) > bytes in the XFS inode. > > Signed-off-by: Christoph Hellwig <hch@xxxxxx> > Reviewed-by: Alex Elder <aelder@xxxxxxx> Looks fine - the pinning code never modifies the i_flags field, so doesn't require the i_flags_lock anywhere so the concerns i had about the previous patch are not relevant here... Reviewed-by: Dave Chinner <dchinner@xxxxxxxxxx> -- Dave Chinner david@xxxxxxxxxxxxx _______________________________________________ xfs mailing list xfs@xxxxxxxxxxx http://oss.sgi.com/mailman/listinfo/xfs