On Sat, Oct 03 2009, Corrado Zoccolo wrote: > Hi, > On Sat, Oct 3, 2009 at 11:00 AM, Mike Galbraith <efault@xxxxxx> wrote: > > On Sat, 2009-10-03 at 09:24 +0200, Jens Axboe wrote: > > > >> After shutting down the computer yesterday, I was thinking a bit about > >> this issue and how to solve it without incurring too much delay. If we > >> add a stricter control of the depth, that may help. So instead of > >> allowing up to max_quantum (or larger) depths, only allow gradual build > >> up of that the farther we get away from a dispatch from the sync IO > >> queues. For example, when switching to an async or seeky sync queue, > >> initially allow just 1 in flight. For the next round, if there still > >> hasn't been sync activity, allow 2, then 4, etc. If we see sync IO queue > >> again, immediately drop to 1. > >> > > I would limit just async I/O. Seeky sync queues are automatically > throttled by being sync, and have already high latency, so we > shouldn't increase it artificially. I think, instead, that we should > send multiple seeky requests (possibly coming from different queues) > at once. They will help especially with raid devices, where the seeks > for requests going to different disks will happen in parallel. > Async is the prime offendor, definitely. > >> It could tie in with (or partly replace) the overload feature. The key > >> to good latency and decent throughput is knowing when to allow queue > >> build up and when not to. > > > > Hm. Starting at 1 sounds a bit thin (like IDLE), multiple iterations to > > build/unleash any sizable IO, but that's just my gut talking. > > > On the other hand, sending 1 write first and then waiting it to > complete before submitting new ones, will help performing more merges, > so the subsequent requests will be bigger and thus more efficient. Usually async writes stack up very quickly, so as long as you don't drain completely, the merging will happen automagically anyway. -- Jens Axboe -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
