Re: [PATCH 03/12] mm: shmem: don't expose 'deny' and 'force' via the kernel command line

[Baolin Wang]

On 9/18/26 1:45 PM, Barry Song wrote:
> On Fri, Sep 18, 2026 at 11:58 AM Baolin Wang
> <baolin.wang@xxxxxxxxxxxxxxxxx> wrote:
> > Based on the previous discussion with Lorenzo[1], the 'deny' and 'force'
> > options are testing-only configurations and shouldn't be exposed via the
> > shmem kernel command line.
> >
> > So drop these debug options from the shmem kernel command line and update
> > the documentation to make this clear.
>
> If it is a debug option, it should not be exposed through sysfs :-)

Ah, let's keep it compatible for now.

> Maybe we could rephrase this to say that it can only be switched at
> runtime through the `shmem_enabled` sysfs interface?

After the changes to transhuge.rst, this is essentially what it means. 
Anyway, I can make it more explicit in the commit message.

> > [1] https://lore.kernel.org/all/6e4aae7b-2c71-4109-945c-6481efde3636@xxxxxxxxxxxxxxxxx/
> > Suggested-by: Lorenzo Stoakes (ARM) <ljs@xxxxxxxxxx>
> > Signed-off-by: Baolin Wang <baolin.wang@xxxxxxxxxxxxxxxxx>
> > ---
> [...]
> > +static int shmem_parse_huge(const char *str, bool skip_deny_force)
> >   {
>
> The name is a bit weird. What about `allow_deny_force`?

Sounds good. Will do if no other objections.

Thanks for reviewing.