diff options
author | David S. Miller <davem@sunset.davemloft.net> | 2007-07-15 04:08:03 -0400 |
---|---|---|
committer | David S. Miller <davem@sunset.davemloft.net> | 2007-07-16 07:05:15 -0400 |
commit | b37d40d1759a383208ef47be8af94ebe59031691 (patch) | |
tree | 32870d4ce58147a4c8ae1e90dc115e9ae79db436 /arch/sparc64/kernel/smp.c | |
parent | b53bcb6799f8f2c0f385f24a8bb054f142c7d83a (diff) |
[SPARC64]: Fix leak when DR added cpu does not bootup.
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'arch/sparc64/kernel/smp.c')
-rw-r--r-- | arch/sparc64/kernel/smp.c | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/arch/sparc64/kernel/smp.c b/arch/sparc64/kernel/smp.c index 69a1183c622d..6956845b03fa 100644 --- a/arch/sparc64/kernel/smp.c +++ b/arch/sparc64/kernel/smp.c | |||
@@ -90,7 +90,6 @@ static volatile unsigned long callin_flag = 0; | |||
90 | void __devinit smp_callin(void) | 90 | void __devinit smp_callin(void) |
91 | { | 91 | { |
92 | int cpuid = hard_smp_processor_id(); | 92 | int cpuid = hard_smp_processor_id(); |
93 | struct trap_per_cpu *tb = &trap_block[cpuid];; | ||
94 | 93 | ||
95 | __local_per_cpu_offset = __per_cpu_offset(cpuid); | 94 | __local_per_cpu_offset = __per_cpu_offset(cpuid); |
96 | 95 | ||
@@ -119,11 +118,6 @@ void __devinit smp_callin(void) | |||
119 | atomic_inc(&init_mm.mm_count); | 118 | atomic_inc(&init_mm.mm_count); |
120 | current->active_mm = &init_mm; | 119 | current->active_mm = &init_mm; |
121 | 120 | ||
122 | if (tb->hdesc) { | ||
123 | kfree(tb->hdesc); | ||
124 | tb->hdesc = NULL; | ||
125 | } | ||
126 | |||
127 | while (!cpu_isset(cpuid, smp_commenced_mask)) | 121 | while (!cpu_isset(cpuid, smp_commenced_mask)) |
128 | rmb(); | 122 | rmb(); |
129 | 123 | ||
@@ -345,6 +339,7 @@ static struct thread_info *cpu_new_thread = NULL; | |||
345 | 339 | ||
346 | static int __devinit smp_boot_one_cpu(unsigned int cpu) | 340 | static int __devinit smp_boot_one_cpu(unsigned int cpu) |
347 | { | 341 | { |
342 | struct trap_per_cpu *tb = &trap_block[cpu]; | ||
348 | unsigned long entry = | 343 | unsigned long entry = |
349 | (unsigned long)(&sparc64_cpu_startup); | 344 | (unsigned long)(&sparc64_cpu_startup); |
350 | unsigned long cookie = | 345 | unsigned long cookie = |
@@ -389,6 +384,11 @@ static int __devinit smp_boot_one_cpu(unsigned int cpu) | |||
389 | } | 384 | } |
390 | cpu_new_thread = NULL; | 385 | cpu_new_thread = NULL; |
391 | 386 | ||
387 | if (tb->hdesc) { | ||
388 | kfree(tb->hdesc); | ||
389 | tb->hdesc = NULL; | ||
390 | } | ||
391 | |||
392 | return ret; | 392 | return ret; |
393 | } | 393 | } |
394 | 394 | ||