From: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
To: Matthew Garrett <mjg59@srcf.ucam.org>
Cc: Len Brown <lenb@kernel.org>, Thomas Renninger <trenn@suse.de>,
Arjan van de Ven <arjan@linux.intel.com>,
linux-acpi <linux-acpi@vger.kernel.org>,
"Moore, Robert" <robert.moore@intel.com>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Andi Kleen <ak@linux.intel.com>,
Christian Kornacker <ckornacker@suse.de>
Subject: Re: ACPI OSI disaster on latest HP laptops - critical temperature shutdown
Date: Sat, 2 Aug 2008 11:38:33 -0300 [thread overview]
Message-ID: <20080802143833.GC14069@khazad-dum.debian.net> (raw)
In-Reply-To: <20080802054204.GB12646@srcf.ucam.org>
On Sat, 02 Aug 2008, Matthew Garrett wrote:
> On Fri, Aug 01, 2008 at 07:36:57PM -0300, Henrique de Moraes Holschuh wrote:
> > On Fri, 01 Aug 2008, Len Brown wrote:
> > > It is better to expose ourselves to the known tested Windows functionality
> > > -- even if it seems arbitrary, at least it is tested. The !Windows case
> > > results in running _completely_ untested BIOS code.
> >
> > Actually, we should masquerade properly as the latest Windows version
> > available for that machine, then. AFAIK, Windows does not set ALL the OSI
> > strings, just one. We ARE running untested code in some BIOSes because of
> > it.
>
> The BIOSes I've tested check _OSI in order of Windows release, which is
> consistent with Windows returning OSI strings for all previous versions.
> Do you have any examples that suggest this isn't the case?
Yes. IBM ThinkPads store the result of each version test separately, and I
recall I saw at least one DSDT code path that didn't test all of them in
order to select a branch of code to run.
It will be hard to find that DSDT, though. It was sometime ago :(
> > Maybe it would be better if every ACPICA-using OS defined a
> > _OSI(NotWindows), plus the relevant Windows OSI string they want to support,
> > and Intel would send word that this string is to be used ONLY to disable all
> > Windows bug workarounds, not to activate or deactivate any specific
> > functionality?
>
> Not all BIOSes would support this, so we'd need to support the Windows
> workarounds anyway. At that point, there's no real benefit in having
> multiple codepaths.
Sorry, but I will disagree.
Anything that can help in the future with the vendors that are better at
Linux support is a good thing. You are right that we will still have to
deal with the others, but there are such things as vendor-specific windows
workarounds (they didn't want to change their firmware, or they couldn't, or
the others didn't care to add the workaround, etc). If that vendor uses the
"NotWindows" OSI correctly, we would not need to take any special action.
--
"One disk to rule them all, One disk to find them. One disk to bring
them all and in the darkness grind them. In the Land of Redmond
where the shadows lie." -- The Silicon Valley Tarot
Henrique Holschuh
next prev parent reply other threads:[~2008-08-02 14:38 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2008-07-24 15:27 ACPI OSI disaster on latest HP laptops - critical temperature shutdowns Thomas Renninger
2008-07-24 15:42 ` Arjan van de Ven
2008-07-25 0:04 ` ACPI OSI disaster on latest HP laptops - critical temperature shutdown Len Brown
2008-07-25 10:44 ` Andi Kleen
2008-07-25 11:19 ` Thomas Renninger
2008-07-25 15:26 ` Rafael J. Wysocki
2008-07-26 12:42 ` Andi Kleen
[not found] ` <alpine.LFD.1.10.0807261406230.2958@localhost.localdomain>
2008-08-01 21:08 ` Len Brown
2008-08-03 17:23 ` Thomas Renninger
[not found] ` <alpine.LFD.1.10.0807250948320.3884@localhost.localdomain>
2008-08-01 21:07 ` Len Brown
2008-08-01 22:36 ` Henrique de Moraes Holschuh
2008-08-02 5:42 ` Matthew Garrett
2008-08-02 14:38 ` Henrique de Moraes Holschuh [this message]
2008-08-02 14:44 ` Norbert Preining
2008-08-02 14:51 ` Henrique de Moraes Holschuh
2008-08-04 21:33 ` Norbert Preining
2008-08-02 15:41 ` Matthew Garrett
2008-08-02 17:49 ` Henrique de Moraes Holschuh
2008-08-02 19:49 ` Matthew Garrett
[not found] ` <alpine.LFD.1.10.0807261409290.2958@localhost.localdomain>
2008-08-01 21:08 ` Len Brown
2008-07-25 22:10 ` Eric Piel
2008-07-25 22:19 ` Moore, Robert
[not found] ` <alpine.LFD.1.10.0807261311420.2958@localhost.localdomain>
2008-08-01 21:07 ` Len Brown
[not found] ` <alpine.LFD.1.10.0807261434380.2958@localhost.localdomain>
2008-08-01 21:02 ` ACPI OSI disaster on latest HP laptops - critical temperature shutdowns Len Brown
2008-08-13 19:22 ` Pavel Machek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20080802143833.GC14069@khazad-dum.debian.net \
--to=hmh@hmh.eng.br \
--cc=ak@linux.intel.com \
--cc=arjan@linux.intel.com \
--cc=ckornacker@suse.de \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mjg59@srcf.ucam.org \
--cc=robert.moore@intel.com \
--cc=trenn@suse.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®