On Tue, Jul 29, 2008 at 09:30:00PM +0200, Christoph Hellwig wrote:
> This is the full btree unficiation based on Dave's initial patches plus
> various cleanups. This second versions post now contains the complete
> btree refactoring, and passes XFSQA.
Cool. Are the first few patches unchanged from the ones I reviewed
earlier?
> The real news is the missing btree operations, including all the hairy
> ones. These have been reimplemented looking at the three original
> btrees and Daves code, in doubt going with the former. That means the
> small helpers to insert/remove/move records are gone now in favour of
> following the original code more closely. This fixed the one off error
> showing up in the intial refactoring.
I'll have a close look at this ;)
> There's a lot of new comments and empty lines, so by diffstat the new
> code isn't all that much smaller:
>
> 36 files changed, 6041 insertions(+), 7128 deletions(-)
'bout the same as my original change series...
> but the (debug) binary modules is almost 15kb smaller:
>
> text data bss dec hex filename
> 631577 4227 3092 638896 9bfb0 fs/xfs/xfs.ko.old
> 616929 4435 3124 624488 98768 fs/xfs/xfs.ko.btree
And again, roughly the same. The difference will really be seen
when we implement new btrees.....
Cheers,,
Dave.
--
Dave Chinner
david@xxxxxxxxxxxxx
|