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

From: Jeff Hartmann <[email-masked]>
Date: Mon, 22 Jan 2001 10:38:29 -0700

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;
>
>
>
> 	-- Dan

entry->pagelist is of type struct page *.

-Jeff

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