Commit eb7235eb authored by Alexey Dobriyan's avatar Alexey Dobriyan Committed by Linus Torvalds

slub: make ->remote_node_defrag_ratio unsigned int

->remote_node_defrag_ratio is in range 0..1000.

This also adds a check and modifies the behavior to return an error
code.  Before this patch invalid values were ignored.

Link: http://lkml.kernel.org/r/20180305200730.15812-9-adobriyan@gmail.comSigned-off-by: default avatarAlexey Dobriyan <adobriyan@gmail.com>
Acked-by: default avatarChristoph Lameter <cl@linux.com>
Cc: Pekka Enberg <penberg@kernel.org>
Cc: David Rientjes <rientjes@google.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent ac914d08
...@@ -124,7 +124,7 @@ struct kmem_cache { ...@@ -124,7 +124,7 @@ struct kmem_cache {
/* /*
* Defragmentation by allocating from a remote node. * Defragmentation by allocating from a remote node.
*/ */
int remote_node_defrag_ratio; unsigned int remote_node_defrag_ratio;
#endif #endif
#ifdef CONFIG_SLAB_FREELIST_RANDOM #ifdef CONFIG_SLAB_FREELIST_RANDOM
......
...@@ -5289,20 +5289,21 @@ SLAB_ATTR(shrink); ...@@ -5289,20 +5289,21 @@ SLAB_ATTR(shrink);
#ifdef CONFIG_NUMA #ifdef CONFIG_NUMA
static ssize_t remote_node_defrag_ratio_show(struct kmem_cache *s, char *buf) static ssize_t remote_node_defrag_ratio_show(struct kmem_cache *s, char *buf)
{ {
return sprintf(buf, "%d\n", s->remote_node_defrag_ratio / 10); return sprintf(buf, "%u\n", s->remote_node_defrag_ratio / 10);
} }
static ssize_t remote_node_defrag_ratio_store(struct kmem_cache *s, static ssize_t remote_node_defrag_ratio_store(struct kmem_cache *s,
const char *buf, size_t length) const char *buf, size_t length)
{ {
unsigned long ratio; unsigned int ratio;
int err; int err;
err = kstrtoul(buf, 10, &ratio); err = kstrtouint(buf, 10, &ratio);
if (err) if (err)
return err; return err;
if (ratio > 100)
return -ERANGE;
if (ratio <= 100)
s->remote_node_defrag_ratio = ratio * 10; s->remote_node_defrag_ratio = ratio * 10;
return length; return length;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment