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]"