On Tue, Apr 12 2005, Chen, Kenneth W wrote: > Chen, Kenneth W wrote on Tuesday, April 05, 2005 5:13 PM > > Jens Axboe wrote on Tuesday, April 05, 2005 7:54 AM > > > On Tue, Mar 29 2005, Chen, Kenneth W wrote: > > > > Jens Axboe wrote on Tuesday, March 29, 2005 12:04 PM > > > > > No such promise was ever made, noop just means it does 'basically > > > > > nothing'. It never meant FIFO in anyway, we cannot break the semantics > > > > > of block layer commands just for the hell of it. > > > > > > > > Acknowledged and understood, will try your patch shortly. > > > > > > Did you test it? > > > > Experiment is in the queue, should have a result in a day or two. > > > Jens, your patch works! We are seeing a little bit increase in
Super. > indirect branch calls with your patch where our patch tries to remove > elevator_merge_fn() completely. But the difference is all within > noise range. Yeah that is expected. Thanks for testing! > If there is no other issues (I don't see any), we would like to see > this patch merged upstream. Thanks. I will pass it along. -- Jens Axboe - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/