linux-pci.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: "Michael S. Tsirkin" <mst@redhat.com>
To: Greg KH <gregkh@linuxfoundation.org>
Cc: linux-kernel@vger.kernel.org, Bjorn Helgaas <bhelgaas@google.com>,
	Jonathan Corbet <corbet@lwn.net>,
	"David S. Miller" <davem@davemloft.net>,
	Hans Verkuil <hans.verkuil@cisco.com>,
	Mauro Carvalho Chehab <mchehab@osg.samsung.com>,
	Alexei Starovoitov <ast@plumgrid.com>,
	stephen hemminger <stephen@networkplumber.org>,
	Masahiro Yamada <yamada.m@jp.panasonic.com>,
	Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
	Andy Lutomirski <luto@amacapital.net>,
	Rasmus Villemoes <linux@rasmusvillemoes.dk>,
	Stephane Eranian <eranian@google.com>,
	Huang Rui <ray.huang@amd.com>,
	Peter Neubauer <pneubauer@bluerwhite.org>,
	linux-pci@vger.kernel.org, linux-doc@vger.kernel.org,
	linux-api@vger.kernel.org
Subject: Re: [PATCH 01/86] pci: export pci_ids.h
Date: Mon, 30 Mar 2015 13:41:28 +0200	[thread overview]
Message-ID: <20150330134002-mutt-send-email-mst@redhat.com> (raw)
In-Reply-To: <20150330113516.GA19501@kroah.com>

On Mon, Mar 30, 2015 at 01:35:16PM +0200, Greg KH wrote:
> On Mon, Mar 30, 2015 at 01:19:14PM +0200, Michael S. Tsirkin wrote:
> > Why there much more that two packages, I can find
> > at least 5 copies in the wild. Why? I think it's because it's part of
> > linux ABI that doesn't have matching headers. People are asked to build
> > their own, so of course they copy each other.
> > 
> > Once linux exports these headers everyone can stop
> > duplicating each other and others, and just use linux headers too.
> 
> As I'm not the PCI maintainer anymore, I'll let Bjorn decide about this.
> 
> But, your original patch series isn't ok, we don't want to export the
> pci vendor/device ids to userspace, that isn't the job of the kernel to
> do so.  If we should be exporting the pci class and config structures or
> not is Bjorn's decision.
> 
> greg k-h


I sent v2 doing exactly that.
Thanks a lot for the review.

-- 
MST

  reply	other threads:[~2015-03-30 11:42 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-03-29 13:36 [PATCH 00/86] pci: export pci_ids.h and related cleanups Michael S. Tsirkin
2015-03-29 13:37 ` [PATCH 01/86] pci: export pci_ids.h Michael S. Tsirkin
2015-03-29 15:49   ` Joe Perches
2015-03-29 20:40   ` Greg KH
2015-03-30  6:48     ` Michael S. Tsirkin
2015-03-30  6:55       ` Greg KH
2015-03-30  7:15         ` Michael S. Tsirkin
2015-03-30  7:53           ` Greg KH
2015-03-30  8:31             ` Michael S. Tsirkin
2015-03-30 10:07               ` Greg KH
2015-03-30 10:46                 ` Michael S. Tsirkin
2015-03-30 10:57                   ` Greg KH
2015-03-30 11:19                     ` Michael S. Tsirkin
2015-03-30 11:35                       ` Greg KH
2015-03-30 11:41                         ` Michael S. Tsirkin [this message]
2015-03-29 13:38 ` [PATCH 12/86] x86/pci/sta2x11: use uapi/linux/pci_ids.h directly Michael S. Tsirkin
2015-03-29 13:41 ` [PATCH 58/86] pci-label: " Michael S. Tsirkin
2015-03-29 13:43 ` [PATCH 77/86] linux/pci: " Michael S. Tsirkin
2015-03-29 13:43 ` [PATCH 78/86] linux/pci: drop include/linux/pci_ids.h Michael S. Tsirkin
2015-03-29 17:59 ` [PATCH 00/86] pci: export pci_ids.h and related cleanups Joe Perches
2015-03-30  6:52   ` Michael S. Tsirkin
2015-03-29 23:15 ` Stefan Richter
2015-04-02  7:44 ` Jean Delvare
2015-04-02  7:49   ` Michael S. Tsirkin
2015-04-02  8:23     ` Christoph Hellwig
2015-04-02  9:04       ` Jean Delvare
2015-04-02 10:09         ` Michael S. Tsirkin
2015-04-02 11:15           ` Jean Delvare
2015-04-02 12:05             ` Michael S. Tsirkin
2015-04-02 13:17               ` Jean Delvare
2015-04-02 12:09       ` Michael S. Tsirkin
2015-04-02 14:34         ` Alex Williamson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20150330134002-mutt-send-email-mst@redhat.com \
    --to=mst@redhat.com \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=ast@plumgrid.com \
    --cc=bhelgaas@google.com \
    --cc=corbet@lwn.net \
    --cc=davem@davemloft.net \
    --cc=eranian@google.com \
    --cc=gregkh@linuxfoundation.org \
    --cc=hans.verkuil@cisco.com \
    --cc=linux-api@vger.kernel.org \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=linux@rasmusvillemoes.dk \
    --cc=luto@amacapital.net \
    --cc=mchehab@osg.samsung.com \
    --cc=pneubauer@bluerwhite.org \
    --cc=ray.huang@amd.com \
    --cc=stephen@networkplumber.org \
    --cc=yamada.m@jp.panasonic.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).