diff options
author | David Rientjes <rientjes@google.com> | 2014-08-06 19:06:54 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2014-08-06 21:01:19 -0400 |
commit | ed4d4902ebdd7ca8b5a51daaf6bebf4b172895cc (patch) | |
tree | d2da9377a11a90047d640c554220a3cfb4d1249c | |
parent | 238d3c13f0cce38752072dc90f4e828abdfec143 (diff) |
mm, hugetlb: remove hugetlb_zero and hugetlb_infinity
They are unnecessary: "zero" can be used in place of "hugetlb_zero" and
passing extra2 == NULL is equivalent to infinity.
Signed-off-by: David Rientjes <rientjes@google.com>
Cc: Joonsoo Kim <iamjoonsoo.kim@lge.com>
Reviewed-by: Naoya Horiguchi <n-horiguchi@ah.jp.nec.com>
Reviewed-by: Luiz Capitulino <lcapitulino@redhat.com>
Cc: "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r-- | include/linux/hugetlb.h | 1 | ||||
-rw-r--r-- | kernel/sysctl.c | 9 | ||||
-rw-r--r-- | mm/hugetlb.c | 1 |
3 files changed, 3 insertions, 8 deletions
diff --git a/include/linux/hugetlb.h b/include/linux/hugetlb.h index a23c096b3080..6e6d338641fe 100644 --- a/include/linux/hugetlb.h +++ b/include/linux/hugetlb.h | |||
@@ -87,7 +87,6 @@ pte_t *huge_pmd_share(struct mm_struct *mm, unsigned long addr, pud_t *pud); | |||
87 | #endif | 87 | #endif |
88 | 88 | ||
89 | extern unsigned long hugepages_treat_as_movable; | 89 | extern unsigned long hugepages_treat_as_movable; |
90 | extern const unsigned long hugetlb_zero, hugetlb_infinity; | ||
91 | extern int sysctl_hugetlb_shm_group; | 90 | extern int sysctl_hugetlb_shm_group; |
92 | extern struct list_head huge_boot_pages; | 91 | extern struct list_head huge_boot_pages; |
93 | 92 | ||
diff --git a/kernel/sysctl.c b/kernel/sysctl.c index 75b22e22a72c..75875a741b5e 100644 --- a/kernel/sysctl.c +++ b/kernel/sysctl.c | |||
@@ -1240,8 +1240,7 @@ static struct ctl_table vm_table[] = { | |||
1240 | .maxlen = sizeof(unsigned long), | 1240 | .maxlen = sizeof(unsigned long), |
1241 | .mode = 0644, | 1241 | .mode = 0644, |
1242 | .proc_handler = hugetlb_sysctl_handler, | 1242 | .proc_handler = hugetlb_sysctl_handler, |
1243 | .extra1 = (void *)&hugetlb_zero, | 1243 | .extra1 = &zero, |
1244 | .extra2 = (void *)&hugetlb_infinity, | ||
1245 | }, | 1244 | }, |
1246 | #ifdef CONFIG_NUMA | 1245 | #ifdef CONFIG_NUMA |
1247 | { | 1246 | { |
@@ -1250,8 +1249,7 @@ static struct ctl_table vm_table[] = { | |||
1250 | .maxlen = sizeof(unsigned long), | 1249 | .maxlen = sizeof(unsigned long), |
1251 | .mode = 0644, | 1250 | .mode = 0644, |
1252 | .proc_handler = &hugetlb_mempolicy_sysctl_handler, | 1251 | .proc_handler = &hugetlb_mempolicy_sysctl_handler, |
1253 | .extra1 = (void *)&hugetlb_zero, | 1252 | .extra1 = &zero, |
1254 | .extra2 = (void *)&hugetlb_infinity, | ||
1255 | }, | 1253 | }, |
1256 | #endif | 1254 | #endif |
1257 | { | 1255 | { |
@@ -1274,8 +1272,7 @@ static struct ctl_table vm_table[] = { | |||
1274 | .maxlen = sizeof(unsigned long), | 1272 | .maxlen = sizeof(unsigned long), |
1275 | .mode = 0644, | 1273 | .mode = 0644, |
1276 | .proc_handler = hugetlb_overcommit_handler, | 1274 | .proc_handler = hugetlb_overcommit_handler, |
1277 | .extra1 = (void *)&hugetlb_zero, | 1275 | .extra1 = &zero, |
1278 | .extra2 = (void *)&hugetlb_infinity, | ||
1279 | }, | 1276 | }, |
1280 | #endif | 1277 | #endif |
1281 | { | 1278 | { |
diff --git a/mm/hugetlb.c b/mm/hugetlb.c index 7a0fcb33973e..d9ad93b55585 100644 --- a/mm/hugetlb.c +++ b/mm/hugetlb.c | |||
@@ -35,7 +35,6 @@ | |||
35 | #include <linux/node.h> | 35 | #include <linux/node.h> |
36 | #include "internal.h" | 36 | #include "internal.h" |
37 | 37 | ||
38 | const unsigned long hugetlb_zero = 0, hugetlb_infinity = ~0UL; | ||
39 | unsigned long hugepages_treat_as_movable; | 38 | unsigned long hugepages_treat_as_movable; |
40 | 39 | ||
41 | int hugetlb_max_hstate __read_mostly; | 40 | int hugetlb_max_hstate __read_mostly; |