Re: [ros-dev] Re: [ros-svn] [navaraf] 16445: Put the page tables and hyperspace at the right place in memory (ie. Windows compatible).

[Hartmut Birr]

Filip Navara wrote:
> Sorry, I was confused by incorrect graph in the Windows Internals, 4th
> edition book which shows an 8Mb hyperspace region (in fact the high
> 4Mb are unused). Do you think the attached patch corrects the problem?
>
> Regards,
> Filip

Hi,

your patch does fix the problem. I prefer to use 0xc0800000 - 0xc08fffff
for the hyperspace region. It removes the test for pae or nonpae mode.

- Hartmut

[Alex Ionescu]

Hartmut Birr wrote:
> Filip Navara wrote:
> > Sorry, I was confused by incorrect graph in the Windows Internals, 4th
> > edition book which shows an 8Mb hyperspace region (in fact the high
> > 4Mb are unused).

Yes, that graph is totally wrong and limited...shame for such a good book.
> > Do you think the attached patch corrects the problem?
> >
> > Regards,
> > Filip
>
> Hi,
>
> your patch does fix the problem. I prefer to use 0xc0800000 - 0xc08fffff
> for the hyperspace region.

But that's not NT-compatible anymore. First of all that's only 0xFFFFF 
bytes, while the Hyperspace region should be 0x400000 bytes. And 
secondly, the region is marked as reserved.  I agree with Filip to use 
0xc0400000-0xc0800000
> It removes the test for pae or nonpae mode.
> Hartmut

Best regards,
Alex Ionescu

[James Tabor]

Alex Ionescu wrote:
> Hartmut Birr wrote:
> > Filip Navara wrote:
> > > Sorry, I was confused by incorrect graph in the Windows Internals, 4th
> > > edition book which shows an 8Mb hyperspace region (in fact the high
> > > 4Mb are unused).
>
> Yes, that graph is totally wrong and limited...shame for such a good book.

Hum? If anything I know about M$ and their enablers, you always have to read
between the lines.
> > > Do you think the attached patch corrects the problem?
> > >
> > > Regards,
> > > Filip
> >
> > Hi,
> >
> > your patch does fix the problem. I prefer to use 0xc0800000 - 0xc08fffff
> > for the hyperspace region.
>
> But that's not NT-compatible anymore. First of all that's only 0xFFFFF 
> bytes, while the Hyperspace region should be 0x400000 bytes. And 
> secondly, the region is marked as reserved.  I agree with Filip to use 
> 0xc0400000-0xc0800000
> > It removes the test for pae or nonpae mode.
> > Hartmut
>
> Best regards,
> Alex Ionescu

[Hartmut Birr]

Alex Ionescu wrote:
> > your patch does fix the problem. I prefer to use 0xc0800000 - 0xc08fffff
> > for the hyperspace region.
>
> First of all that's only 0xFFFFF bytes, while the Hyperspace region
> should be 0x400000 bytes.

I meant 0xc0800000 - 0xc0bfffff.

- Hartmut