public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Matthew Garrett <mjg@redhat.com>
To: Thomas Renninger <trenn@suse.de>
Cc: Len Brown <lenb@kernel.org>, Zhao Yakui <yakui.zhao@intel.com>,
	"Linux-acpi@vger.kernel.org" <Linux-acpi@vger.kernel.org>,
	me@markdoughty.co.uk, ibm-acpi-devel@lists.sourceforge.net
Subject: Re: [PATCH] : ACPI : Use RSDT instead of XSDT by adding boot option of "acpi=rsdt"
Date: Mon, 12 Jan 2009 23:38:57 +0000	[thread overview]
Message-ID: <20090112233857.GB6849@srcf.ucam.org> (raw)
In-Reply-To: <200901122317.08849.trenn@suse.de>

On Mon, Jan 12, 2009 at 11:17:07PM +0100, Thomas Renninger wrote:
> On Monday 12 January 2009 03:16:55 pm Matthew Garrett wrote:
> > On Mon, Jan 12, 2009 at 03:13:04PM +0100, Thomas Renninger wrote:
> > > 99.998% of all machines work fine with the current, spec conform
> > > implementation.
> > > There are the two ThinkPads for which the vendor admitted that the BIOS
> > > is broken which work better with the rsdt blacklisted.
> >
> > Two Thinkpads, an entire range of HP workstations, how many others?
> You wanted to provide more info about the HPs. Are they even sold? Have you 
> informed HP that their BIOSes are broken?
> It's easy and safe (also in respect of not breaking Windows) for them to fix 
> them. If we do not wait some more months, I am confident they are going to do 
> it, tell me privately if you need contacts.

Yes, this has been discussed with HP. Some of the machines are still 
being sold - some are out of support and so will not receive updated 
BIOSes. http://www.google.com/search?q=32%2F64X%20address%20mismatch 
shows that it's not just limited to Thinkpads and HPs.

> > We have no idea which bugs are being triggered by this. Refusing to fix it 
> > because we'll potentially break some machines that may not even exist is 
> > not a sensible plan.
> The plan is to stay to the spec. Is that so wrong?

When sticking to the spec results in hardware that fails to work, then 
yes, sticking to the spec is the wrong thing to do. We violate all sorts 
of specs in all sorts of places simply because hardware and software are 
written by humans who make mistakes. When producing an operating system, 
the aim is for it to work on the hardware our users want supported.

> There are netbooks which only support Linux and I hope there will be a lot 
> more machines only supporting Linux as OS in the future.
> Such Windows compatibility, Spec violating fixes will break them.
> It's the broken BIOSes, those machines the vendor never ever tried booting 
> Linux which need the boot param and blacklist adding...

Find me one x86 netbook vendor who hasn't validated Windows on their 
system.

> Maybe we come away with this change this time, maybe we hurt someone badly who
> did everything right: stick to the spec, tested and verified that Linux works 
> fine.

Or, more likely, we make this change and people find that their hardware 
starts working.

-- 
Matthew Garrett | mjg59@srcf.ucam.org

  reply	other threads:[~2009-01-12 23:39 UTC|newest]

Thread overview: 37+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-10-31 18:42 [PATCH] ACPI: EC: clean up tmp variable before reuse Alexey Starikovskiy
2008-11-03  8:02 ` Zhao Yakui
2008-11-03  8:24 ` [PATCH]: ACPI: Initialize EC global lock based on the return value of _GLK Zhao Yakui
2008-11-04  7:41 ` [PATCH]: ACPI Cleanup :Initialize EC global lock based on the return status Zhao Yakui
2008-11-04  8:05   ` Alexey Starikovskiy
2008-11-04  8:58     ` Rafael J. Wysocki
2008-11-04  9:21       ` Alexey Starikovskiy
2008-11-04  9:37     ` Zhao Yakui
2008-11-04  9:38       ` Alexey Starikovskiy
2008-11-05  1:05         ` Zhao Yakui
2008-11-05  7:24           ` Alexey Starikovskiy
2008-12-17  8:55 ` [PATCH] : ACPI : Use RSDT instead of XSDT by adding boot option of "acpi=rsdt" Zhao Yakui
2009-01-09  6:35   ` Len Brown
2009-01-09 10:54     ` Thomas Renninger
2009-01-09 10:59       ` Len Brown
2009-01-09 12:16         ` Thomas Renninger
2009-01-09 12:34           ` Matthew Garrett
2009-01-12 14:13             ` Thomas Renninger
2009-01-12 14:16               ` Matthew Garrett
2009-01-12 22:17                 ` Thomas Renninger
2009-01-12 23:38                   ` Matthew Garrett [this message]
2009-01-09 10:58   ` Blacklist known broken machines to use the rsdt and enabled Cstates on R40e Thomas Renninger
2009-01-09 10:58   ` [PATCH 1/2] Blacklist known broken machines (ThinkPad R40e and R50e) to use rsdt instead xsdt Thomas Renninger
2009-01-09 10:58   ` [PATCH 2/2] R40e using rsdt (previous patch) makes all Cstates work -> remove blacklisting Thomas Renninger
2008-12-30  4:01 ` [PATCH] : ACPI : Add the MWAIT C-state mask to avoid overflow Zhao Yakui
2009-01-04  4:04 ` Zhao Yakui
2009-01-09  6:28   ` Len Brown
2009-01-12  7:07 ` [PATCH] : ACPI : Use clocksource to get the C-state time instead of ACPI PM timer Zhao Yakui
2009-01-12  7:58   ` Rafael J. Wysocki
2009-01-12  9:31     ` Zhao Yakui
2009-01-12 12:27       ` Rafael J. Wysocki
2009-01-12  9:39     ` Zhao Yakui
2009-01-12 22:09   ` Pallipadi, Venkatesh
2009-01-13  1:26     ` Zhao Yakui
2009-01-13  1:42     ` Zhao Yakui
2009-01-13  3:50 ` [RESEND] " Zhao Yakui
2009-01-20  2:52   ` Len Brown

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=20090112233857.GB6849@srcf.ucam.org \
    --to=mjg@redhat.com \
    --cc=Linux-acpi@vger.kernel.org \
    --cc=ibm-acpi-devel@lists.sourceforge.net \
    --cc=lenb@kernel.org \
    --cc=me@markdoughty.co.uk \
    --cc=trenn@suse.de \
    --cc=yakui.zhao@intel.com \
    /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