public inbox for linux-acpi@vger.kernel.org
 help / color / mirror / Atom feed
From: Len Brown <lenb@kernel.org>
To: "Éric Piel" <Eric.Piel@tremplin-utc.net>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
	Tilman Schmidt <tilman@imap.cc>,
	Dave Hansen <haveblue@us.ibm.com>,
	Andrew Morton <akpm@linux-foundation.org>,
	linux-kernel@vger.kernel.org, Thomas Renninger <trenn@suse.de>,
	Len Brown <len.brown@intel.com>,
	Christoph Hellwig <hch@infradead.org>,
	Markus Gaugusch <dsdt@gaugusch.at>,
	linux-acpi@vger.kernel.org, Al Viro <viro@zeniv.linux.org.uk>,
	Arjan van de Ven <arjanv@redhat.com>
Subject: Re: [2.6.25-rc5-mm1] BUG: spinlock bad magic early during boot
Date: Mon, 17 Mar 2008 13:27:15 -0400	[thread overview]
Message-ID: <200803171327.16695.lenb@kernel.org> (raw)
In-Reply-To: <47DC66A8.1020905@tremplin-utc.net>

I agree with Linus' decision to revert/disable this feature.
I think it is appropriate to muck with this in -mm, but not in -rc6
Kudos to Christoph, who recommended we revert it back at -rc2.

> > What's the problem with just loading a new DSDT later? Potentially as in 
> > *much* later: including when user-space is all up-and-running? 

I don't think re-loading the DSDT at run-time would be practical.

First, booting with the OEM DSDT may nullify the benefit
of overriding the OEM DSDT -- the damage may have already been done.

Secondly, unwinding everything that depends on the DSDT is on the
order of kexec or suspend/resume.  We're talking about all the stuff
that PNP does at boot time, plus device discovery and driver binding.

The feature on the table here is an initrd DSDT override.
We already have the ability to statically compile a DSDT
override into the kernel image.  That capability is sufficient
for kernel developers.

The initrd version of the DSDT override is really for one scenario.
Somebody who has a BIOS that even Windows can't deal with -- so
no amount of "Windows bug compatbility" will help Linux with it.
They must be capable eough to generate or acquire a modified DSDT.
They must be unwilling/unable to re-build their kenrel from scratch
each time they update it.  Eg. following debian unstable updates etc.

I think that customer deserves support, particularly because they get
bragging rights that Linux works better on a box build for Windows
than Windows does:-)
However, I don't think there are enough customers like this to
justify a huge effort that would add risk to Linux.

-Len

  reply	other threads:[~2008-03-17 17:29 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <20080311011434.ad8c8d7d.akpm@linux-foundation.org>
2008-03-12  1:14 ` 2.6.25-rc5-mm1 Dave Young
     [not found] ` <47D87238.8080305@imap.cc>
     [not found]   ` <20080313183439.GA12798@suse.de>
2008-03-13 19:57     ` [2.6.25-rc5-mm1] WARNING: at drivers/base/sys.c:173 Dave Jones
     [not found]   ` <20080313195656.GA32463@codemonkey.org.uk>
2008-03-14  0:01     ` Tilman Schmidt
2008-03-14  0:44       ` Dave Jones
2008-03-14  0:57       ` Zhao Yakui
2008-03-14  9:58         ` Tilman Schmidt
2008-03-15 12:16         ` Tilman Schmidt
     [not found] ` <47D86D43.2060108@imap.cc>
     [not found]   ` <1205441216.4971.65.camel@nimitz.home.sr71.net>
     [not found]     ` <47D9C853.3040701@imap.cc>
     [not found]       ` <1205517802.12763.18.camel@nimitz.home.sr71.net>
2008-03-14 20:06         ` [2.6.25-rc5-mm1] BUG: spinlock bad magic early during boot Dave Hansen
2008-03-14 20:20           ` Linus Torvalds
2008-03-14 20:51           ` Eric Piel
2008-03-14 21:35             ` Dave Hansen
2008-03-14 22:50               ` Eric Piel
2008-03-14 23:29                 ` Dave Hansen
2008-03-15 12:47                   ` Tilman Schmidt
2008-03-15 19:21                     ` Linus Torvalds
2008-03-15 19:42                       ` Éric Piel
2008-03-15 20:19                         ` Linus Torvalds
2008-03-16  0:15                           ` Éric Piel
2008-03-17 17:27                             ` Len Brown [this message]
     [not found]                               ` <1205858252.21619.233.camel@queen.suse.de>
2008-03-18 20:32                                 ` Len Brown
2008-03-20 14:28                                   ` Thomas Renninger
2008-03-17 17:59                           ` Len Brown
2008-03-21 13:17                           ` Pavel Machek
2008-03-23 16:00                             ` Dave Hansen
2008-03-24 16:03                               ` Pavel Machek
2008-03-24 17:05                                 ` Eric Piel
2008-03-24 17:19                                   ` Pavel Machek
2008-03-24 17:23                                   ` Dave Hansen
2008-03-27  9:23                               ` Helge Hafting
2008-03-17 18:05                         ` Len Brown
2008-03-16 20:11                     ` Dave Hansen
2008-03-17 12:23                       ` Peter Zijlstra
2008-03-19 23:50                         ` Tilman Schmidt
2008-03-17 17:48                 ` 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=200803171327.16695.lenb@kernel.org \
    --to=lenb@kernel.org \
    --cc=Eric.Piel@tremplin-utc.net \
    --cc=akpm@linux-foundation.org \
    --cc=arjanv@redhat.com \
    --cc=dsdt@gaugusch.at \
    --cc=haveblue@us.ibm.com \
    --cc=hch@infradead.org \
    --cc=len.brown@intel.com \
    --cc=linux-acpi@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=tilman@imap.cc \
    --cc=torvalds@linux-foundation.org \
    --cc=trenn@suse.de \
    --cc=viro@zeniv.linux.org.uk \
    /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