Messages in this thread Patch in this message | | | Date | Mon, 08 Jul 2013 16:50:24 +0800 | From | Chen Gang <> | Subject | [PATCH] kernel/smp.c: free related resources when failure occurs in hotplug_cfd() |
| |
When failure occurs in hotplug_cfd(), need release related resources, or will cause memory leak.
Also beautify the related code.
Signed-off-by: Chen Gang <gang.chen@asianux.com> --- kernel/smp.c | 13 +++++++++---- 1 files changed, 9 insertions(+), 4 deletions(-)
diff --git a/kernel/smp.c b/kernel/smp.c index 02a885d..c264623 100644 --- a/kernel/smp.c +++ b/kernel/smp.c @@ -45,15 +45,20 @@ hotplug_cfd(struct notifier_block *nfb, unsigned long action, void *hcpu) switch (action) { case CPU_UP_PREPARE: case CPU_UP_PREPARE_FROZEN: - if (!zalloc_cpumask_var_node(&cfd->cpumask, GFP_KERNEL, - cpu_to_node(cpu))) + if (!zalloc_cpumask_var_node(&cfd->cpumask, + GFP_KERNEL, cpu_to_node(cpu))) return notifier_from_errno(-ENOMEM); - if (!zalloc_cpumask_var_node(&cfd->cpumask_ipi, GFP_KERNEL, - cpu_to_node(cpu))) + + if (!zalloc_cpumask_var_node(&cfd->cpumask_ipi, + GFP_KERNEL, cpu_to_node(cpu))) { + free_cpumask_var(cfd->cpumask); return notifier_from_errno(-ENOMEM); + } + cfd->csd = alloc_percpu(struct call_single_data); if (!cfd->csd) { free_cpumask_var(cfd->cpumask); + free_cpumask_var(cfd->cpumask_ipi); return notifier_from_errno(-ENOMEM); } break; -- 1.7.7.6
| |