On Mon, Feb 21, 2022 at 04:58:11PM -0800, Randy Dunlap <rdunlap@xxxxxxxxxxxxx> wrote: > __setup() handlers should return 1 if the command line option is handled > and 0 if not (or maybe never return 0; it just pollutes init's environment). Interesting. > Instead of relying on this '.' quirk, just return 1 to indicate that > the boot option has been handled. But your patch would return 1 even when no accepted value was passed, i.e. is the command line option considered handled in that case? Did you want to return 1 only when the cgroup.memory= value is recognized? Thanks, Michal
