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.
|