From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 06/19] enic: move enic PMD to drivers/net directory Date: Wed, 20 May 2015 18:14:28 +0200 Message-ID: <2011983.lTvXhXMBTI@xps13> References: <1431450315-13179-1-git-send-email-bruce.richardson@intel.com> <1524103.VkqT0BVE0a@xps13> <59AF69C657FD0841A61C55336867B5B034539A33@IRSMSX103.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: "Richardson, Bruce" Return-path: Received: from mail-wg0-f51.google.com (mail-wg0-f51.google.com [74.125.82.51]) by dpdk.org (Postfix) with ESMTP id 466F2590C for ; Wed, 20 May 2015 18:15:14 +0200 (CEST) Received: by wghq2 with SMTP id q2so58260888wgh.1 for ; Wed, 20 May 2015 09:15:14 -0700 (PDT) In-Reply-To: <59AF69C657FD0841A61C55336867B5B034539A33@IRSMSX103.ger.corp.intel.com> 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" 2015-05-20 16:04, Richardson, Bruce: > From: Thomas Monjalon [mailto:thomas.monjalon@6wind.com] > > 2015-05-15 16:56, Bruce Richardson: > > > move enic PMD to drivers/net directory > > > > > > Signed-off-by: Bruce Richardson > > > --- > > > drivers/net/Makefile | 2 +- > > > drivers/net/enic/LICENSE | 27 + > > > drivers/net/enic/Makefile | 71 ++ > > > drivers/net/enic/enic.h | 200 +++++ > > > drivers/net/enic/enic_clsf.c | 259 ++++++ > > > drivers/net/enic/enic_compat.h | 147 ++++ > > > drivers/net/enic/enic_ethdev.c | 640 +++++++++++++++ > > > drivers/net/enic/enic_main.c | 1117 > > ++++++++++++++++++++++++++ > > > drivers/net/enic/enic_res.c | 219 +++++ > > > drivers/net/enic/enic_res.h | 168 ++++ > > > drivers/net/enic/rte_pmd_enic_version.map | 4 + > > > drivers/net/enic/vnic/cq_desc.h | 126 +++ > > > drivers/net/enic/vnic/cq_enet_desc.h | 261 ++++++ > > > drivers/net/enic/vnic/rq_enet_desc.h | 76 ++ > > > drivers/net/enic/vnic/vnic_cq.c | 117 +++ > > > drivers/net/enic/vnic/vnic_cq.h | 151 ++++ > > > drivers/net/enic/vnic/vnic_dev.c | 1054 > > ++++++++++++++++++++++++ > > > drivers/net/enic/vnic/vnic_dev.h | 212 +++++ > > > drivers/net/enic/vnic/vnic_devcmd.h | 774 ++++++++++++++++++ > > > drivers/net/enic/vnic/vnic_enet.h | 78 ++ > > > drivers/net/enic/vnic/vnic_intr.c | 78 ++ > > > drivers/net/enic/vnic/vnic_intr.h | 126 +++ > > > drivers/net/enic/vnic/vnic_nic.h | 88 ++ > > > drivers/net/enic/vnic/vnic_resource.h | 97 +++ > > > drivers/net/enic/vnic/vnic_rq.c | 245 ++++++ > > > drivers/net/enic/vnic/vnic_rq.h | 282 +++++++ > > > drivers/net/enic/vnic/vnic_rss.c | 85 ++ > > > drivers/net/enic/vnic/vnic_rss.h | 61 ++ > > > drivers/net/enic/vnic/vnic_stats.h | 86 ++ > > > drivers/net/enic/vnic/vnic_wq.c | 245 ++++++ > > > drivers/net/enic/vnic/vnic_wq.h | 283 +++++++ > > > drivers/net/enic/vnic/wq_enet_desc.h | 114 +++ > > > > I think that vnic/ should be renamed to base/ > > Yes, I was wondering about that. However, this wasn't a driver I was familiar with and its origins, so I left it as-is. Sujith imported it with this message: "VNIC common code partially shared with ENIC kernel mode driver" http://dpdk.org/ml/archives/dev/2014-November/008716.html