Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit a9919c79 authored by Rasmus Villemoes's avatar Rasmus Villemoes Committed by Linus Torvalds
Browse files

mm: only define hashdist variable when needed 



For !CONFIG_NUMA, hashdist will always be 0, since it's setter is
otherwise compiled out.  So we can save 4 bytes of data and some .text
(although mostly in __init functions) by only defining it for
CONFIG_NUMA.

Signed-off-by: default avatarRasmus Villemoes <linux@rasmusvillemoes.dk>
Acked-by: default avatarDavid Rientjes <rientjes@google.com>
Reviewed-by: default avatarMichal Hocko <mhocko@suse.cz>
Signed-off-by: default avatarAndrew Morton <akpm@linux-foundation.org>
Signed-off-by: default avatarLinus Torvalds <torvalds@linux-foundation.org>
parent a67a31fa

include/linux/bootmem.h

+4 −4
Original line number Diff line number Diff line
@@ -357,12 +357,12 @@ extern void *alloc_large_system_hash(const char *tablename, 
/* Only NUMA needs hash distribution. 64bit NUMA architectures have

 * sufficient vmalloc space.

 */

#if defined(CONFIG_NUMA) && defined(CONFIG_64BIT)

#define HASHDIST_DEFAULT 1

#ifdef CONFIG_NUMA

#define HASHDIST_DEFAULT IS_ENABLED(CONFIG_64BIT)

extern int hashdist;		/* Distribute hashes across NUMA nodes? */

#else

#define HASHDIST_DEFAULT 0

#define hashdist (0)

#endif

extern int hashdist;		/* Distribute hashes across NUMA nodes? */





#endif /* _LINUX_BOOTMEM_H */

mm/page_alloc.c

+1 −1
Original line number Diff line number Diff line
@@ -6013,9 +6013,9 @@ int percpu_pagelist_fraction_sysctl_handler(struct ctl_table *table, int write, 
	return ret;

}



#ifdef CONFIG_NUMA

int hashdist = HASHDIST_DEFAULT;



#ifdef CONFIG_NUMA

static int __init set_hashdist(char *str)

{

	if (!str)