| From: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| Subject: mm/damon: use kstrtobool() instead of strtobool() |
| Date: Tue, 1 Nov 2022 22:14:08 +0100 |
| |
| strtobool() is the same as kstrtobool(). However, the latter is more used |
| within the kernel. |
| |
| In order to remove strtobool() and slightly simplify kstrtox.h, switch to |
| the other function name. |
| |
| While at it, include the corresponding header file (<linux/kstrtox.h>) |
| |
| Link: https://lkml.kernel.org/r/ed2b46489a513988688decb53850339cc228940c.1667336095.git.christophe.jaillet@wanadoo.fr |
| Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr> |
| Reviewed-by: SeongJae Park <sj@kernel.org> |
| Signed-off-by: Andrew Morton <akpm@linux-foundation.org> |
| --- |
| |
| mm/damon/lru_sort.c | 3 ++- |
| mm/damon/reclaim.c | 3 ++- |
| 2 files changed, 4 insertions(+), 2 deletions(-) |
| |
| --- a/mm/damon/lru_sort.c~mm-damon-use-kstrtobool-instead-of-strtobool |
| +++ a/mm/damon/lru_sort.c |
| @@ -8,6 +8,7 @@ |
| #define pr_fmt(fmt) "damon-lru-sort: " fmt |
| |
| #include <linux/damon.h> |
| +#include <linux/kstrtox.h> |
| #include <linux/module.h> |
| |
| #include "modules-common.h" |
| @@ -241,7 +242,7 @@ static int damon_lru_sort_enabled_store( |
| bool enable; |
| int err; |
| |
| - err = strtobool(val, &enable); |
| + err = kstrtobool(val, &enable); |
| if (err) |
| return err; |
| |
| --- a/mm/damon/reclaim.c~mm-damon-use-kstrtobool-instead-of-strtobool |
| +++ a/mm/damon/reclaim.c |
| @@ -8,6 +8,7 @@ |
| #define pr_fmt(fmt) "damon-reclaim: " fmt |
| |
| #include <linux/damon.h> |
| +#include <linux/kstrtox.h> |
| #include <linux/module.h> |
| |
| #include "modules-common.h" |
| @@ -187,7 +188,7 @@ static int damon_reclaim_enabled_store(c |
| bool enable; |
| int err; |
| |
| - err = strtobool(val, &enable); |
| + err = kstrtobool(val, &enable); |
| if (err) |
| return err; |
| |
| _ |