summaryrefslogtreecommitdiffstats
path: root/fs/xfs/xfs_dir2_readdir.c
diff options
context:
space:
mode:
authorDave Chinner <dchinner@redhat.com>2014-12-04 09:43:13 +1100
committerDave Chinner <david@fromorbit.com>2014-12-04 09:43:13 +1100
commit2d3d0c53df99587e1d58759f805c3aae79fac453 (patch)
tree0ee91663769ac5c8b3e67d833a596efa1a15b72d /fs/xfs/xfs_dir2_readdir.c
parentcdc9cec7c0ff521edf8c0e9c9432bf8fdccfc702 (diff)
downloadlinux-2d3d0c53df99587e1d58759f805c3aae79fac453.tar.gz
linux-2d3d0c53df99587e1d58759f805c3aae79fac453.tar.bz2
linux-2d3d0c53df99587e1d58759f805c3aae79fac453.zip
xfs: lobotomise xfs_trans_read_buf_map()
There's a case in that code where it checks for a buffer match in a transaction where the buffer is not marked done. i.e. trying to catch a buffer we have locked in the transaction but have not completed IO on. The only way we can find a buffer that has not had IO completed on it is if it had readahead issued on it, but we never do readahead on buffers that we have already joined into a transaction. Hence this condition cannot occur, and buffers locked and joined into a transaction should always be marked done and not under IO. Remove this code and re-order xfs_trans_read_buf_map() to remove duplicated IO dispatch and error handling code. Signed-off-by: Dave Chinner <dchinner@redhat.com> Reviewed-by: Christoph Hellwig <hch@lst.de> Signed-off-by: Dave Chinner <david@fromorbit.com>
Diffstat (limited to 'fs/xfs/xfs_dir2_readdir.c')
0 files changed, 0 insertions, 0 deletions