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 | |
| parent | b53bcb6799f8f2c0f385f24a8bb054f142c7d83a (diff) | |
[SPARC64]: Fix leak when DR added cpu does not bootup.
Signed-off-by: David S. Miller <davem@davemloft.net>
| -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 | ||
