Skip to contentSkip to conversation

Reboot PATCH (ACPI related)

PATCH Lukas Hejtmanek 2 participants 3 messages May 12, 2007 – May 14, 2007 ACPI

lore
Share & tools
lore
#0 lore raw

Hello,

why the attached patch is still not included in mainline kernel? It fixes boot problems after suspend to RAM.

Signature · 1 line
Lukáš Hejtmánek
i386/kernel/reboot.c renamed from i386/kernel/reboot.c.old +7 -0
@@ -31,6 +31,12 @@
 #ifdef CONFIG_SMP
 static int reboot_cpu = -1;
 #endif
+#ifdef CONFIG_ACPI
+extern u32 acpi_disable(void);
+#else
+static u32 acpi_disable(void){};
+#endif
+
 static int __init reboot_setup(char *str)
 {
 	while(1) {
@@ -322,6 +328,7 @@
 
 void machine_emergency_restart(void)
 {
+	acpi_disable();
 	if (!reboot_thru_bios) {
 		if (efi_enabled) {
 			efi.reset_system(EFI_RESET_COLD, EFI_SUCCESS, 0, NULL);
#1 lore raw
Quoted 24 lines · On Sat, 12 May 2007 22:10:07 +0200 Lukas Hejtmanek <[email protected]> wrote:

why the attached patch is still not included in mainline kernel? It fixes boot problems after suspend to RAM.

Signature · 4 lines
Lukáš Hejtmánek


[reboot.patch  text/x-diff (564B)]
i386/kernel/reboot.c renamed from i386/kernel/reboot.c.old +6 -0
@@ -31,6 +31,12 @@
 #ifdef CONFIG_SMP
 static int reboot_cpu = -1;
 #endif
+#ifdef CONFIG_ACPI
+extern u32 acpi_disable(void);
+#else
+static u32 acpi_disable(void){};
+#endif
+
 static int __init reboot_setup(char *str)
 {
 	while(1) {

i386/kernel/reboot.c line 334

Quoted 7 lines
i386/kernel/reboot.c renamed from i386/kernel/reboot.c.old +1 -0
@@ -322,6 +328,7 @@
 
 void machine_emergency_restart(void)
 {
+	acpi_disable();
 	if (!reboot_thru_bios) {
 		if (efi_enabled) {
 			efi.reset_system(EFI_RESET_COLD, EFI_SUCCESS, 0, NULL);

We'd prefer to fix the problem, whatever it is, rather than just turning APCI off.

Please ensure that a full bug report has been entered at bugzilla.kernel.org, thanks.
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html

#2 lore raw
Quoted 5 lines · Andrew Morton

We'd prefer to fix the problem, whatever it is, rather than just turning APCI off.

Please ensure that a full bug report has been entered at bugzilla.kernel.org, thanks.

The patch resulted from the following bug report:
http://bugzilla.kernel.org/show_bug.cgi?id=6655

Signature · 5 lines
Lukáš Hejtmánek
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html