diff options
author | Thomas Renninger <trenn@suse.de> | 2007-05-02 13:27:22 -0400 |
---|---|---|
committer | Andi Kleen <andi@basil.nowhere.org> | 2007-05-02 13:27:22 -0400 |
commit | 35060b6a9a4e1c89bc6fbea61090e302dbc61847 (patch) | |
tree | 4f9ceb9b30e87799a011413523a332bc6bb262a8 /arch/i386/kernel/cpu/amd.c | |
parent | a3193348d407baaa7aef79decfa0e9a7fef74a17 (diff) |
[PATCH] i386: Don't delete cpu_devs data to identify different x86 types in late_initcall
In arch/i386/cpu/common.c there is:
cpu_devs[X86_VENDOR_INTEL]
cpu_devs[X86_VENDOR_CYRIX]
cpu_devs[X86_VENDOR_AMD]
...
They are all filled with data early.
The data (struct) got set to NULL for all, but Intel in different
late_initcall (exit_cpu_vendor) calls.
I don't see what sense this makes at all, maybe something that got
forgotten with the HOTPLUG_CPU extenstions?
Please check/review whether initdata, cpuinitdata is still ok and this
still works with HOTPLUG_CPU and without, it should...
Signed-off-by: Thomas Renninger <trenn@suse.de>
Signed-off-by: Andi Kleen <ak@suse.de>
Cc: davej@redhat.com
Diffstat (limited to 'arch/i386/kernel/cpu/amd.c')
-rw-r--r-- | arch/i386/kernel/cpu/amd.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/arch/i386/kernel/cpu/amd.c b/arch/i386/kernel/cpu/amd.c index 197cda62caa3..4fec702afd7e 100644 --- a/arch/i386/kernel/cpu/amd.c +++ b/arch/i386/kernel/cpu/amd.c | |||
@@ -319,13 +319,3 @@ int __init amd_init_cpu(void) | |||
319 | cpu_devs[X86_VENDOR_AMD] = &amd_cpu_dev; | 319 | cpu_devs[X86_VENDOR_AMD] = &amd_cpu_dev; |
320 | return 0; | 320 | return 0; |
321 | } | 321 | } |
322 | |||
323 | //early_arch_initcall(amd_init_cpu); | ||
324 | |||
325 | static int __init amd_exit_cpu(void) | ||
326 | { | ||
327 | cpu_devs[X86_VENDOR_AMD] = NULL; | ||
328 | return 0; | ||
329 | } | ||
330 | |||
331 | late_initcall(amd_exit_cpu); | ||