cvs commit: src/sys/pci agp.c agpvar.h

[Eric Anholt]

On Tue, 2005-12-20 at 13:18 -0700, Scott Long wrote:
> John Baldwin wrote:
> 
> > jhb         2005-12-20 20:05:21 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/pci              agp.c agpvar.h 
> >   Log:
> >   Don't map the AGP aperture into contiguous KVA.  The various graphics
> >   drivers already map sections into KVA as needed anyway.  Note that this
> >   will probably break the nvidia driver, but I will coordinate to get that
> >   fixed.
> >   
> >   MFC after:      2 weeks
> >   
> >   Revision  Changes    Path
> >   1.49      +1 -3      src/sys/pci/agp.c
> >   1.3       +0 -1      src/sys/pci/agpvar.h
> 
> So this means no more panicing when using a card with 256 or 512MB of RAM?
> 
> Scott

Your AGP aperture size is different from your card's framebuffer size.
The DRM was guilty for some time of needlessly mapping all of your
card's framebuffer, but that should have been fixed a few months ago.
Default AGP aperture size is chosen by the BIOS, on the other hand.

I'm really glad John's dealing with this, since it was likely a major
stability issue on newer hardware (i915 has either a 128MB or 256MB
aperture, for example).

-- 
Eric Anholt                                     eta at lclark.edu
http://people.freebsd.org/~anholt/              anholt at FreeBSD.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 187 bytes
Desc: This is a digitally signed message part
Url : http://lists.freebsd.org/pipermail/cvs-src/attachments/20051220/e605dd8a/attachment.bin