From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id CB2332BBD for ; Fri, 18 May 2018 10:12:47 +0200 (CEST) X-Amp-Result: UNSCANNABLE X-Amp-File-Uploaded: False Received: from fmsmga005.fm.intel.com ([10.253.24.32]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 18 May 2018 01:12:46 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,413,1520924400"; d="scan'208";a="229603772" Received: from bricha3-mobl.ger.corp.intel.com ([10.237.221.55]) by fmsmga005.fm.intel.com with SMTP; 18 May 2018 01:12:43 -0700 Received: by (sSMTP sendmail emulation); Fri, 18 May 2018 09:12:42 +0100 Date: Fri, 18 May 2018 09:12:42 +0100 From: Bruce Richardson To: "Wang, Xiao W" Cc: "dev@dpdk.org" Message-ID: <20180518081241.GA28620@bricha3-MOBL.ger.corp.intel.com> References: <20180517201526.28658-1-bruce.richardson@intel.com> <20180517201526.28658-9-bruce.richardson@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Research and Development Ireland Ltd. User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH-18.08 08/15] net/ifc: rename to ifcvf X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 18 May 2018 08:12:48 -0000 On Fri, May 18, 2018 at 02:52:36AM +0100, Wang, Xiao W wrote: > Hi Bruce, > > > -----Original Message----- > > From: Richardson, Bruce > > Sent: Friday, May 18, 2018 4:15 AM > > To: dev@dpdk.org > > Cc: Richardson, Bruce ; Wang, Xiao W > > > > Subject: [PATCH-18.08 08/15] net/ifc: rename to ifcvf > > > > All files in the directory and the resulting driver have prefix of ifcvf, > > not just ifc, so rename directory for accuracy. Also rename the map file > > to standard name for meson build in the process. > > Naming the directory as "ifc" allows us to add ifcpf driver into it in the future. > > BRs, > Xiao > At which point you will have to rename the version file, the rst documentation file, the resulting shared library file etc. Right now, most of the references are to ifcvf, with the only exception being the folder name. This patch makes things consistent by having everything refer to ifcvf. If you prefer, I can do a v2 of this set renaming everything to ifc, but that would be a lot bigger a job, and would also result in the driver file itself getting a new name too. Unless there are immediate plans to add an ifcpf driver, I think this change makes more sense. /Bruce