From owner-freebsd-hackers@FreeBSD.ORG Mon Apr 4 14:52:03 2011 Return-Path: Delivered-To: hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 6E2CD1065674; Mon, 4 Apr 2011 14:52:03 +0000 (UTC) (envelope-from philip@paeps.cx) Received: from rincewind.paeps.cx (rincewind.paeps.cx [IPv6:2002:596a:f092::149]) by mx1.freebsd.org (Postfix) with ESMTP id 09BC08FC1B; Mon, 4 Apr 2011 14:52:03 +0000 (UTC) Received: by rincewind.paeps.cx (Postfix, from userid 1001) id 70AC2D7445C; Mon, 4 Apr 2011 16:51:59 +0200 (CEST) Date: Mon, 4 Apr 2011 16:51:59 +0200 From: Philip Paeps To: Garrett Cooper Message-ID: <20110404145159.GO71940@rincewind.paeps.cx> Mail-Followup-To: Garrett Cooper , hackers@freebsd.org References: <20110404141016.GL71940@rincewind.paeps.cx> <20110404141944.GN71940@rincewind.paeps.cx> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: X-PGP-Fingerprint: 356B AE02 4763 F739 2FA2 E438 2649 E628 C5D3 4D05 X-Date: Prickle-Prickle, Discord 21, 3177 YOLD X-Phase-of-Moon: The Moon is Waxing Crescent (1% of Full) X-Philip-Conspiracy: There is no conspiracy Organization: Happily Disorganized User-Agent: Mutt/1.5.21 (2010-09-15) Cc: hackers@freebsd.org Subject: Re: Updating PCI vendors database X-BeenThere: freebsd-hackers@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Technical Discussions relating to FreeBSD List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 04 Apr 2011 14:52:03 -0000 On 2011-04-04 07:31:53 (-0700), Garrett Cooper wrote: > On Mon, Apr 4, 2011 at 7:19 AM, Philip Paeps wrote: > > On 2011-04-04 16:10:16 (+0200), Philip Paeps wrote: > >> Would anyone object if I updated this list to source the same database used > >> by Linux distributions at http://pciids.sourceforge.net/v2.2/pci.ids? > > > > It occurs to me that people would want to verify that this list does actually > > work and that we gain (rather than lose) coverage from it. > > > > A sanity test I've run on a couple of machines: > > > >  % fetch http://pciids.sourceforge.net/v2.2/pci.ids > >  % pciconf -lv > /tmp/pciconflv.old > >  % PCICONF_VENDOR_DATABASE=pci.ids pciconf -lv >/tmp/pciconflv.new > >  % diff -u /tmp/pciconflv.old /tmp/pciconflv.new > > > > In all cases I've seen so far, the new list yields better (more correct and up > > to date) results than the exising list.  In no cases has pciconf complained > > about the new list. > > I've copy-pasted the discussion I brought this up to Warner/Brooks > several months ago for review. I think at that point, the lists we're currently sourcing still existed. As of this morning, I don't seem to be able to find Craig Hart's list of PCI IDs anywhere on the net. It's certainly no long available at the address listed in the source tree. > The big problem is that the descriptions with the previous source and the > new source clash, so this would cause a huge amount of diff churn; This would be a problem I agree, but not a huge one. If the churn would be too large, I would be hesitant to push this to any stable branches. But I don't think the churn should be a problem for HEAD. It doesn't look to me like the churn is too large on the machines I've used it on. Generally, the changes I've seen is devices which lacked a device description now have one, and devices which had a wrong or incomplete description now have a complete and (as far as I can tell) correct one. This feels like an improvement to me. :) > plus I think there are a few entries missing from each area (at least there > were the last time I looked -- maybe our pci_vendors is more spartan than > the new source is today). The new list is much more complete than the list we have currently. For one thing, it also includes subvendors and subdevices, which the current list lacks. Our pciconf doesn't care about those currently, but could be made to care. I also don't think we should underestimate the value of sharing a list with Linux (especially if the licence on the list is friendly to sharing it) and potentially other operating system vendors. - Philip -- Philip Paeps Please don't Cc me, I am philip@freebsd.org subscribed to the list.