From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 8264991A9 for ; Mon, 31 Aug 2015 12:23:42 +0200 (CEST) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 31 Aug 2015 03:23:40 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,440,1437462000"; d="scan'208";a="779262113" Received: from irsmsx151.ger.corp.intel.com ([163.33.192.59]) by fmsmga001.fm.intel.com with ESMTP; 31 Aug 2015 03:23:40 -0700 Received: from irsmsx108.ger.corp.intel.com ([169.254.11.12]) by IRSMSX151.ger.corp.intel.com ([169.254.4.37]) with mapi id 14.03.0224.002; Mon, 31 Aug 2015 11:23:34 +0100 From: "Iremonger, Bernard" To: "John W. Linville" Thread-Topic: [dpdk-dev] [RFC PATCH 0/6] remove pci driver from vdevs Thread-Index: AQHQ4PAprrsGaqoMN02VOjrYJZek+Z4hEK2AgACRCYCABD9tgA== Date: Mon, 31 Aug 2015 10:23:33 +0000 Message-ID: <8CEF83825BEC744B83065625E567D7C219F493A6@IRSMSX108.ger.corp.intel.com> References: <1440690041-32391-1-git-send-email-bernard.iremonger@intel.com> <20150827174357.GC8113@tuxdriver.com> <8CEF83825BEC744B83065625E567D7C219F4883E@IRSMSX108.ger.corp.intel.com> <20150828175133.GC10601@tuxdriver.com> In-Reply-To: <20150828175133.GC10601@tuxdriver.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [163.33.239.180] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [RFC PATCH 0/6] remove pci driver from vdevs X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 31 Aug 2015 10:23:42 -0000 Hi John, > -----Original Message----- > From: John W. Linville [mailto:linville@tuxdriver.com] > Sent: Friday, August 28, 2015 6:52 PM > To: Iremonger, Bernard > Cc: dev@dpdk.org > Subject: Re: [dpdk-dev] [RFC PATCH 0/6] remove pci driver from vdevs >=20 > On Fri, Aug 28, 2015 at 08:15:47AM +0000, Iremonger, Bernard wrote: > > Hi John, > > > > > -----Original Message----- > > > From: John W. Linville [mailto:linville@tuxdriver.com] > > > Sent: Thursday, August 27, 2015 6:44 PM > > > To: Iremonger, Bernard > > > Cc: dev@dpdk.org > > > Subject: Re: [dpdk-dev] [RFC PATCH 0/6] remove pci driver from vdevs > > > > > > On Thu, Aug 27, 2015 at 04:40:35PM +0100, Bernard Iremonger wrote: > > > > There is a dummy pci driver in the vdev PMD's at present. > > > > This RFC proposes to remove the pci driver from the vdev PMD's. > > > > Changes have been made to librte_ether to handle vdevs which do > > > > not > > > have a pci driver. > > > > > > > > The pdev PMD's should work as before with the changes to > > > > librte_ether The vdev PMD's which still have a pci driver should > > > > work as before with the > > > librte_ether changes. > > > > > > > > The following vdev PMD's have had the pci driver removed > > > > > > > > bonding PMD > > > > null PMD > > > > pcap PMD > > > > ring PMD > > > > > > Any reason there is no patch for the af_packet driver? > > > > > > John > > > > I have just modified the Intel vdev PMD's. > > It would be best if the owners of the non Intel vdev's submitted patche= s > for their drivers. >=20 > What constitutes an "Intel vdev PMD"? I thought these were all part of t= he > DPDK project? It seems odd to me for you to pick and choose like this. I should probably have written vdev PMD's contributed by Intel. I am not familiar with the other vdev PMD's and thought it best that they s= hould be modified by their owners/maintainers if required. >=20 > What is the overall purpose of this RFC?=20 The purpose of this RFC is to remove the need for a PCI device driver from = Vdev's that that do not use a PCI driver. Removing the PCI driver is imple= mented in the ethdev changes. I have modified some of the Vdev's to verify = that the ethdev changes work. > What benefit accrues to those vdev > PMDs that implement this change? What penalty is imposed on those that > do not change? 6Wind have decided that only cleanup patches will be allowed in future for = Vdevs that have a dummy PCI driver. Any change in functionality for these V= devs will not be allowed. Please see email below from 6Wind http://dpdk.org/ml/archives/dev/2015-July/022107.html >=20 > John > -- > John W. Linville Someday the world will need a hero, and you > linville@tuxdriver.com might be all we have. Be ready. Regards, Bernard.