Re: [PATCH v3 1/8] user_namespace: Use acquire/release for nr_extents synchronization

From: Andy Shevchenko

Date: Wed Sep 02 2026 - 09:36:04 EST


On Wed, Sep 02, 2026 at 04:03:23PM +0300, Andy Shevchenko wrote:
> On Wed, Sep 02, 2026 at 03:47:58PM +0800, Jinjie Ruan wrote:
> > Replace smp_wmb()/smp_rmb() with smp_store_release()/smp_load_acquire()
> > when publishing and consuming `nr_extents`. This expresses the
> > publish/subscribe pattern more clearly and allows architectures with
> > native acquire/release instructions (e.g. arm64's STLR/LDAR) to avoid
> > the cost of full one-way barriers (DMB ISHST/ISHLD).
> >
> > No functional change intended.
>
> Please, locate Cc list...
>
> > Cc: Alexander Viro <viro@xxxxxxxxxxxxxxxxxx>
> > Cc: Christian Brauner <brauner@xxxxxxxxxx>
> > Cc: Jan Kara <jack@xxxxxxx>
> > Cc: Seth Forshee <sforshee@xxxxxxxxxx>
> > Cc: Kees Cook <kees@xxxxxxxxxx>
> > Cc: Aleksa Sarai <cyphar@xxxxxxxxxx>o
>
> ...(all the above)...

Important remark: this is only about Cc, the regular tags have to be
in the commit message trailers (such as Reviewed-by).

> > Assisted-by: DeepSeek:DeepSeek-V3
> > Signed-off-by: Jinjie Ruan <ruanjinjie@xxxxxxxxxx>
> > ---
>
> ...somewhere here. It will have the same effect on the email thread and it will
> unload the commit message from the unneeded noise.
>
> > v2:
> > - Add missing load replace in copy_mnt_idmap()
> > ---
> > fs/mnt_idmapping.c | 5 ++---
> > kernel/user_namespace.c | 24 +++++++++++++-----------
> > 2 files changed, 15 insertions(+), 14 deletions(-)
>
> ...
>
> This comment applies to the whole series.
> Also note, `b4 trailers` can handle that, you can keep the
>
> ---
> Cc: ...
>
> in the local Git. You can find some details in the subthread here
> https://lore.kernel.org/lkml/20260123113708.416727-19-bigeasy@xxxxxxxxxxxxx/

--
With Best Regards,
Andy Shevchenko