Re: [PATCH] fs: mark lookup_slow() as noinline

[Christian Brauner <brauner@kernel.org>]

On Tue, Nov 25, 2025 at 10:54:25AM +0100, Mateusz Guzik wrote:
> On Tue, Nov 25, 2025 at 10:05 AM Christian Brauner <brauner@kernel.org> wrote:
> >
> > On Wed, Nov 19, 2025 at 03:49:30PM +0100, Mateusz Guzik wrote:
> > > Otherwise it gets inlined notably in walk_component(), which convinces
> > > the compiler to push/pop additional registers in the fast path to
> > > accomodate existence of the inlined version.
> > >
> > > Shortens the fast path of that routine from 87 to 71 bytes.
> > >
> > > Signed-off-by: Mateusz Guzik <mjguzik@gmail.com>
> > > ---
> >
> > Fwiw, the biggest problem is that we need to end up with something
> > obvious so that we don't accidently destroy any potential performance
> > gain in say 2 years because everyone forgot why we did things this way.
> >
> 
> I don't think there is a way around reviewing patches with an eye on
> what they are doing to the fast path, on top of periodically checking
> if whatever is considered the current compiler decided to start doing
> something funky with it.
> 
> I'm going to save a rant about benchmarking changes like these in the
> current kernel for another day.

Without knocking any tooling that we currently have but I don't think we
have _meaningful_ performance testing - especially not automated.