From: Dave Hansen <[email protected]>

This begins the work of moving the percpu pageset sysctl code
out of page_alloc.c.  update_all_zone_pageset_limits() is the
now the only interface that the sysctl code *really* needs out
of page_alloc.c.

This helps make it very clear what the interactions are between
the actual sysctl code and the core page alloc code.

Signed-off-by: Dave Hansen <[email protected]>
---

 linux.git-davehans/mm/page_alloc.c |   27 ++++++++++++++++-----------
 1 file changed, 16 insertions(+), 11 deletions(-)

diff -puN mm/page_alloc.c~separate-pageset-code-from-sysctl mm/page_alloc.c
--- linux.git/mm/page_alloc.c~separate-pageset-code-from-sysctl 2013-10-15 
09:57:06.415636275 -0700
+++ linux.git-davehans/mm/page_alloc.c  2013-10-15 09:57:06.421636541 -0700
@@ -5768,6 +5768,19 @@ int lowmem_reserve_ratio_sysctl_handler(
        return 0;
 }
 
+void update_all_zone_pageset_limits(void)
+{
+       struct zone *zone;
+       unsigned int cpu;
+
+       mutex_lock(&pcp_batch_high_lock);
+       for_each_populated_zone(zone)
+               for_each_possible_cpu(cpu)
+                       pageset_set_high_and_batch(zone,
+                                       per_cpu_ptr(zone->pageset, cpu));
+       mutex_unlock(&pcp_batch_high_lock);
+}
+
 /*
  * percpu_pagelist_fraction - changes the pcp->high for each zone on each
  * cpu.  It is the fraction of total pages in each zone that a hot per cpu
@@ -5776,20 +5789,12 @@ int lowmem_reserve_ratio_sysctl_handler(
 int percpu_pagelist_fraction_sysctl_handler(ctl_table *table, int write,
        void __user *buffer, size_t *length, loff_t *ppos)
 {
-       struct zone *zone;
-       unsigned int cpu;
-       int ret;
-
-       ret = proc_dointvec_minmax(table, write, buffer, length, ppos);
+       int ret = proc_dointvec_minmax(table, write, buffer, length, ppos);
        if (!write || (ret < 0))
                return ret;
 
-       mutex_lock(&pcp_batch_high_lock);
-       for_each_populated_zone(zone)
-               for_each_possible_cpu(cpu)
-                       pageset_set_high_and_batch(zone,
-                                       per_cpu_ptr(zone->pageset, cpu));
-       mutex_unlock(&pcp_batch_high_lock);
+       update_all_zone_pageset_limits();
+
        return 0;
 }
 
_
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to