Re: [PATCH] x86: Add reboot quirk for Dell Precision M4600

Re: [PATCH] x86: Add reboot quirk for Dell Precision M4600

[Len Brown]

On Sat, 6 Aug 2011, H. Peter Anvin wrote:

> Ingo Molnar <mingo@elte.hu> wrote:
> 
> >* Mel Gorman <mgorman@suse.de> wrote:
> >
> >> ACPI reboot does not work for Dell Precision M4600. Detect this
> >> machine via DMI and force the use of the PCI reboot method.
> >>     
> >> Reported-and-tested-by: Bruno Friedmann <bruno@ioda-net.ch>
> >> Signed-off-by: Mel Gorman <mgorman@suse.de>
> >> ---
> >>  arch/x86/kernel/reboot.c |    9 +++++++++
> >>  1 files changed, 9 insertions(+), 0 deletions(-)
> >
> >Weird - ACPI is the default Windows method of reboot - how come it 
> >does not work on that box? Does Windows work around this specific 
> >machine (unlikely), or is our ACPI boot sequence not 100% compatible 
> >with the Windows method (more likely IMO)?
> 
> It seems almost all the affected machines are Dell, too... very odd, no?

Dell website says that the M4600 is current,
and ships with Windows 7.  It is a "mobile workstation",
so it is more likely to have a windows "feature" driver,
say, than Dell's servers.

Matthew,
Did you do your reboot comopatibility test using Win7,
or something older?

BTW, it would be nice if ACPI incompatibilities and workarounds
somehow got stuck in the ACPI code rather than the arch code.
Unfortunately this kind sets a working reboot method instead
of disabling a non-working one -- so what it overrides has
changed when the default changed.

thanks,
-Len Brown, Intel Open Source Technolgy Center

Re: [PATCH] x86: Add reboot quirk for Dell Precision M4600

[Bruno Friedmann]

On 08/06/2011 05:31 PM, Len Brown wrote:
> On Sat, 6 Aug 2011, H. Peter Anvin wrote:
> 
>> Ingo Molnar <mingo@elte.hu> wrote:
>>
>>> * Mel Gorman <mgorman@suse.de> wrote:
>>>
>>>> ACPI reboot does not work for Dell Precision M4600. Detect this
>>>> machine via DMI and force the use of the PCI reboot method.
>>>>     
>>>> Reported-and-tested-by: Bruno Friedmann <bruno@ioda-net.ch>
>>>> Signed-off-by: Mel Gorman <mgorman@suse.de>
>>>> ---
>>>>  arch/x86/kernel/reboot.c |    9 +++++++++
>>>>  1 files changed, 9 insertions(+), 0 deletions(-)
>>>
>>> Weird - ACPI is the default Windows method of reboot - how come it 
>>> does not work on that box? Does Windows work around this specific 
>>> machine (unlikely), or is our ACPI boot sequence not 100% compatible 
>>> with the Windows method (more likely IMO)?
>>
>> It seems almost all the affected machines are Dell, too... very odd, no?
> 
> Dell website says that the M4600 is current,
> and ships with Windows 7.  It is a "mobile workstation",
> so it is more likely to have a windows "feature" driver,
> say, than Dell's servers.

And this computer can be buyed with RedHat 6.0 by phone call.
If you check the support website download and drivers you will some drivers for Redhat
(mainly the new version of alsa)
http://dell.to/r1IKhp

> 
> Matthew,
> Did you do your reboot comopatibility test using Win7,
> or something older?
> 
> BTW, it would be nice if ACPI incompatibilities and workarounds
> somehow got stuck in the ACPI code rather than the arch code.
> Unfortunately this kind sets a working reboot method instead
> of disabling a non-working one -- so what it overrides has
> changed when the default changed.
> 
> thanks,
> -Len Brown, Intel Open Source Technolgy Center
> 

In any case, if you want anymore testing, and dumps of something, just drop me an email
and the howto get it formatted as you want it.

Actually I'm in the desktop summit with the computer until 9th August night.

I don't know if we can find an owner of the M6600 (the 17") version to get feedback about that hardware which share
a lot with the M4600.

Also Dell can upgrade their Bios.

The only error or significant warning I saw in boot.msg is
<3>[   36.851649] sdhci-pci 0000:0b:00.1: Invalid iomem size. You may experience problems.




-- 

Bruno Friedmann
Ioda-Net Sàrl www.ioda-net.ch

openSUSE Member & Ambassador
GPG KEY : D5C9B751C4653227
irc: tigerfoot
--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html