xfs: a few small tweaks for overwrites in xfs_vm_writepage
authorChristoph Hellwig <hch@infradead.org>
Fri, 10 Dec 2010 08:42:18 +0000 (08:42 +0000)
committerAlex Elder <aelder@sgi.com>
Thu, 16 Dec 2010 22:05:44 +0000 (16:05 -0600)
commit6ac7248ec5f20cb44a063d7c7191b8e0068b5a28
tree26e43646609e98d52d89ae7f4eecbf994b42cf43
parent221cb2517e8fc9a1d67c7a8a9c19fc5a916b583f
xfs: a few small tweaks for overwrites in xfs_vm_writepage

Don't trylock the buffer.  We are the only one ever locking it for a
regular file address space, and trylock was only copied from the
generic code which did it due to the old buffer based writeout in
jbd.  Also make sure to only write out the buffer if the iomap
actually is valid, because we wouldn't have a proper mapping
otherwise.  In practice we will never get an invalid mapping here as
the page lock guarantees truncate doesn't race with us, but better
be safe than sorry.  Also make sure we allocate a new ioend when
crossing boundaries between mappings, just like we do for delalloc
and unwritten extents.  Again this currently doesn't matter as the
I/O end handler only cares for the boundaries for unwritten extents,
but this makes the code fully correct and the same as for
delalloc/unwritten extents.

Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <dchinner@redhat.com>
Signed-off-by: Alex Elder <aelder@sgi.com>
fs/xfs/linux-2.6/xfs_aops.c