[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [tyndur-devel] [PATCH v2 6/7] kernel2: Richtige Flags fuer Page Directories



On Sun, Jan 03 17:06, Kevin Wolf wrote:
> On Sun, Jan 03, 2010 at 05:03:20PM +0100, Kevin Wolf wrote:
> > ! kernel2: Die richtigen Flags fuer ein Page Directory sind nicht
> >   unbedingt die Flags der ersten zu mappenden Page. (Bei mir war das
> >   sogar ein unmap und flags entsprechend 0 - fragt mich nicht...)
> > 
> > Signed-off-by: Kevin Wolf <kevin@xxxxxxxxxx>
> > ---
> >  src/kernel2/src/arch/i386/mm/mm_context.c |    3 ++-
> >  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> Ups, ich glaube, das ist die falsche Position, wenn es bisectable sein
> soll. Stellt euch einfach vor, dieser Patch käme vor Patch 4.
> 
> > 
> > diff --git a/src/kernel2/src/arch/i386/mm/mm_context.c b/src/kernel2/src/arch/i386/mm/mm_context.c
> > index a9fc8eb..e4eb79b 100644
> > --- a/src/kernel2/src/arch/i386/mm/mm_context.c
> > +++ b/src/kernel2/src/arch/i386/mm/mm_context.c
> > @@ -283,7 +283,8 @@ static bool map_page
> >      {
> >          page_table = (page_table_t) pmm_alloc(1);
> >          // kprintf("Pagetable=0x%08x\n", page_table);
> > -        page_directory[vpage / PAGE_TABLE_LENGTH] = (dword) page_table | flags;
> > +        page_directory[vpage / PAGE_TABLE_LENGTH] =
> > +            (dword) page_table | PTE_P | PTE_W | PTE_U;
> >         
> >          // An dieser Stelle kann die Page Table noch nicht initialisiert
> >          // werden, weil sie noch nicht gemappt ist.
> > -- 
> > 1.6.0.2
> > 

Jo, an der richtigen Stelle:

Acked-by: Antoine Kaufmann <toni@xxxxxxxxxx>

-- 
Antoine Kaufmann
<toni@xxxxxxxxxxxxxxxx>

Attachment: pgpma5rPlbVEi.pgp
Description: PGP signature