From owner-freebsd-hackers@FreeBSD.ORG Mon Apr 4 16:01:16 2011 Return-Path: Delivered-To: freebsd-hackers@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 89A99106567F for ; Mon, 4 Apr 2011 16:01:16 +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 0C6FC8FC13 for ; Mon, 4 Apr 2011 16:01:16 +0000 (UTC) Received: by rincewind.paeps.cx (Postfix, from userid 1001) id 64C0FD7445E; Mon, 4 Apr 2011 18:01:15 +0200 (CEST) Date: Mon, 4 Apr 2011 18:01:15 +0200 From: Philip Paeps To: freebsd-hackers@freebsd.org Message-ID: <20110404160115.GS71940@rincewind.paeps.cx> Mail-Followup-To: freebsd-hackers@freebsd.org References: <20110404141016.GL71940@rincewind.paeps.cx> <4D99D81A.7000903@FreeBSD.org> <20110404145748.GP71940@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) 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 16:01:16 -0000 On 2011-04-04 08:51:03 (-0700), Garrett Cooper wrote: > On Mon, Apr 4, 2011 at 7:57 AM, Philip Paeps wrote: > > On 2011-04-04 17:39:22 (+0300), Andriy Gapon wrote: > >> on 04/04/2011 17:10 Philip Paeps said the following: > >> > 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 helps that our pciconf looks to be compatible with that format.  We > >> > just ignore subvendor and subdevice, but it doesn't appear to matter > >> > that the file contains this information. > >> > [...] > >> > >> This is just for your information: > >> http://thread.gmane.org/gmane.os.freebsd.current/127979/focus=128577 > >> Maybe you'll find something useful there. > > > > Thanks.  I've just read through that discussion.  It doesn't look like > > there were any "serious" objections to pulling in the new list, other than > > verifying that the new list contains all the entries the current list > > contains. > > > > I am not entirely sure what the little side-discussion about using # > > instead of ; as a comment marker is about.  It looks to me like the > > pci.ids file "Just Works"[tm] with our pciconf(8).  Maybe Garrett can > > elaborate on that? > > Well, close. You can substitute the hashes with semicolons, but you > need to get rid of subvendor / subdevice. The presence of the subvendor/subdevice entries doesn't appear to affect the correctness of the output on the machines I've tested it on. Cursorily reading through the pciconf(8) source code suggests it ignores that data (and as it happens the presence of subvendors and subdevices). I think the "most correct" thing to do would be to teach pciconf(8) about subvendors and subdevices and pull in the new list as-is. I'll take a look at adding support for that later today. It shouldn't be much work. > Setting PCICONF_VENDOR_DATABASE to pci.ids worked, but all the items on my > box are different in the new file (all device descriptions, but also some of > the vendor descriptions dealing with my nVidia card, Realtek NIC, and LSI > card) [it would be interesting to see who's correct in terms of corporation > branding :)..]. So this definitely isn't MFC-able :). It doesn't look like they're fundamentally different through. The strings changed, but still seem to be describing the same devices. I'm happy to agree that this isn't MFC-able. > > I think we should just go with the new list, but I'll hold off for a bit > > to let others object. ;) > > +1, just for the fact that our sources are becoming stale. I wonder though > what other OSes like NetBSD/OpenBSD/[Open]Solaris/IlluminOS use however for > tracking PCI IDs, as the sources for pci.ids aren't necessarily the vendor > itself and in some cases are end-users. I thought some of our other sources > were based on data provided by vendors. A comment from jfv@ in the thread from a few months ago, suggests that at least Intel contributes directly to the pci.ids list. One measurement isn't a valid statistic though. - Philip -- Philip Paeps Please don't Cc me, I am philip@freebsd.org subscribed to the list.