On Mon, Aug 25, 2008 at 01:01:47PM +0100, Jamie Lokier wrote: > Dave Chinner wrote: > > To keep on top of this, we keep adding new variations and types and > > expect the filesystems to make best use of them (without > > documentation) to optimise for certain situations. Example - the > > new(ish) BIO_META tag that only CFQ understands. I can change the > > way XFS issues bios to use this tag to make CFQ behave the same way > > it used to w.r.t. metadata I/O from XFS, but then the deadline and > > AS will probably regress because they don't understand that tag and > > still need the old optimisations that just got removed. Ditto for > > prioritised bio dispatch - CFQ supports it but none of the others > > do. > > There's nothing wrong with adding BIO_META (for example) and other > hints in _principle_. You should be able to ignore it with no adverse > effects. If its not used by a filesystem (and there's nothing else > competing to use the same disk), I would hope to see the same > performance as other kernels which don't have it. Right, but it's what we need to do to make use of that optimisation that is the problem. For XFS, it needs to replace the current BIO_SYNC hints we use (even for async I/O) to get metadata dispatched quickly. i.e. CFQ looks at the sync flag first then the meta flag. Hence to take advantage of it, we need to remove the BIO_SYNC hints we currently use which will change the behaviour on all other elevators as a side effect. This is the optimisation problem I'm refering to - the BIO_SYNC usage was done years ago to get metadata dispatched quickly because that is what all the elevators did with sync I/O. Now to optimise for CFQ we need to remove that BIO_SYNC optimisation which is still valid for the other elevators.... > If the elevators are being changed in such a way that old filesystem > code which doesn't use new hint bits is running significantly slower, > surely that's blatant elevator regression, and that's where the bugs > should be reported and fixed? Sure, but in reality getting ppl to go through the pain of triage is extremely rare because it only takes 10s to change elevators and make the problem go away... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- 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
