Re: Name hashing function causing a perf regression

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Sep 15, 2014 at 8:55 AM, Josef Bacik <jbacik@xxxxxx> 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
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]