linux-kernel - Re: [PATCH 01/86] pci: export pci

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 30 Mar 2015 08:55:22 +0200
From:	Greg KH <gregkh@...uxfoundation.org>
To:	"Michael S. Tsirkin" <mst@...hat.com>
Cc:	linux-kernel@...r.kernel.org, Bjorn Helgaas <bhelgaas@...gle.com>,
	Jonathan Corbet <corbet@....net>,
	"David S. Miller" <davem@...emloft.net>,
	Hans Verkuil <hans.verkuil@...co.com>,
	Mauro Carvalho Chehab <mchehab@....samsung.com>,
	Alexei Starovoitov <ast@...mgrid.com>,
	stephen hemminger <stephen@...workplumber.org>,
	Masahiro Yamada <yamada.m@...panasonic.com>,
	Andy Shevchenko <andriy.shevchenko@...ux.intel.com>,
	Andy Lutomirski <luto@...capital.net>,
	Rasmus Villemoes <linux@...musvillemoes.dk>,
	Stephane Eranian <eranian@...gle.com>,
	Huang Rui <ray.huang@....com>,
	Peter Neubauer <pneubauer@...erwhite.org>,
	linux-pci@...r.kernel.org, linux-doc@...r.kernel.org,
	linux-api@...r.kernel.org
Subject: Re: [PATCH 01/86] pci: export pci_ids.h

On Mon, Mar 30, 2015 at 08:48:44AM +0200, Michael S. Tsirkin wrote:
> On Sun, Mar 29, 2015 at 10:40:47PM +0200, Greg KH wrote:
> > On Sun, Mar 29, 2015 at 03:37:01PM +0200, Michael S. Tsirkin wrote:
> > > The macros in pci_ids.h are pretty useful for userspace
> > > using the pci sysfs interface.
> > > At the moment userspace is forced to duplicate these macros
> > > (e.g. QEMU does this), it is better to expose them in
> > > /usr/include/linux/pci_ids.h so everyone can just include
> > > this header.
> > > 
> > > Signed-off-by: Michael S. Tsirkin <mst@...hat.com>
> > > ---
> > >  include/linux/pci_ids.h      | 2998 +-----------------------------------------
> > >  include/uapi/linux/pci_ids.h | 2997 +++++++++++++++++++++++++++++++++++++++++
> > 
> > No, please use the pci ids file from the upstream pci id database
> > instead.
> 
> 
> >  We shouldn't be putting these all in one file,
> 
> pci.txt says:
> 	Please add PCI_VENDOR_ID_xxx for vendors
> You want to change this policy, and get rid of
> vendor ids in pci_ids.h completely?

Please read the top of pci_ids.h.  It hasn't had new ids added to it in
a long time.

> Bjorn, what do you think of this?
> 
> >  and pulling
> > them out of drivers isn't ok.
> 
> This patchset is not pulling any files out of drivers fwiw.

It did for the USB gadget driver patch that I commented on.

> > Userspace shouldn't need to know any of these, use libpci.
> 
> Unless I'm mistaken, libpci does not export a header with defines. It
> has a text file pci.ids, but parsing that when all I want is e.g. locate
> all intel devices is just too much overhead. No one wants that, so
> people just duplicate headers.

Why would userspace need the pci id of anything?  Again, just use
libpci, isn't it fast enough?  Don't duplicate existing logic.

Or use the hw database that libudev exports, which is already on your
machine and exports all of the pci ids from libpci directly.

> Standard class IDs are even sillier to duplicate.

Again, why does userspace need this?

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/