* [patch]VMSPLIT_2G conflicts with PAE
@ 2006-12-01 3:48 Shaohua Li
2006-12-01 7:55 ` Jens Axboe
2006-12-02 13:05 ` Pavel Machek
0 siblings, 2 replies; 5+ messages in thread
From: Shaohua Li @ 2006-12-01 3:48 UTC (permalink / raw)
To: lkml; +Cc: Andrew Morton
PAGE_OFFSET is 0x78000000 with VMSPLIT_2G, this address is in the middle
of the second pgd entry with pae enabled. This breaks assumptions
(address is aligned to pgd entry's address) in a lot of places like
pagetable_init. Fixing the assumptions is hard (eg, low mapping). SO I
just changed the address to 0x80000000.
Signed-off-by: Shaohua Li <shaohua.li@intel.com>
diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
index 8ff1c6f..fddfb26 100644
--- a/arch/i386/Kconfig
+++ b/arch/i386/Kconfig
@@ -532,7 +532,7 @@ endchoice
config PAGE_OFFSET
hex
default 0xB0000000 if VMSPLIT_3G_OPT
- default 0x78000000 if VMSPLIT_2G
+ default 0x80000000 if VMSPLIT_2G
default 0x40000000 if VMSPLIT_1G
default 0xC0000000
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [patch]VMSPLIT_2G conflicts with PAE
2006-12-01 3:48 [patch]VMSPLIT_2G conflicts with PAE Shaohua Li
@ 2006-12-01 7:55 ` Jens Axboe
2006-12-01 8:34 ` Shaohua Li
2006-12-02 13:05 ` Pavel Machek
1 sibling, 1 reply; 5+ messages in thread
From: Jens Axboe @ 2006-12-01 7:55 UTC (permalink / raw)
To: Shaohua Li; +Cc: lkml, Andrew Morton
On Fri, Dec 01 2006, Shaohua Li wrote:
> PAGE_OFFSET is 0x78000000 with VMSPLIT_2G, this address is in the middle
> of the second pgd entry with pae enabled. This breaks assumptions
> (address is aligned to pgd entry's address) in a lot of places like
> pagetable_init. Fixing the assumptions is hard (eg, low mapping). SO I
> just changed the address to 0x80000000.
>
> Signed-off-by: Shaohua Li <shaohua.li@intel.com>
>
> diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
> index 8ff1c6f..fddfb26 100644
> --- a/arch/i386/Kconfig
> +++ b/arch/i386/Kconfig
> @@ -532,7 +532,7 @@ endchoice
> config PAGE_OFFSET
> hex
> default 0xB0000000 if VMSPLIT_3G_OPT
> - default 0x78000000 if VMSPLIT_2G
> + default 0x80000000 if VMSPLIT_2G
> default 0x40000000 if VMSPLIT_1G
> default 0xC0000000
0x78000000 was chosen since it gives you the full 2G as low memory, if
you mave it 0x80000000 then you still have a little highmem and need
that turned on.
--
Jens Axboe
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]VMSPLIT_2G conflicts with PAE
2006-12-01 7:55 ` Jens Axboe
@ 2006-12-01 8:34 ` Shaohua Li
0 siblings, 0 replies; 5+ messages in thread
From: Shaohua Li @ 2006-12-01 8:34 UTC (permalink / raw)
To: Jens Axboe; +Cc: lkml, Andrew Morton
On Fri, 2006-12-01 at 08:55 +0100, Jens Axboe wrote:
> On Fri, Dec 01 2006, Shaohua Li wrote:
> > PAGE_OFFSET is 0x78000000 with VMSPLIT_2G, this address is in the middle
> > of the second pgd entry with pae enabled. This breaks assumptions
> > (address is aligned to pgd entry's address) in a lot of places like
> > pagetable_init. Fixing the assumptions is hard (eg, low mapping). SO I
> > just changed the address to 0x80000000.
> >
> > Signed-off-by: Shaohua Li <shaohua.li@intel.com>
> >
> > diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
> > index 8ff1c6f..fddfb26 100644
> > --- a/arch/i386/Kconfig
> > +++ b/arch/i386/Kconfig
> > @@ -532,7 +532,7 @@ endchoice
> > config PAGE_OFFSET
> > hex
> > default 0xB0000000 if VMSPLIT_3G_OPT
> > - default 0x78000000 if VMSPLIT_2G
> > + default 0x80000000 if VMSPLIT_2G
> > default 0x40000000 if VMSPLIT_1G
> > default 0xC0000000
>
> 0x78000000 was chosen since it gives you the full 2G as low memory, if
> you mave it 0x80000000 then you still have a little highmem and need
> that turned on.
Ok, Maybe the x86 relocatable patch (it seems removed the low identity
mapping) can help here. With it, we just need to fix pagetable_init.
Thanks,
Shaohua
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]VMSPLIT_2G conflicts with PAE
2006-12-01 3:48 [patch]VMSPLIT_2G conflicts with PAE Shaohua Li
2006-12-01 7:55 ` Jens Axboe
@ 2006-12-02 13:05 ` Pavel Machek
2006-12-02 18:53 ` Jan Engelhardt
1 sibling, 1 reply; 5+ messages in thread
From: Pavel Machek @ 2006-12-02 13:05 UTC (permalink / raw)
To: Shaohua Li; +Cc: lkml, Andrew Morton
Hi!
> PAGE_OFFSET is 0x78000000 with VMSPLIT_2G, this address is in the middle
> of the second pgd entry with pae enabled. This breaks assumptions
> (address is aligned to pgd entry's address) in a lot of places like
> pagetable_init. Fixing the assumptions is hard (eg, low mapping). SO I
> just changed the address to 0x80000000.
>
> Signed-off-by: Shaohua Li <shaohua.li@intel.com>
>
> diff --git a/arch/i386/Kconfig b/arch/i386/Kconfig
> index 8ff1c6f..fddfb26 100644
> --- a/arch/i386/Kconfig
> +++ b/arch/i386/Kconfig
> @@ -532,7 +532,7 @@ endchoice
> config PAGE_OFFSET
> hex
> default 0xB0000000 if VMSPLIT_3G_OPT
> - default 0x78000000 if VMSPLIT_2G
> + default 0x80000000 if VMSPLIT_2G
> default 0x40000000 if VMSPLIT_1G
> default 0xC0000000
Do we allow user entering arbitrary value here? In any case, it would
be nice to document alignment requirements of this one, because
otherwise someone *will* get it wrong.
--
Thanks for all the (sleeping) penguins.
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [patch]VMSPLIT_2G conflicts with PAE
2006-12-02 13:05 ` Pavel Machek
@ 2006-12-02 18:53 ` Jan Engelhardt
0 siblings, 0 replies; 5+ messages in thread
From: Jan Engelhardt @ 2006-12-02 18:53 UTC (permalink / raw)
To: Pavel Machek; +Cc: Shaohua Li, lkml, Andrew Morton
>> PAGE_OFFSET is 0x78000000 with VMSPLIT_2G, this address is in the middle
>> of the second pgd entry with pae enabled. This breaks assumptions
>> (address is aligned to pgd entry's address) in a lot of places like
>> pagetable_init. Fixing the assumptions is hard (eg, low mapping). SO I
>> just changed the address to 0x80000000.
>
>Do we allow user entering arbitrary value here? In any case, it would
>be nice to document alignment requirements of this one, because
>otherwise someone *will* get it wrong.
There was a reason 0x78000000 was chosen
http://marc.theaimsgroup.com/?l=linux-kernel&m=113690073801820&w=2
Though further back in
http://marc.theaimsgroup.com/?l=linux-kernel&m=113689790817521&w=2
0x80000000 was indeed posted.
-`J'
--
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2006-12-02 18:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2006-12-01 3:48 [patch]VMSPLIT_2G conflicts with PAE Shaohua Li
2006-12-01 7:55 ` Jens Axboe
2006-12-01 8:34 ` Shaohua Li
2006-12-02 13:05 ` Pavel Machek
2006-12-02 18:53 ` Jan Engelhardt
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox