Re: [RFC]block: add flush request at head

[Tejun Heo]

Hey,

On Tue, Apr 26, 2011 at 08:49:15AM +0800, Shaohua Li wrote:
> On Mon, 2011-04-25 at 16:21 +0800, Tejun Heo wrote:
> > Hello, Shaohua.
> > 
> > On Mon, Apr 25, 2011 at 09:01:59AM +0800, Shaohua Li wrote:
> > > this is a regression from 2.6.39-rc2 compared to 2.6.39-rc1, so this
> > > isn't related to the flush rewritten. Workload is sysbench fileio,
> > > please see the first mail at the thread for detail.
> > 
> > Understood.  Let's talk on the other thread.
>
> This issue isn't related to the optimization patch in another thread.
> And that patch can't recover the regression, which does improve
> throughput even without the regression. So please look at issue again.

IIUC, the regression happened because, before, back-to-back flushes
were basically optimized out by hardware but, after, due to regular
writes thrown into the mix, aren't.  If that's the case, I would still
prefer to solve this from issue side instead of completion if possible
(it might not be tho).

Or is the latency introduced for each flush actually making difference
for the specific benchmark?  Hmmm... maybe that's the case given that
your patches merging back-to-back flushes doesn't recover the whole
regression.

I don't know.  Darrick, can you please chime in?  Do you see
regression between front and back queueing of flushes?  The original
thread is

 http://thread.gmane.org/gmane.linux.kernel/1127779

and the offending commit is 53d63e6b0dfb95882ec0219ba6bbd50cde423794
(block: make the flush insertion use the tail of the dispatch list).

Thanks.

-- 
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/