Vivek,
On Wed, Feb 03, 2010 at 06:38:03AM +0800, Vivek Goyal wrote:
> On Tue, Feb 02, 2010 at 11:28:35PM +0800, Wu Fengguang wrote:
> > Andrew,
> >
> > This is to lift default readahead size to 512KB, which I believe yields
> > more I/O throughput without noticeably increasing I/O latency for today's HDD.
> >
>
> Hi Fengguang,
>
> I was doing a quick test with the patches. I was using fio to run some
> sequential reader threads. I have got one access to one Lun from an HP
> EVA. In my case it looks like with the patches throughput has come down.
Thank you for the quick testing!
This patchset does 3 things:
1) 512K readahead size
2) new readahead algorithms
3) new readahead tracing/stats interfaces
(1) will impact performance, while (2) _might_ impact performance in
case of bugs.
Would you kindly retest the patchset with readahead size manually set
to 128KB? That would help identify the root cause of the performance
drop:
DEV=sda
echo 128 > /sys/block/$DEV/queue/read_ahead_kb
The readahead stats provided by the patchset are very useful for
analyzing the problem:
mount -t debugfs none /debug
# for each benchmark:
echo > /debug/readahead/stats # reset counters
# do benchmark
cat /debug/readahead/stats # check counters
Thanks,
Fengguang
--
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
