Re: [PATCH RFC - TAKE TWO - 00/12] New version of the BFQ I/O Scheduler
From: Mike Galbraith <hidden>
Date: 2014-06-03 04:12:50
Also in:
lkml
On Mon, 2014-06-02 at 13:33 -0400, Tejun Heo wrote:
Hello, Pavel. On Mon, Jun 02, 2014 at 01:14:33PM +0200, Pavel Machek wrote:quoted
Now.. I see it is more work for storage maintainers, because there'll be more code to maintain in the interim. But perhaps user advantages are worth it?I'm quite skeptical about going that route. Not necessarily because of the extra amount of work but more the higher probability of getting into situation where we can neither push forward or back out. It's difficult to define clear deadline and there will likely be unforeseen challenges in the planned convergence of the two schedulers, eventually, it isn't too unlikely to be in a situation where we have to admit defeat and just keep both schedulers. Note that developer overhead isn't the only factor here. Providing two slightly different alternatives inevitably makes userland grow dependencies on subtleties of both and there's a lot less pressure to make judgement calls and take appropriate trade-offs, which have fairly high chance of deadlocking progress towards any direction.
But OTOH.. This thing (allegedly) fixes issues that have existed for ages, issues which have (also allegedly) not been fixed in all that time despite a number of people having done a lot of this and that over the years. If the claims are true, seems to me that would make BFQ a bit special, and perhaps worth some extra leeway and effort to ensure that what we are being offered on a silver plate doesn't molder away out of tree forever. If it were say put in staging, and it were stated right up front that it isn't ever going to go further (Jens already said that more or less), and _will_ drop dead if it stagnates, that would surely increase the test base to shake out problem spots (surely it has some), and allow users who meet an issue in either IO scheduler to verify it with the flick of a switch every step of the way to whichever ending, and maybe even motivate other IO people to help with the merge and/or to compare their changes at the flick of that same switch. -Mike