mm: page_owner: use kstrtobool() to parse bool option
authorSergei Trofimovich <slyfox@gentoo.org>
Fri, 30 Apr 2021 05:55:05 +0000 (22:55 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 30 Apr 2021 18:20:36 +0000 (11:20 -0700)
I tried to use page_owner=1 for a while noticed too late it had no effect
as opposed to similar init_on_alloc=1 (these work).

Let's make them consistent.

The change decreses binary size slightly:
   text    data     bss     dec     hex filename
  12408     321      17   12746    31ca mm/page_owner.o.before
  12320     321      17   12658    3172 mm/page_owner.o.after

Link: https://lkml.kernel.org/r/20210401210909.3532086-1-slyfox@gentoo.org
Signed-off-by: Sergei Trofimovich <slyfox@gentoo.org>
Reviewed-by: Matthew Wilcox (Oracle) <willy@infradead.org>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/page_owner.c

index 590c2c89af744c401e038f6daec5225c391e5b17..5c941ca9861d123f2b33cd6a8b646c0945201a24 100644 (file)
@@ -42,13 +42,7 @@ static void init_early_allocated_pages(void);
 
 static int __init early_page_owner_param(char *buf)
 {
-       if (!buf)
-               return -EINVAL;
-
-       if (strcmp(buf, "on") == 0)
-               page_owner_enabled = true;
-
-       return 0;
+       return kstrtobool(buf, &page_owner_enabled);
 }
 early_param("page_owner", early_page_owner_param);