Re: [PATCH 5/5] x86: Move enabling of PSE and PGE out of init_memory_mapping

[Jacob Shin <jacob.shin@amd.com>]

On Tue, Aug 14, 2012 at 10:49:16AM +0200, Borislav Petkov wrote:
> On Mon, Aug 13, 2012 at 04:20:40PM -0700, Tejun Heo wrote:
> > I meant the patch should come before the patch making multiple calls
> > to init_memory_mapping().

Ah, okay .. got it.

Hm .. for some reason Tejun's emails are no longer making it into my
inbox, there must be some filtering going on at the corporate IT level.
I'll try and inquire IT about it ..

Sorry, :-(

> 
> Ah, this makes sense.
> 
> > > > Also, there seem to be other stuff which need to be moved out.
> > > 
> > > Which are those pls?
> > 
> > Rebuilding pgtable on each invocation?
> 
> You mean pagetable_reserve() right?

This is actually needed on every call to init_memory_mapping(),

My patch 2/5 changes find_early_table_space() to find just enough space
to map start to end. The pagetable_reserve() will then reserve what we
actually used. Since init_memory_mapping() is called again and again
with different start to end ranges, we find space for the page tables
and reserve them every time.

> 
> Yes, Jacob, you basically might want to carve out all functionality
> from init_memory_mapping() which is independent from its start and end
> args and do the carving in a pre-patch or two to 1/5.
> 
> Thanks.
> 
> -- 
> Regards/Gruss,
>     Boris.
>