RE: [Dri-devel] i810 agpgart curiosity

2002-01-14 Thread Sottek, Matthew J
>The next line I quoted: > new->physical = virt_to_phys((void *) new->memory[0]); >... >takes that masked physical address and applies virt_to_phys to it, >which just seems wrong. Like I said, I don't have an i810, and I >haven't exhaustively analyzed it to see, for instance, whether >new->ph

Re: [Dri-devel] i810 agpgart curiosity

2002-01-14 Thread Bjorn Helgaas
On Monday 14 January 2002 9:57 am, Sottek, Matthew J wrote: > Why do you think that isn't a virtual address? agp_alloc_page > gets a page and returns page_addresss(page) which is the kernel > virtual address for the page, right? agp_alloc_page (agp_generic_alloc_page in the i810 case) indeed retu

RE: [Dri-devel] i810 agpgart curiosity

2002-01-14 Thread Sottek, Matthew J
Why do you think that isn't a virtual address? agp_alloc_page gets a page and returns page_addresss(page) which is the kernel virtual address for the page, right? Kernel memory is paged so it doesn't have to be a user address to be virtual. BTW: It works just fine. Nobody on an i810/i815 would h