Differential D3391

Prefer pciids if available as a database for pciconf(8)
ClosedPublic
Actions

Authored by bapt on Aug 15 2015, 8:28 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Jul 11, 2:52 AM

	Unknown Object (File)
	Fri, Jun 27, 6:38 PM

	Unknown Object (File)
	Wed, Jun 25, 9:20 PM

	Unknown Object (File)
	Mon, Jun 23, 9:22 AM

	Unknown Object (File)
	Mon, Jun 23, 8:25 AM

	Unknown Object (File)
	May 25 2025, 11:34 AM

	Unknown Object (File)
	May 23 2025, 4:28 AM

	Unknown Object (File)
	May 14 2025, 11:01 AM

Subscribers

Details

Reviewers

None

Group Reviewers

Commits

rS287746: MFC: r287522
rS287522: Prefer pciids database from ports if present

Diff Detail

Repository

rS FreeBSD src repository - subversion

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 106
Build 106: arc lint + arc unit

Event Timeline

bapt updated this revision to Diff 7968.Aug 15 2015, 8:28 PM

bapt retitled this revision from to Prefer pciids if available as a database for pciconf(8).

bapt updated this object.

bapt edited the test plan for this revision. (Show Details)

Herald added a reviewer: manpages. · View Herald TranscriptAug 15 2015, 8:28 PM

Herald added a subscriber: imp. · View Herald Transcript

bapt added a reviewer: wollman.Aug 15 2015, 9:01 PM

wblock added a subscriber: wblock.Aug 16 2015, 2:59 AM

wblock added inline comments.

usr.sbin/pciconf/pciconf.8
286	Let's rearrange this to be clearer: PCI vendor and device information is read from .Pa /usr/local/share/pciids/pci.ids . If that file is not present, it is read from .Pa /usr/share/misc/pci_vendors .
usr.sbin/pciconf/pciconf.c
552	Is a change that defines _PATH_LPCIVDB missing?

bapt added inline comments.Aug 17 2015, 3:19 PM

usr.sbin/pciconf/pciconf.c
552	It is on the first part of this review (in pathname.h)

wblock added inline comments.Aug 18 2015, 2:49 AM

usr.sbin/pciconf/pciconf.c
552	Huh. I could swear that did not appear when I first saw this.

No objection in principle but it's been far too long since I had anything to do with this code to express an opinion on the change itself.

Closed by commit rS287522: Prefer pciids database from ports if present (authored by bapt). · Explain WhySep 6 2015, 8:05 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

usr.sbin/

pciconf/

1 line

2 lines

7 lines

Diff 7968

usr.sbin/pciconf/pathnames.h

Loading...

usr.sbin/pciconf/pciconf.8

Loading...

usr.sbin/pciconf/pciconf.c

Loading...