Lachlan McIlroy wrote:
Russell, this fixes xfs_iext_irec_compact_full(). If we don't move
all the records from the next page into the current page then we need
to update the er_extoff of the modified page as we move the remaining
extents up. Would you mind giving it a go?
--- a/fs/xfs/xfs_inode.c 2008-09-18 18:48:46.000000000 +1000
+++ b/fs/xfs/xfs_inode.c 2008-09-18 18:57:18.000000000 +1000
@@ -4623,6 +4623,7 @@ xfs_iext_irec_compact_full(
+ erp_next->er_extoff += ext_diff;
Cool I'll give it some run through when I done traveling.
I still think compact_full should simply be eliminated since
it really doesn't help, and it's obviously confusing code.
Or we should make sure it works and get rid of compact_pages
since compact_full behaves just like compact_pages when not
doing partial moves.