On Thu, 2006-09-28 at 08:12 +0200, Joel Dahl wrote:
> On Wed, 2006-09-27 at 06:38 +0000, Eric Anholt wrote:
> > anholt      2006-09-27 06:38:54 UTC
> > 
> >   FreeBSD src repository
> > 
> >   Modified files:
> >     sys/pci              agp_i810.c 
> >   Log:
> >   Add support for 945G/GM AGP chipsets.
> >   
> >   The key problem was that the aperture size detection using the MSAC bit
> >   doesn't work -- the bit appears to be set even when it shouldn't be.  
> > Linux
> >   takes a different approach, testing for a bit of the GMADR (PCIR_BAR(2)) 
> > being
> >   set.  However, as I don't think that's a safe way to test aperture size, 
> > we
> >   just allocate the resource and check its size.  This also pointed out that
> >   agp_generic_attach hadn't been allocating our aperture resource, which may
> >   have caused problems in some cases.
> >   
> >   Also corrected is a minor copy-and-pasteo in an error case.
> 
> Hmm, I have a HP NX7400 laptop with an Intel i945 chipset, and it's
> running a very fresh current (GENERIC, no changes made whatsoever).
> However, I see the following in my dmesg when I start X (I *think* it
> started appearing just after your commit to agp_i810.c):

Ok, I can confirm that reverting back to revision 1.38 of agp_i810.c
fixes the problem.

-- 
Joel

_______________________________________________
cvs-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/cvs-all
To unsubscribe, send any mail to "[EMAIL PROTECTED]"

Reply via email to