Subject: Re: [Dri-devel] PPC Lockup (ati-pcigart-branch)

From: Gareth Hughes <[email-masked]>
Date: Tue, 23 Jan 2001 04:38:42 +1100

Dan Malek wrote:
>
> Michel Dänzer wrote:
>
> >                 entry->pagelist[j]= pte_page( *pte );
>
> Bzzzzt....you lose :-).
>
> The pte_page returns the kernel's page_struct for a physical
> memory page, not the physical address which you are after......
> You ignored the compiler warning here, didn't you :-).
>
> Just do this:
>
>         entry->pagelist[j] = (unsigned long)(pte_val(*pte)) & PAGE_MASK;

Umm, entry->pagelist should contain a list of "struct page *"s, not
physical addresses.  I believe your code does not do this...

-- Gareth

** Sent via the linuxppc-dev mail list. See http://lists.linuxppc.org/