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
gt; Sent: Friday, January 11, 2002 10:31 AM > To: [EMAIL PROTECTED] > Subject: [Dri-devel] i810 agpgart curiosity > > > The following code in agpgart_be.c looks bogus to me: > > static agp_memory *intel_i810_alloc_by_type(size_t pg_count, int type) > ... > i

RE: [Dri-devel] i810 agpgart curiosity

2002-01-14 Thread Sottek, Matthew J
i815 would have a working hardware cursor without it. -Matt -Original Message- From: Bjorn Helgaas [mailto:[EMAIL PROTECTED]] Sent: Friday, January 11, 2002 10:31 AM To: [EMAIL PROTECTED] Subject: [Dri-devel] i810 agpgart curiosity The following code in agpgart_be.c looks bogus to m

[Dri-devel] i810 agpgart curiosity

2002-01-11 Thread Bjorn Helgaas
The following code in agpgart_be.c looks bogus to me: static agp_memory *intel_i810_alloc_by_type(size_t pg_count, int type) ... if(type == AGP_PHYS_MEMORY) { ... new->memory[0] = agp_bridge.agp_alloc_page(); ... new->m