lkml.org 
[lkml]   [2013]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] rwsem: steal writing sem for better performance

* Alex Shi <alex.shi@intel.com> wrote:

> On 02/05/2013 10:58 PM, Ingo Molnar wrote:
> >
> > * Alex Shi <alex.shi@intel.com> wrote:
> >
> >> Commit 5a50508 change to rwsem from mutex, that cause aim7
> >> fork_test dropped 50%. Yuanhan liu does a good analysis,
> >> find it caused by strict sequential writing. Ingo suggest
> >> stealing sem writing from front task in waiting queue.
> >> https://lkml.org/lkml/2013/1/29/84 So has this patch.
> >>
> >> In this patch, I just allow writing steal happen when the
> >> first waiter is also writer. Then the performance fully
> >> recovered.
> >
> > All the aim7 fork_test performance regression is recovered?
>
> Sure. 100% recovered on my NHM EP, NHM EX, SNB EP 2S and 4S
> machines.

That's totally awesome!

Linus, Andrew, what is your thinking about the patch and about
the timing of the patch?

It looks correct and straightforward to me, but rwsem races
tended to be tricky to find and fix in the past...

We are in -rc7 territory which is a bit late for fundamental
locking changes.

OTOH this is a regression fix, and not a small one.

I'll queue it up optimistically in an urgent branch for the time
being, to see what happens with a few days of testing ... If
there's any problems with it then it's clearly ineligible.

Thanks,

Ingo


\
 
 \ /
  Last update: 2013-02-06 13:42    [W:0.120 / U:22.128 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site