From mboxrd@z Thu Jan 1 00:00:00 1970 From: Panu Matilainen Subject: Re: [PATCH] pci: Add the class_id support in pci probe Date: Wed, 13 Jan 2016 14:22:30 +0200 Message-ID: <56964186.1060800@redhat.com> References: <1451357606-117892-1-git-send-email-ziye.yang@intel.com> <20151231091214.0acd9d4e@xeon-e3> <20160113115503.GC7216@bricha3-MOBL3> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Cc: Ziye Yang , dev@dpdk.org To: Bruce Richardson , Stephen Hemminger Return-path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 681BF379E for ; Wed, 13 Jan 2016 13:22:33 +0100 (CET) In-Reply-To: <20160113115503.GC7216@bricha3-MOBL3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 01/13/2016 01:55 PM, Bruce Richardson wrote: > On Thu, Dec 31, 2015 at 09:12:14AM -0800, Stephen Hemminger wrote: >> On Tue, 29 Dec 2015 10:53:26 +0800 >> Ziye Yang wrote: >> >>> This patch is used to add the class_id support >>> for pci_probe since some devices need the class_info >>> (class_code, subclass_code, programming_interface) >>> >>> Signed-off-by: Ziye Yang >> >> Since rte_pci is exposed to application this breaks the ABI. > > But applications are not going to be defining rte_pci_ids values internally, are > they? That is for drivers to use. Is this really an ABI breakage for applications that we > need to be concerned about? There might not be applications using it but drivers are ABI consumers too - think of 3rd party drivers and such. - Panu -