[lkml]   [2012]   [Feb]   [7]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH] block: strip out locking optimization in put_io_context()
On 2012-02-07 17:22, Tejun Heo wrote:
> Hello, Jens.
> On Tue, Feb 07, 2012 at 07:49:19AM +0100, Jens Axboe wrote:
>>> I couldn't find any statiscally meaningful advantage of the
>>> optimization with tight fork/exit tests w/ forced ioc creation on
>>> fork, which gotta be the most pathological test case for the code
>>> path. So, let's remove the ugly optimization. If I missed sth, we
>>> can resurrect the simpler optimization later. Jens, this is on top of
>>> linus#master without Shaohua's patch.
>> OK, then I'm fine with cleaning it up. Applied, thanks Tejun.
> Hmmm... how about merging Shaohua's smaller fix first until we figure
> out what's going on with the performance regression he's seeing?

That was already merged in my tree. I don't see how it makes much
difference in tracking the regression. You said that removing it made no
difference for the find test case, so I'd be more comfortable getting
rid of the nasty optimization.

I'll send it when I have pending tomorrow, so there's still a full day
to change things. We can just shuffle patches before then, not a

Jens Axboe

 \ /
  Last update: 2012-02-07 17:31    [W:0.103 / U:0.132 seconds]
©2003-2018 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site