[PATCH 1/5] xfs: fix inode validity check in xfs_iflush_cluster
Christoph Hellwig
hch at infradead.org
Thu Apr 7 10:47:31 CDT 2016
On Wed, Apr 06, 2016 at 07:22:50PM +1000, Dave Chinner wrote:
> From: Dave Chinner <dchinner at redhat.com>
>
> Some careless idiot wrote crap code in commit 1a3e8f3 ("xfs: convert
> inode cache lookups to use RCU locking") back in late 2010, and so
> xfs_iflush_cluster checks the wrong inode for whether it is still
> valid under RCU protection. Fix it to lock and check the correct
> inode.
>
> Part of the reason for the screwup was the unconventional naming of
> the cluster inode variable - iq - so also rename all the cluster
> inode variables to use a more conventional prefixes to reduce
> potential future confusion (cilist, cilist_size, cip).
Between all that renaming I didn't actually manage to find the fix :)
Can you split the rename and the fix into separate patches?
More information about the xfs
mailing list