Re: Name hashing function causing a perf regression

[Linus Torvalds <torvalds@linux-foundation.org>]

On Mon, Sep 15, 2014 at 8:55 AM, Josef Bacik <jbacik@fb.com> wrote:
>
> I can't test on 3.17 proper since the Fusion IO driver doesn't build
> properly there and I'm not being paid to work on it anymore so I'm not
> fixing it ;).  Thanks for fixing this, I've pulled back 99d263d4c5b2 which
> will do us just fine.

Ok. I'm cc'ing stable to know that

    99d263d4c5b2 ("vfs: fix bad hashing of dentries")

should be added to the queues for 3.10+.

Greg&co - it's a simple fix for a performance regression. Not
end-of-the-world, but if it ends up being in the FB kernel trees,
might as well get it back-ported to the other stable trees too.

The other performance issues I found are actually potentially worse,
but they aren't regressions and they hit only when using namespaces.
Which mostly nobody does on old kernels anyway. More of a "systemd
uses namespaces for /tmp, and then it's quite noticeable as slowing
down pathname lookups there if you benchmark it".

So the single commit Josef mentions is likely sufficient for stable.

               Linus