Bug 141171

Summary: reboot needs adjusting kernel parameters
Product: [openSUSE] SUSE Linux 10.1 Reporter: peter czanik <peter>
Component: KernelAssignee: Pavel Machek <pavel>
Status: RESOLVED FIXED QA Contact: E-mail List <qa-bugs>
Severity: Normal    
Priority: P5 - None    
Version: Alpha 4   
Target Milestone: ---   
Hardware: i686   
OS: Other   
Whiteboard:
Found By: Other Services Priority:
Business Priority: Blocker: ---
Marketing QA Status: --- IT Deployment: ---
Attachments: hwinfo of compaq nx8220 notebook

Description peter czanik 2006-01-02 10:53:07 UTC
Test was done on a Compaq nx8220 notebook with factory from 30th of December.

The machine reboots only, if 'reboot=bios' is specified as a boot parameter.
Comment 1 Olaf Kirch 2006-01-02 11:17:34 UTC
Can you please provide the output of hwinfo of your machine? Thanks!
Comment 2 peter czanik 2006-01-02 13:18:38 UTC
Created attachment 61855 [details]
hwinfo of compaq nx8220 notebook
Comment 3 Olaf Kirch 2006-01-03 13:18:17 UTC
Pavel, can you please look into this? Thanks!
Comment 4 Forgotten User ZhJd0F0L3x 2006-01-03 14:03:59 UTC
there was a discussion on lkml and a patch from Matthew Garrett to enable "reboot=b" after DMI-match on hp notebooks unconditionally.
I'll try to find it in my archives.
Comment 5 Forgotten User ZhJd0F0L3x 2006-01-03 14:36:00 UTC
http://thread.gmane.org/gmane.linux.kernel/352196

Warning: whitespace breakage

From: Matthew Garrett <mjg59 <at> srcf.ucam.org>
Subject: [PATCH] Reboot through the BIOS on newer HP laptops
Newsgroups: gmane.linux.kernel
Date: 2005-11-24 05:21:07 GMT (5 weeks, 5 days, 9 hours and 13 minutes ago)

Newer HP laptops (nc4200, nc6xxx, nc8xxx) hang on reboot with a standard 
configuration. Passing reboot=b makes them work. This patch adds a DMI 
quirk that defaults them to this mode, and doesn't appear to have any 
adverse affects on older HPs.

Signed-off-by: Matthew Garrett <mjg59 <at> srcf.ucam.org>

--- a/arch/i386/kernel/reboot.c.orig	2005-09-20 18:54:50.000000000 +0100
+++ a/arch/i386/kernel/reboot.c	2005-09-20 18:58:11.000000000 +0100
 <at>  <at>  -135,6 +135,14  <at>  <at> 
 			DMI_MATCH(DMI_PRODUCT_NAME, "PowerEdge 2400"),
 		},
 	},
+	{       /* HP laptops have weird reboot issues */
+	        .callback = set_bios_reboot,
+		.ident = "HP Laptop",
+		.matches = {
+		        DMI_MATCH(DMI_SYS_VENDOR, "Hewlett-Packard"),
+			DMI_MATCH(DMI_PRODUCT_NAME, "HP Compaq"),
+		},
+	},
 	{ }
 };

-- 
Matthew Garrett | mjg59 <at> srcf.ucam.org


i can confirm that this is needed on a compaq nc6230, too.
Comment 6 Pavel Machek 2006-01-05 19:42:24 UTC
It looks like this patch is on its way to Linus' tree, so we can just wait :-).
Comment 7 Pavel Machek 2006-01-05 19:43:29 UTC
It looks like this patch is on its way to Linus' tree, so we can just wait :-). It is in -mm tree already.
Comment 8 Forgotten User ZhJd0F0L3x 2006-01-05 21:36:07 UTC
It is very well possible that CODE10 will be based on 2.6.15 which will nog get this patch. Also, additional exposure may further lower the (already very low, i admit) risk of breakage or at least may let us know sooner. So i vote for including this patch in the suse kernels asap.
I am also using an affected machine ;-)
Comment 9 Pavel Machek 2006-01-11 10:13:31 UTC
AFAICS decision was to track at least 2.6.16-rc1, and that should be enough to fix this.