Re: [PATCH] fs: Make file-nr output the total allocated file handles

From: Mateusz Guzik
Date: Fri Apr 11 2025 - 17:15:32 EST


On Fri, Apr 11, 2025 at 04:16:08PM +0200, Christian Brauner wrote:
> On Thu, Apr 10, 2025 at 07:21:17PM +0800, lirongqing wrote:
> > From: Li RongQing <lirongqing@xxxxxxxxx>
> >
> > Make file-nr output the total allocated file handles, not per-cpu
> > cache number, it's more precise, and not in hot path
> >
> > Signed-off-by: Li RongQing <lirongqing@xxxxxxxxx>
> > ---
>
> That means grabbing a lock suddenly. Is there an actual use-case
> behind this?
>

The centralized value can be really grossly inaccurate as CPU count increases.

There is some talks about fixing that, see:
https://lore.kernel.org/linux-mm/20250410175149.1206995-1-mathieu.desnoyers@xxxxxxxxxxxx/

In the meantime, given that this is only accessed when reading the /proc
file, this should be fine?

Note it still wont delay bumps/decs as long as they fit the batch (which
is the common case).