Re: review: rework bulkstat readahead

[Christoph Hellwig <hch@infradead.org>]

On Tue, Jul 25, 2006 at 01:46:51PM +1000, Nathan Scott wrote:
> Hi all,
>     
> This patch rearranges the logic inside xfs_bulkstat to issue
> readahead in a way that performs alot better.  The most critical 
> piece is moving the readahead out of the final read-and-format
> loop into the earlier build-up-irbuf-array loop.  This means:
>     
> - buffer readaheads are all issued at once, up front, before we 
> start to issue the blocking reads waiting on each inode cluster;
>     
> - as a by-product, the readahead which is done for the internal
> nodes of the AGI btree (from within xfs_inobt_increment) is now 
> issued in the correct order relative to all the leaf readahead. 
> 
> I've also removed the incorrect endian swapping on the irbuf array
> elements, that was always unnecessary as they're only ever held in
> memory here.

looks good and sorry for not noticing the unessecary swapping when doing
the sparse annotations.