Re: [PATCH] mm/memcontrol: use kstrtobool for swapaccount param parsing

From: Andrew Morton
Date: Thu Sep 08 2022 - 18:03:05 EST


On Thu, 8 Sep 2022 16:34:52 +0800 Liu Shixin <liushixin2@xxxxxxxxxx> wrote:

> --- a/mm/swap_cgroup.c
> +++ b/mm/swap_cgroup.c
> @@ -194,7 +194,7 @@ int swap_cgroup_swapon(int type, unsigned long max_pages)
> return 0;
> nomem:
> pr_info("couldn't allocate enough memory for swap_cgroup\n");
> - pr_info("swap_cgroup can be disabled by swapaccount=0 boot option\n");
> + pr_info("swap_cgroup can be disabled by swapaccount=[oO][Ff]/N/n/0 boot option\n");

I'm not sure this really needed changing. "=0" was OK and the message
now looks rather silly. But whatever.