]> bbs.cooldavid.org Git - net-next-2.6.git/commit
xfs: remove incorrect assert in xfs_vm_writepage
authorChristoph Hellwig <hch@infradead.org>
Wed, 10 Nov 2010 21:39:11 +0000 (21:39 +0000)
committerAlex Elder <aelder@sgi.com>
Wed, 10 Nov 2010 21:51:10 +0000 (15:51 -0600)
commitece413f59f257682de4a2e2e42af33b016af53f3
treefe26d60458a4bfbd39e7e319e41c0ceca011914d
parentc6f6cd0608b1826ee1797cf57a808416e4bdb806
xfs: remove incorrect assert in xfs_vm_writepage

In commit 20cb52ebd1b5ca6fa8a5d9b6b1392292f5ca8a45, titled
"xfs: simplify xfs_vm_writepage" I added an assert that any !mapped and
uptodate buffers are not dirty.  That asserts turns out to trigger a lot
when running fsx on filesystems with small block sizes.  The reason for
that is that the assert is simply incorrect.  !mapped and uptodate
just mean this buffer covers a hole, and whenever we do a set_page_dirty
we mark all blocks in the page dirty, no matter if they have data or
not.  So remove the assert, and update the comment above the condition
to match reality.

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