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

[Kevin Wolf <kevin@xxxxxxxxxx>]

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
>