Re: Performance regression from switching lock to rw-sem foranon-vma tree

From: Ingo Molnar
Date: Tue Jul 23 2013 - 05:53:20 EST



* Peter Zijlstra <peterz@xxxxxxxxxxxxx> wrote:

> On Tue, Jul 23, 2013 at 11:45:13AM +0200, Ingo Molnar wrote:
>
> > Why not just try the delayed addition approach first? The spinning is
> > time limited AFAICS, so we don't _have to_ recognize those as writers
> > per se, only if the spinning fails and it wants to go on the waitlist.
> > Am I missing something?
> >
> > It will change patterns, it might even change the fairness balance -
> > but is a legit change otherwise, especially if it helps performance.
>
> Be very careful here. Some people (XFS) have very specific needs. Walken
> and dchinner had a longish discussion on this a while back.

Agreed - yet it's worth at least trying it out the quick way, to see the
main effect and to see whether that explains the performance assymetry and
invest more effort into it.

Thanks,

Ingo
--
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/