public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Dave Hansen <haveblue@us.ibm.com>
To: michael@ellerman.id.au
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
	PPC64 External List <linuxppc64-dev@ozlabs.org>
Subject: Re: Why no bigphysarea in mainline?
Date: Thu, 17 Mar 2005 06:35:32 -0800	[thread overview]
Message-ID: <1111070132.19021.31.camel@localhost> (raw)
In-Reply-To: <200503172057.06570.michael@ellerman.id.au>

On Thu, 2005-03-17 at 20:57 +1100, Michael Ellerman wrote:
> I realise bigphysarea is a bit of a hack, but it's no where near as
> big a hack as using mem=X to limit the kernel's memory and then using
> the rest of memory for your device driver.

Well, the fact that you can get away with that is a coincidence.  What
if you have 4GB of RAM on an x86 machine, you do mem=3G, and you start
using that top GB of memory for your driver?  You eventually write into
the PCI config space.  Ooops.  You get strange errors that way.

Doing mem= for drivers isn't just a hack, it's *WRONG*.  It's a ticking
time bomb that magically happens to work on some systems.  It will not
work consistently on a discontiguous memory system, or a memory hotplug
system.

> If no one has any fundamental objections I think it'd be good to get
> this merged into mainline so people start using it rather than mem=X
> hacks. To that end please let me know what you think is wrong with
> the patch as it stands (below).

Could you give some examples of drivers which are in the kernel that
could benefit from this patch?  We don't tend to put things like this
in, unless they have actual users.  We don't tend to change code for
out-of-tree users, either.

-- Dave


  reply	other threads:[~2005-03-17 14:36 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-03-17  9:57 Why no bigphysarea in mainline? Michael Ellerman
2005-03-17 14:35 ` Dave Hansen [this message]
2005-03-18  3:42   ` Michael Ellerman

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=1111070132.19021.31.camel@localhost \
    --to=haveblue@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linuxppc64-dev@ozlabs.org \
    --cc=michael@ellerman.id.au \
    /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