[PATCH v3] mm: KSM: fix data type
From: Zhansaya Bagdauletkyzy <hidden>
Date: 2021-08-06 11:14:02
Also in:
lkml
Subsystem:
memory management, memory management - ksm (kernel samepage merging), the rest · Maintainers:
Andrew Morton, David Hildenbrand, Linus Torvalds
ksm_stable_node_chains_prune_millisecs is declared as int, but in stable__node_chains_prune_millisecs_store(), it can store values up to UINT_MAX. Change its type to unsigned int. Signed-off-by: Zhansaya Bagdauletkyzy <redacted> --- v2 -> v3: As suggested by Hugh Dickins, - discard changes to ksm_run v1 -> v2: - merge two patches into one v2: https://lore.kernel.org/lkml/20210716055800.GA255851@asus (local) v1: https://lkml.org/lkml/2021/7/15/866 mm/ksm.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/mm/ksm.c b/mm/ksm.c
index 3fa9bc8a67cf..025338128cd9 100644
--- a/mm/ksm.c
+++ b/mm/ksm.c@@ -259,7 +259,7 @@ static unsigned long ksm_stable_node_chains; static unsigned long ksm_stable_node_dups; /* Delay in pruning stale stable_node_dups in the stable_node_chains */ -static int ksm_stable_node_chains_prune_millisecs = 2000; +static unsigned int ksm_stable_node_chains_prune_millisecs = 2000; /* Maximum number of page slots sharing a stable node */ static int ksm_max_page_sharing = 256;
@@ -3105,11 +3105,11 @@ stable_node_chains_prune_millisecs_store(struct kobject *kobj, struct kobj_attribute *attr, const char *buf, size_t count) { - unsigned long msecs; + unsigned int msecs; int err; - err = kstrtoul(buf, 10, &msecs); - if (err || msecs > UINT_MAX) + err = kstrtouint(buf, 10, &msecs); + if (err) return -EINVAL; ksm_stable_node_chains_prune_millisecs = msecs;
--
2.25.1