linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jens Axboe <axboe@suse.de>
To: Mikael Pettersson <mikpe@csd.uu.se>
Cc: linux-kernel@vger.kernel.org, Andrew Morton <akpm@osdl.org>,
	Linus Torvalds <torvalds@osdl.org>
Subject: Re: 2G memory split
Date: Tue, 10 Jan 2006 14:54:05 +0100	[thread overview]
Message-ID: <20060110135404.GF3389@suse.de> (raw)
In-Reply-To: <17347.47882.735057.154898@alkaid.it.uu.se>

On Tue, Jan 10 2006, Mikael Pettersson wrote:
> Jens Axboe writes:
>  > Hi,
>  > 
>  > It does annoy me that any 1G i386 machine will end up with 1/8th of the
>  > memory as highmem. A patch like this one has been used in various places
>  > since the early 2.4 days at least, is there a reason why it isn't merged
>  > yet? Note I just hacked this one up, but similar patches abound I'm
>  > sure. Bugs are mine.
>  > 
>  > Signed-off-by: Jens Axboe <axboe@suse.de>
>  > 
>  > diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
>  > index d849c68..0b2457b 100644
>  > --- a/arch/i386/Kconfig
>  > +++ b/arch/i386/Kconfig
>  > @@ -444,6 +464,24 @@ config HIGHMEM64G
>  >  
>  >  endchoice
>  >  
>  > +choice
>  > +	depends on NOHIGHMEM
>  > +	prompt "Memory split"
>  > +	default DEFAULT_3G
>  > +	help
>  > +	  Select the wanted split between kernel and user memory. On a 1G
>  > +	  machine, the 3G/1G default split will result in 128MiB of high
>  > +	  memory. Selecting a 2G/2G split will make all of memory available
>  > +	  as low memory. Note that this will make your kernel incompatible
>  > +	  with binary only kernel modules.
> 
> 2G/2G is not the only viable alternative. On my 1GB x86 box I'm

Yes I know, as I wrote to Ingo I wanted to keep it really simple. It can
easily be extended, of course.

> using "lowmem1g" patches for both 2.4 and 2.6, which results in
> 2.75G for user-space. I'm sure others have other preferences.
> Any standard option for this should either have several hard-coded
> alternatives, or should support arbitrary values (within reason).

That's just asking for trouble, imho. We should provide some defaults
(that work well on 1G and 2G machines, for instance) and stick to that.

> (See http://www.csd.uu.se/~mikpe/linux/patches/*/patch-i386-lowmem1g-*
> if you're interested.)

It's similar to what I've been doing so far (just changing page.h to
0xb0000000). 0x80000000 might be a bad default as suggested by Byron, as
it just misses the full 2G.

0xb0000000 is a much better default, but I didn't think that would fly
as a patch.

-- 
Jens Axboe


  reply	other threads:[~2006-01-10 13:52 UTC|newest]

Thread overview: 87+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-01-10 12:58 2G memory split Jens Axboe
2006-01-10 13:29 ` Ingo Molnar
2006-01-10 13:37   ` Jens Axboe
2006-01-10 13:43     ` Byron Stanoszek
2006-01-10 13:47       ` Jens Axboe
2006-01-10 14:39       ` Jens Axboe
2006-01-10 14:44         ` Ingo Molnar
2006-01-10 15:03           ` [PATCH] Address space split configuration Jens Axboe
2006-01-10 15:11             ` Mark Lord
2006-01-10 15:24             ` Mikael Pettersson
2006-01-10 16:14         ` 2G memory split Linus Torvalds
2006-01-10 16:40           ` Mark Lord
2006-01-10 16:52             ` Linus Torvalds
2006-01-10 18:33               ` Coywolf Qi Hunt
2006-01-10 17:07           ` Mark Lord
2006-01-10 17:28             ` Linus Torvalds
2006-01-10 17:32             ` Jens Axboe
2006-01-10 17:37               ` Mark Lord
2006-01-10 18:58                 ` Mark Lord
2006-01-10 19:16                   ` [PATCH ] VMSPLIT config options (with default config fixed) Mark Lord
2006-01-10 17:37                     ` Jeff V. Merkey
2006-01-10 19:27                     ` Jens Axboe
2006-01-11  1:13                     ` J.A. Magallon
2006-01-11 10:15                       ` Jens Axboe
2006-01-11 16:00                     ` Greg Norris
2006-01-11 17:13                       ` Mark Lord
2006-01-11 17:44                         ` Greg Norris
2006-02-05 18:42                         ` Barry K. Nathan
2006-02-01 22:23                     ` Herbert Poetzl
2006-02-02 11:04                       ` Ulrich Mueller
2006-02-02 20:55                         ` Jan Engelhardt
2006-02-03 22:39                           ` Mark Lord
2006-02-04 10:23                             ` Ulrich Mueller
2006-02-04 11:06                               ` Jan Engelhardt
2006-02-04 10:36                             ` Jens Axboe
2006-02-04 11:05                             ` Jan Engelhardt
2006-02-04 13:57                               ` Mark Lord
2006-02-05 15:32                                 ` J.A. Magallon
2006-02-05 15:38                               ` Arjan van de Ven
2006-02-05 21:14                                 ` Jan Engelhardt
2006-02-05 21:19                                   ` Arjan van de Ven
2006-02-06 14:56                                     ` Jan Engelhardt
2006-02-07  0:41                                       ` Herbert Poetzl
2006-02-07  2:51                                         ` Mark Rustad
2006-02-07  9:38                                         ` Bernd Petrovitsch
2006-02-07 12:19                                           ` RFC: add an ADVANCED_USER option Adrian Bunk
2006-02-07 14:05                                             ` Ulrich Mueller
2006-02-07 14:42                                               ` Adrian Bunk
2006-02-09 16:06                                                 ` Jan Engelhardt
2006-01-10 17:48             ` 2G memory split Bernd Eckenfels
2006-01-10 16:30               ` [PATCH] " Jeff V. Merkey
2006-01-10 18:50                 ` Lennart Sorensen
2006-01-10 17:13                   ` Jeff V. Merkey
2006-01-10 19:42               ` Jens Axboe
2006-01-10 20:17                 ` Bernd Eckenfels
2006-01-10 20:28                   ` Jens Axboe
2006-01-11  8:39                     ` Bernd Eckenfels
2006-01-11 10:06                       ` Jens Axboe
2006-01-10 18:14           ` Martin Bligh
2006-01-10 18:34             ` Linus Torvalds
2006-01-10 16:56               ` Jeff V. Merkey
2006-01-10 17:01                 ` Jeff V. Merkey
2006-01-10 18:45                 ` Mark Lord
2006-01-10 18:46                 ` Martin Bligh
2006-01-10 18:58                 ` Jens Axboe
2006-01-10 17:17                   ` Jeff V. Merkey
2006-01-10 20:55                 ` Alan Cox
2006-01-10 19:12                   ` Jeff V. Merkey
2006-01-10 21:02                     ` Jens Axboe
2006-01-10 19:30                       ` Jeff V. Merkey
2006-01-10 18:39               ` Martin Bligh
2006-01-10 18:55               ` Dave Hansen
2006-01-10 19:01                 ` Mark Lord
2006-01-10 19:05                   ` Dave Hansen
2006-01-10 17:07         ` Sergey Vlasov
2006-04-10 14:11           ` Kirill Korotaev
2006-04-10 14:39             ` Mark Lord
2006-01-10 18:28         ` Coywolf Qi Hunt
2006-01-10 13:47 ` Mikael Pettersson
2006-01-10 13:54   ` Jens Axboe [this message]
2006-01-10 14:09     ` Gerd Hoffmann
2006-01-10 14:21       ` Jens Axboe
2006-01-10 14:25       ` Jens Axboe
2006-01-10 20:42   ` Jan Engelhardt
2006-01-11  0:25     ` Con Kolivas
2006-01-10 14:12 ` Mark Lord
2006-01-10 14:22   ` Jens Axboe

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=20060110135404.GF3389@suse.de \
    --to=axboe@suse.de \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mikpe@csd.uu.se \
    --cc=torvalds@osdl.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;
as well as URLs for NNTP newsgroup(s).