> From: Torsten Kaiser <just.for.lkml at googlemail.com>
> Commit fb59581404ab7ec5075299065c22cb211a9262a9 removed
> xfs_flushinval_pages() and changed its callers to use
> filemap_write_and_wait() and truncate_pagecache_range() directly.
> But in xfs_swap_extents() this change accidental switched the argument
> for 'tip' to 'ip'. This patch switches it back to 'tip'
The patch works in casual testing, and I can defragment files using
Linux 3.8-rc4. It solves the problem listed here:
However, I'm having trouble getting the filesystems to fragment again
so that a more formal test can be done...and that's one of the many
reasons that I use XFS ;-)
xfstests #042, #222, and #227 passed outright, with and without a lot
of debug options in the kernel. Very nice!
Without debug options, xfstests #111 failed with a simple "Directory
defragmentation not supported" message but did not cause problems for
later tests. With debug options, xfstests #111 causes a screenful of
oops and a stopped PC--it did this when defrag was broken, too--and I
don't know whether to blame the defrag code, the test code, or the
debug code. I'm willing to write down that oops screen, but only on
In other words, I'm happy with the patch. Good work!