diff options
author | Eric W. Biederman <ebiederm@xmission.com> | 2005-07-26 13:36:01 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2005-07-26 17:35:42 -0400 |
commit | 59586e5a262a29361c45c929ea3253d4aec830b0 (patch) | |
tree | 1219b4943bc04515973a44c6e3a184e4f6f026f0 /arch/i386 | |
parent | 16dcb4bbda579c4e3d80048b755ac124d8fab21a (diff) |
[PATCH] Don't export machine_restart, machine_halt, or machine_power_off.
machine_restart, machine_halt and machine_power_off are machine
specific hooks deep into the reboot logic, that modules
have no business messing with. Usually code should be calling
kernel_restart, kernel_halt, kernel_power_off, or
emergency_restart. So don't export machine_restart,
machine_halt, and machine_power_off so we can catch buggy users.
Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
Diffstat (limited to 'arch/i386')
-rw-r--r-- | arch/i386/kernel/reboot.c | 5 | ||||
-rw-r--r-- | arch/i386/mach-visws/reboot.c | 5 | ||||
-rw-r--r-- | arch/i386/mach-voyager/voyager_basic.c | 5 |
3 files changed, 0 insertions, 15 deletions
diff --git a/arch/i386/kernel/reboot.c b/arch/i386/kernel/reboot.c index b3e584849961..1b24bc7271af 100644 --- a/arch/i386/kernel/reboot.c +++ b/arch/i386/kernel/reboot.c | |||
@@ -337,14 +337,10 @@ void machine_restart(char * __unused) | |||
337 | machine_real_restart(jump_to_bios, sizeof(jump_to_bios)); | 337 | machine_real_restart(jump_to_bios, sizeof(jump_to_bios)); |
338 | } | 338 | } |
339 | 339 | ||
340 | EXPORT_SYMBOL(machine_restart); | ||
341 | |||
342 | void machine_halt(void) | 340 | void machine_halt(void) |
343 | { | 341 | { |
344 | } | 342 | } |
345 | 343 | ||
346 | EXPORT_SYMBOL(machine_halt); | ||
347 | |||
348 | void machine_power_off(void) | 344 | void machine_power_off(void) |
349 | { | 345 | { |
350 | lapic_shutdown(); | 346 | lapic_shutdown(); |
@@ -355,5 +351,4 @@ void machine_power_off(void) | |||
355 | pm_power_off(); | 351 | pm_power_off(); |
356 | } | 352 | } |
357 | 353 | ||
358 | EXPORT_SYMBOL(machine_power_off); | ||
359 | 354 | ||
diff --git a/arch/i386/mach-visws/reboot.c b/arch/i386/mach-visws/reboot.c index 95e4676594e1..9e9296676f93 100644 --- a/arch/i386/mach-visws/reboot.c +++ b/arch/i386/mach-visws/reboot.c | |||
@@ -22,8 +22,6 @@ void machine_restart(char * __unused) | |||
22 | outb(PIIX4_RESET_VAL, PIIX4_RESET_PORT); | 22 | outb(PIIX4_RESET_VAL, PIIX4_RESET_PORT); |
23 | } | 23 | } |
24 | 24 | ||
25 | EXPORT_SYMBOL(machine_restart); | ||
26 | |||
27 | void machine_power_off(void) | 25 | void machine_power_off(void) |
28 | { | 26 | { |
29 | unsigned short pm_status; | 27 | unsigned short pm_status; |
@@ -43,10 +41,7 @@ void machine_power_off(void) | |||
43 | outl(PIIX_SPECIAL_STOP, 0xCFC); | 41 | outl(PIIX_SPECIAL_STOP, 0xCFC); |
44 | } | 42 | } |
45 | 43 | ||
46 | EXPORT_SYMBOL(machine_power_off); | ||
47 | |||
48 | void machine_halt(void) | 44 | void machine_halt(void) |
49 | { | 45 | { |
50 | } | 46 | } |
51 | 47 | ||
52 | EXPORT_SYMBOL(machine_halt); | ||
diff --git a/arch/i386/mach-voyager/voyager_basic.c b/arch/i386/mach-voyager/voyager_basic.c index 8680080a6a89..b3eda46e0fe9 100644 --- a/arch/i386/mach-voyager/voyager_basic.c +++ b/arch/i386/mach-voyager/voyager_basic.c | |||
@@ -278,8 +278,6 @@ machine_restart(char *cmd) | |||
278 | } | 278 | } |
279 | } | 279 | } |
280 | 280 | ||
281 | EXPORT_SYMBOL(machine_restart); | ||
282 | |||
283 | void | 281 | void |
284 | mca_nmi_hook(void) | 282 | mca_nmi_hook(void) |
285 | { | 283 | { |
@@ -315,12 +313,9 @@ machine_halt(void) | |||
315 | machine_power_off(); | 313 | machine_power_off(); |
316 | } | 314 | } |
317 | 315 | ||
318 | EXPORT_SYMBOL(machine_halt); | ||
319 | |||
320 | void machine_power_off(void) | 316 | void machine_power_off(void) |
321 | { | 317 | { |
322 | if (pm_power_off) | 318 | if (pm_power_off) |
323 | pm_power_off(); | 319 | pm_power_off(); |
324 | } | 320 | } |
325 | 321 | ||
326 | EXPORT_SYMBOL(machine_power_off); | ||