From owner-svn-src-all@FreeBSD.ORG Mon Nov 9 18:40:20 2009 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from [127.0.0.1] (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by hub.freebsd.org (Postfix) with ESMTP id 84A85106568B; Mon, 9 Nov 2009 18:40:19 +0000 (UTC) (envelope-from jkim@FreeBSD.org) From: Jung-uk Kim To: Robert Noland Date: Mon, 9 Nov 2009 13:39:52 -0500 User-Agent: KMail/1.6.2 References: <200911062032.nA6KWRXb027876@svn.freebsd.org> <200911091322.40212.jkim@FreeBSD.org> <1257791067.27939.25.camel@balrog.2hip.net> In-Reply-To: <1257791067.27939.25.camel@balrog.2hip.net> MIME-Version: 1.0 Content-Disposition: inline Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <200911091340.06459.jkim@FreeBSD.org> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, John Baldwin Subject: Re: svn commit: r199002 - in head/sys: dev/fb dev/pci isa X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 09 Nov 2009 18:40:20 -0000 On Monday 09 November 2009 01:24 pm, Robert Noland wrote: > On Mon, 2009-11-09 at 13:22 -0500, Jung-uk Kim wrote: > > On Friday 06 November 2009 04:47 pm, John Baldwin wrote: > > > On Friday 06 November 2009 3:32:26 pm Jung-uk Kim wrote: > > > > Author: jkim > > > > Date: Fri Nov 6 20:32:26 2009 > > > > New Revision: 199002 > > > > URL: http://svn.freebsd.org/changeset/base/199002 > > > > > > > > Log: > > > > Remove duplicate suspend/resume code from vga_pci.c and let > > > > vga(4) > > > > > > register > > > > > > > itself to an associated PCI device if it exists. It is > > > > little bit hackish but it should fix build without frame > > > > buffer driver since r198964. Fix some style(9) nits in > > > > vga_isa.c while we are here. > > > > > > Hmm, did you consider having vga_isa use an identify routine to > > > attach itself as a child of vgapci0? The hack of knowing the > > > first thing in the softc is a pointer is really gross and I'd > > > rather avoid it. Just creating a child of vgapci0 will > > > automatically cause suspend and resume to work w/o vgapci > > > having to have any special knowledge about vga_isa. > > > > In fact, I did consider it. However, lack of my new bus foo > > prevented me from doing the right thing AND fixing the build > > breakage at the same. I'll try little harder today. > > > > BTW, syscons <-> fb <-> vga <-> vgapci <-> etc. relationship is > > amazingly confusing. :-( > > Agreed, this is one of the things that has kept me from working on > kms... Oh, are you actually thinking about implementing KMS? How about KGI? http://www.kgi-project.org/ Whatever we do in this area, we will stall this newly resurrected project again, unfortunately. In fact, I am afraid I might have already. :-( Jung-uk Kim