percpu: fix trivial bugs in pcpu_build_alloc_info()
authorPavel V. Panteleev <pp_84@mail.ru>
Thu, 17 Jun 2010 08:07:25 +0000 (10:07 +0200)
committerTejun Heo <tj@kernel.org>
Thu, 17 Jun 2010 08:07:25 +0000 (10:07 +0200)
Fix the following two trivial bugs in pcpu_build_alloc_info()

* we should memset group_cnt to 0 by size of group_cnt, not size of
  group_map (both are of the same size, so the bug isn't dangerous)

* we can delete useless variable group_cnt_max.

Signed-off-by: Pavel V. Panteleev <pp_84@mail.ru>
Signed-off-by: Tejun Heo <tj@kernel.org>
mm/percpu.c

index 39f7dfd59585a118f4a8c6be480bf3b866a50f25..46485e1b26fccf4b609ad0b39c5353105fed66fb 100644 (file)
@@ -1086,7 +1086,7 @@ struct pcpu_alloc_info * __init pcpu_build_alloc_info(
        static int group_map[NR_CPUS] __initdata;
        static int group_cnt[NR_CPUS] __initdata;
        const size_t static_size = __per_cpu_end - __per_cpu_start;
-       int group_cnt_max = 0, nr_groups = 1, nr_units = 0;
+       int nr_groups = 1, nr_units = 0;
        size_t size_sum, min_unit_size, alloc_size;
        int upa, max_upa, uninitialized_var(best_upa);  /* units_per_alloc */
        int last_allocs, group, unit;
@@ -1096,7 +1096,7 @@ struct pcpu_alloc_info * __init pcpu_build_alloc_info(
 
        /* this function may be called multiple times */
        memset(group_map, 0, sizeof(group_map));
-       memset(group_cnt, 0, sizeof(group_map));
+       memset(group_cnt, 0, sizeof(group_cnt));
 
        /*
         * Determine min_unit_size, alloc_size and max_upa such that
@@ -1130,7 +1130,6 @@ struct pcpu_alloc_info * __init pcpu_build_alloc_info(
                }
                group_map[cpu] = group;
                group_cnt[group]++;
-               group_cnt_max = max(group_cnt_max, group_cnt[group]);
        }
 
        /*