Date: Thu, 15 Dec 2005 13:41:20 -0600 From: Eric Anderson <anderson@centtech.com> To: John Baldwin <jhb@freebsd.org> Cc: anholt@freebsd.org, current@freebsd.org Subject: Re: hostb(4) and vgapci(4) patch Message-ID: <43A1C6E0.1050203@centtech.com> In-Reply-To: <200512151132.45474.jhb@freebsd.org> References: <200512141720.01572.jhb@freebsd.org> <43A0E3B3.6030903@centtech.com> <200512151132.45474.jhb@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
John Baldwin wrote: >On Wednesday 14 December 2005 10:32 pm, Eric Anderson wrote: > > >>John Baldwin wrote: >> >> >>>I have a patch that is an attempt to untangle a few things in relation to >>>Host-PCI bridges and VGA PCI devices. Basically, the change is to create >>>a more "real" hostb driver as well as a new vgapci driver and to change >>>agp, drm, and acpi_video to attach to these drivers. This means among >>>other things: >>> >>>- In theory you can now kldload agp after boot since it still has a place >>>to attach to. >>>- i830/915 drm is no longer a child of agp, instead both become children >>>of vgapci0. >>>- You can now use acpi_video with drm as both attach as children of >>>vgapci0. - This provides a way for us to possibly solve the DPMS problem >>>for suspend/resume (including a cleaner way to do the hack dpms patch I >>>posted to acpi@ a long while ago that several people still use). >>> >>>Some other details include: >>> >>>- agp devices no longer map the _entire_ aperture into contiguous KVA >>>meaning that it might be possible now to use a 256 MB aperture without >>>panicing - I've added a new pci_if.m method for locating a specific >>>capability for a PCI device. >>> >>>I have tested this on my laptop and verified that dri still works, but it >>>needs some wider testing, especially the i830/i915 case is slightly more >>>complicated. Also, this is not going to work with the nvidia-driver >>>currently, but that's something that can be fixed in the future. If the >>>agp non-mapping does fix the 256 MB aperture issues then I will probably >>>MFC that part to RELENG_6. >>> >>>http://www.FreeBSD.org/~jhb/patches/agp_cvs.patch >>> >>> >>cc -c -O2 -pipe -fno-strict-aliasing -march=pentiumpro -Wall >>-Wredundant-decls -Wnested-externs -Wstrict-prototypes >>-Wmissing-prototypes -Wpointer-arith -Winline -Wcast-qual >>-fformat-extensions -std=c99 -g -nostdinc -I- -I. -I/usr/src/sys >>-I/usr/src/sys/contrib/altq -D_KERNEL -DHAVE_KERNEL_OPTION_HEADERS >>-include opt_global.h -fno-common -finline-limit=8000 --param >>inline-unit-growth=100 --param large-function-growth=1000 >>-mno-align-long-strings -mpreferred-stack-boundary=2 -mno-mmx >>-mno-3dnow -mno-sse -mno-sse2 -mno-sse3 -ffreestanding -Werror >>/usr/src/sys/pci/agp_i810.c >>/usr/src/sys/pci/agp_i810.c: In function `agp_i810_detach': >>/usr/src/sys/pci/agp_i810.c:463: warning: unused variable `child' >>*** Error code 1 >> >> > >If you delete this line to fix the warning does it work ok? > > Yep - I'll report anything else I see. Thanks! Eric -- ------------------------------------------------------------------------ Eric Anderson Sr. Systems Administrator Centaur Technology Anything that works is better than anything that doesn't. ------------------------------------------------------------------------
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?43A1C6E0.1050203>