summaryrefslogtreecommitdiff
path: root/drivers/video
diff options
context:
space:
mode:
authorAlex Elder <elder@dreamhost.com>2012-02-16 22:01:00 +0000
committerBen Myers <bpm@sgi.com>2012-02-25 13:55:49 -0600
commitad637a10f444fc66b1f6d4a28fe30d4c61ed0161 (patch)
treec4512d9553696d9d5e36e40a909c3f7dabdcb1fb /drivers/video
parent9006fb91cfdf22812923f0536c7531c429c1aeab (diff)
xfs: only take the ILOCK in xfs_reclaim_inode()
At the end of xfs_reclaim_inode(), the inode is locked in order to we wait for a possible concurrent lookup to complete before the inode is freed. This synchronization step was taking both the ILOCK and the IOLOCK, but the latter was causing lockdep to produce reports of the possibility of deadlock. It turns out that there's no need to acquire the IOLOCK at this point anyway. It may have been required in some earlier version of the code, but there should be no need to take the IOLOCK in xfs_iget(), so there's no (longer) any need to get it here for synchronization. Add an assertion in xfs_iget() as a reminder of this assumption. Dave Chinner diagnosed this on IRC, and Christoph Hellwig suggested no longer including the IOLOCK. I just put together the patch. Signed-off-by: Alex Elder <elder@dreamhost.com> Reviewed-by: Dave Chinner <dchinner@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Ben Myers <bpm@sgi.com>
Diffstat (limited to 'drivers/video')
0 files changed, 0 insertions, 0 deletions