public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Robert Hancock <hancockr@shaw.ca>
To: Linus Torvalds <torvalds@linux-foundation.org>
Cc: Jonathan Corbet <corbet@lwn.net>, Yinghai Lu <yinghai@kernel.org>,
	Ingo Molnar <mingo@elte.hu>,
	e1000-devel@lists.sourceforge.net,
	LKML <linux-kernel@vger.kernel.org>,
	Steven Rostedt <rostedt@goodmis.org>
Subject: Re: 2.6.28-rc2 hates my e1000e
Date: Sat, 01 Nov 2008 19:26:28 -0600	[thread overview]
Message-ID: <490D01C4.7090106@shaw.ca> (raw)
In-Reply-To: <alpine.LFD.2.00.0811011609050.3483@nehalem.linux-foundation.org>

Linus Torvalds wrote:
> 
> On Sat, 1 Nov 2008, Jonathan Corbet wrote:
>> Looks to me like Linus's patch is the way to go, at least for now...
> 
> I'll make an -rc3 tomorrow. However, I suspect that if we have lots more 
> regressions, we'll just have to revert the resource handling back to the
> 2.6.27 state.
> 
> The problem with resource handling is that even when we can write code 
> that "makes sense", in the end firmware can always do odd things. For 
> example, in your case it really does make sense to keep the already 
> allocated PCI resources in the reserved region, because the firmware 
> obviously did both the reserved region _and_ the PCI BAR allocation.
> 
> At the same time, I'm worried that what Windows does is something totally 
> different, probably odd, and possibly even dependent on some HAL layer 
> motherboard driver or other.  And it's really the case that every single 
> time we change resource allocation - never mind how subtly, or how much 
> sense it makes - it will break some odd setup somewhere.

As far as I understand, Windows (at least with the ACPI HAL, which all 
remotely modern PCs will use) essentially uses the E820 map for 
determining usable RAM addresses only. It doesn't really care about 
what's reserved in that map, it expects the truly reserved ranges to be 
reserved as ACPI motherboard resources. (Though, it seems like it will 
still happily allow device BARs to overlap those ACPI reservations, at 
least if the BIOS put them there to start with..)

  reply	other threads:[~2008-11-02  1:26 UTC|newest]

Thread overview: 21+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <fa.P5jmIAOhSJ192sfWD+qlec07fe8@ifi.uio.no>
2008-10-31  0:45 ` 2.6.28-rc2 hates my e1000e Robert Hancock
2008-10-31  2:58   ` Jonathan Corbet
2008-10-31  4:08     ` Yinghai Lu
2008-10-31 15:07       ` Linus Torvalds
2008-10-31 15:44         ` Linus Torvalds
2008-10-31 16:00           ` Jonathan Corbet
2008-10-31 16:24             ` Linus Torvalds
2008-10-31 16:35               ` Yinghai Lu
2008-10-31 16:46                 ` Linus Torvalds
2008-10-31 16:51               ` Jonathan Corbet
2008-10-31 16:58                 ` Linus Torvalds
2008-10-31 20:20                   ` Yinghai Lu
2008-11-01 15:01                   ` Jonathan Corbet
2008-11-01 17:16                     ` Linus Torvalds
2008-11-01 17:35                       ` Steven Rostedt
2008-11-01 19:50                       ` Yinghai Lu
2008-11-01 22:45                         ` Jonathan Corbet
2008-11-01 22:47                           ` Yinghai Lu
2008-11-01 23:18                           ` Linus Torvalds
2008-11-02  1:26                             ` Robert Hancock [this message]
2008-10-30 23:44 Jonathan Corbet

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=490D01C4.7090106@shaw.ca \
    --to=hancockr@shaw.ca \
    --cc=corbet@lwn.net \
    --cc=e1000-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=rostedt@goodmis.org \
    --cc=torvalds@linux-foundation.org \
    --cc=yinghai@kernel.org \
    /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