mm-add-extra-free-kbytes-tunable-update

All the fixes suggested by Andrew Morton.   Not much of a changelog
since the patch should probably be folded into
mm-add-extra-free-kbytes-tunable.patch

Thank you for pointing these out, Andrew.

Signed-off-by: Rik van Riel <riel@redhat.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>

diff --git a/include/linux/mmzone.h b/include/linux/mmzone.h
index dff711509661c8e5eb74fce083eef26e526a783e..f10a54c35b7a8ee4cb29e88648446eb7832d353b 100644 (file)
--- a/include/linux/mmzone.h
+++ b/include/linux/mmzone.h
@@ -796,7 +796,7 @@ static inline int is_dma(struct zone *zone)
 
 /* These two functions are used to setup the per zone pages min values */
 struct ctl_table;
-int min_free_kbytes_sysctl_handler(struct ctl_table *, int,
+int free_kbytes_sysctl_handler(struct ctl_table *, int,
                                        void __user *, size_t *, loff_t *);
 extern int sysctl_lowmem_reserve_ratio[MAX_NR_ZONES-1];
 int lowmem_reserve_ratio_sysctl_handler(struct ctl_table *, int,

diff --git a/include/linux/swap.h b/include/linux/swap.h
index a1f5d74f65d150d8ba711cf7c961d3bbb5606a45..fbe8473c37207f037a0ab4c8d92a8052866729a0 100644 (file)
--- a/include/linux/swap.h
+++ b/include/linux/swap.h
@@ -215,6 +215,8 @@ struct swap_list_t {
 extern unsigned long totalram_pages;
 extern unsigned long totalreserve_pages;
 extern unsigned long dirty_balance_reserve;
+extern int min_free_kbytes;
+extern int extra_free_kbytes;
 extern unsigned int nr_free_buffer_pages(void);
 extern unsigned int nr_free_pagecache_pages(void);
 

diff --git a/kernel/sysctl.c b/kernel/sysctl.c
index 38e0e70f008de048948b7dc789363651b2857135..84f81c77373d6df79fb956ab3934e2ba36df6874 100644 (file)
--- a/kernel/sysctl.c
+++ b/kernel/sysctl.c
@@ -101,8 +101,6 @@ extern int suid_dumpable;
 extern char core_pattern[];
 extern unsigned int core_pipe_limit;
 extern int pid_max;
-extern int min_free_kbytes;
-extern int extra_free_kbytes;
 extern int pid_max_min, pid_max_max;
 extern int sysctl_drop_caches;
 extern int percpu_pagelist_fraction;
@@ -1196,7 +1194,7 @@ static struct ctl_table vm_table[] = {
                .data           = &min_free_kbytes,
                .maxlen         = sizeof(min_free_kbytes),
                .mode           = 0644,
-               .proc_handler   = min_free_kbytes_sysctl_handler,
+               .proc_handler   = free_kbytes_sysctl_handler,
                .extra1         = &zero,
        },
        {
@@ -1204,7 +1202,7 @@ static struct ctl_table vm_table[] = {
                .data           = &extra_free_kbytes,
                .maxlen         = sizeof(extra_free_kbytes),
                .mode           = 0644,
-               .proc_handler   = min_free_kbytes_sysctl_handler,
+               .proc_handler   = free_kbytes_sysctl_handler,
                .extra1         = &zero,
        },
        {

diff --git a/mm/page_alloc.c b/mm/page_alloc.c
index decfbf0f71c2a10e29b6346aa7451a64b61a890f..14fda9e57f1d2424ecc1d7e7f7d115a82ff80ef4 100644 (file)
--- a/mm/page_alloc.c
+++ b/mm/page_alloc.c
@@ -199,11 +199,12 @@ static char * const zone_names[MAX_NR_ZONES] = {
 int min_free_kbytes = 1024;
 
 /*
- * Extra memory for the system to try freeing. Used to temporarily
- * free memory, to make space for new workloads. Anyone can allocate
- * down to the min watermarks controlled by min_free_kbytes above.
+ * Extra memory for the system to try freeing between the min and
+ * low watermarks.  Useful for workloads that require low latency
+ * memory allocations in bursts larger than the normal gap between
+ * low and min.
  */
-int extra_free_kbytes = 0;
+int extra_free_kbytes;
 
 static unsigned long __meminitdata nr_kernel_pages;
 static unsigned long __meminitdata nr_all_pages;
@@ -5143,11 +5144,11 @@ int __meminit init_per_zone_wmark_min(void)
 module_init(init_per_zone_wmark_min)
 
 /*
- * min_free_kbytes_sysctl_handler - just a wrapper around proc_dointvec() so 
+ * free_kbytes_sysctl_handler - just a wrapper around proc_dointvec() so 
  *     that we can call two helper functions whenever min_free_kbytes
  *     or extra_free_kbytes changes.
  */
-int min_free_kbytes_sysctl_handler(ctl_table *table, int write, 
+int free_kbytes_sysctl_handler(ctl_table *table, int write, 
        void __user *buffer, size_t *length, loff_t *ppos)
 {
        proc_dointvec(table, write, buffer, length, ppos);