From owner-freebsd-current@FreeBSD.ORG Fri Dec 16 21:05:36 2005 Return-Path: X-Original-To: freebsd-current@freebsd.org Delivered-To: freebsd-current@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 05EFE16A43D; Fri, 16 Dec 2005 21:05:36 +0000 (GMT) (envelope-from mistry.7@osu.edu) Received: from mail.united-ware.com (am-productions.biz [69.61.164.22]) by mx1.FreeBSD.org (Postfix) with ESMTP id 4D52F43D5F; Fri, 16 Dec 2005 21:05:35 +0000 (GMT) (envelope-from mistry.7@osu.edu) Received: from [192.168.1.100] (am-productions.biz [69.61.164.22]) (authenticated bits=0) by mail.united-ware.com (8.13.4/8.13.4) with ESMTP id jBGL8eL5032535 (version=TLSv1/SSLv3 cipher=RC4-MD5 bits=128 verify=NO); Fri, 16 Dec 2005 16:08:45 -0500 (EST) (envelope-from mistry.7@osu.edu) From: Anish Mistry To: John Baldwin Date: Fri, 16 Dec 2005 16:07:41 -0500 User-Agent: KMail/1.8.3 References: <200512141720.01572.jhb@freebsd.org> <200512152133.28401.mistry.7@osu.edu> <200512161522.04051.jhb@freebsd.org> In-Reply-To: <200512161522.04051.jhb@freebsd.org> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="nextPart7773695.MfYHKRH1RF"; protocol="application/pgp-signature"; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit Message-Id: <200512161607.41975.mistry.7@osu.edu> X-Spam-Status: No, score=-10.4 required=5.0 tests=ALL_TRUSTED,BAYES_00, MYFREEBSD2,MYFREEBSD3 autolearn=failed version=3.1.0 X-Spam-Checker-Version: SpamAssassin 3.1.0 (2005-09-13) on mail.united-ware.com X-Virus-Scanned: ClamAV 0.87/1210/Thu Dec 15 10:23:22 2005 on mail.united-ware.com X-Virus-Status: Clean Cc: freebsd-current@freebsd.org Subject: Re: hostb(4) and vgapci(4) patch X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 16 Dec 2005 21:05:36 -0000 --nextPart7773695.MfYHKRH1RF Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Content-Disposition: inline On Friday 16 December 2005 03:22 pm, John Baldwin wrote: > On Thursday 15 December 2005 09:33 pm, Anish Mistry wrote: > > On Thursday 15 December 2005 04:31 pm, you wrote: > > > On Thursday 15 December 2005 12:32 am, Anish Mistry wrote: > > > > On Wednesday 14 December 2005 05:20 pm, 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.=20 > > > > > 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 > > > > > > > > Thank you! It seems to work as advertised. I'm running > > > > mach64 DRM with the DPMS patch acpi_video and they both work. > > > > :) One small problem though. When I unload the acpi_video > > > > module and reload it I get the following: > > > > littleguy# kldload acpi_video > > > > acpi_video0: on vgapci0 > > > > acpi_video1: on vgapci0 > > > > littleguy# kldunload acpi_video > > > > acpi_video0: detached > > > > acpi_video1: detached > > > > littleguy# kldunload acpi_video > > > > kldunload: can't find file acpi_video: No such file or > > > > directory littleguy# kldload acpi_video > > > > acpi_video0: on vgapci0 > > > > acpi_video1: on vgapci0 > > > > acpi_video2: on vgapci0 > > > > littleguy# > > > > It also created multiple sysctls with subsequent loads: > > > > hw.acpi.video.crt0.active: 1 > > > > hw.acpi.video.lcd0.active: 1 > > > > hw.acpi.video.tv0.active: 0 > > > > hw.acpi.video.crt1.active: 1 > > > > hw.acpi.video.lcd1.active: 1 > > > > hw.acpi.video.tv1.active: 0 > > > > hw.acpi.video.crt2.active: 1 > > > > hw.acpi.video.lcd2.active: 1 > > > > hw.acpi.video.tv2.active: 0 > > > > > > Revert just the changes to acpi_video.c and then apply the > > > attached patch to see if it fixes the multiple load issue. > > > > Yes. The patch corrects the problem. > > Actually, can you try a simpler one just to be sure? Thanks. I just enabled witness for my previous problem and got LOR #155, but=20 the lines numbers are slightly different, probably just normal=20 updates. I tried to reproduce it with the old patch, but I couldn't. =20 So it seems that something else triggered it during the kldunload. The current patch works fine. =2D-=20 Anish Mistry --nextPart7773695.MfYHKRH1RF Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.2 (FreeBSD) iD8DBQBDoyydxqA5ziudZT0RAtNMAKCQV3X3yBXmLXeeALH3ST+0bRT/yQCgmlpR 6r/d0YRg5XZx98UmXNERc7c= =byvv -----END PGP SIGNATURE----- --nextPart7773695.MfYHKRH1RF--