From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from nbfkord-smmo04.seg.att.com (nbfkord-smmo04.seg.att.com [209.65.160.86]) by dpdk.org (Postfix) with ESMTP id C44216910 for ; Fri, 25 Nov 2016 14:23:31 +0100 (CET) Received: from unknown [193.34.186.16] (EHLO webmail.solarflare.com) by nbfkord-smmo04.seg.att.com(mxl_mta-7.2.4-7) with ESMTP id 35b38385.2b081ae8a940.153693.00-2486.159571.nbfkord-smmo04.seg.att.com (envelope-from ); Fri, 25 Nov 2016 13:23:31 +0000 (UTC) X-MXL-Hash: 58383b531f964356-dd82294e0dcc8b8a7e32b57348b086df43ae2040 Received: from unknown [193.34.186.16] (EHLO webmail.solarflare.com) by nbfkord-smmo04.seg.att.com(mxl_mta-7.2.4-7) over TLS secured channel with ESMTP id 15b38385.0.153657.00-2396.159532.nbfkord-smmo04.seg.att.com (envelope-from ); Fri, 25 Nov 2016 13:23:30 +0000 (UTC) X-MXL-Hash: 58383b5240ee3770-45af9afd9911dec6c788b967055efcebf550ff2d Received: from [192.168.38.17] (84.52.89.52) by ukex01.SolarFlarecom.com (10.17.10.4) with Microsoft SMTP Server (TLS) id 15.0.1044.25; Fri, 25 Nov 2016 13:23:23 +0000 To: Thomas Monjalon References: <1479740470-6723-1-git-send-email-arybchenko@solarflare.com> <4008902.bqyKbWRKzy@xps13> CC: Ferruh Yigit , , Bruce Richardson From: Andrew Rybchenko Message-ID: Date: Fri, 25 Nov 2016 16:23:18 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.5.0 MIME-Version: 1.0 In-Reply-To: <4008902.bqyKbWRKzy@xps13> X-Originating-IP: [84.52.89.52] X-ClientProxiedBy: ocex03.SolarFlarecom.com (10.20.40.36) To ukex01.SolarFlarecom.com (10.17.10.4) X-TM-AS-Product-Ver: SMEX-11.0.0.1191-8.000.1202-22720.003 X-TM-AS-Result: No--26.282000-0.000000-31 X-TM-AS-User-Approved-Sender: Yes X-TM-AS-User-Blocked-Sender: No X-AnalysisOut: [v=2.1 cv=K6YkaniI c=1 sm=1 tr=0 a=8P+NB+fYZDP74ap4g4d9Kw==] X-AnalysisOut: [:17 a=RB3BGLmKESwA:10 a=L24OOQBejmoA:10 a=Z628TyH2xqVQVyhT] X-AnalysisOut: [9oQA:9 a=pILNOxqGKmIA:10 a=og8CCl2UTNZr_EF96AsA:9 a=vBOaQd] X-AnalysisOut: [ImVs3Ua9iJ:21 a=_W_S_7VecoQA:10] X-Spam: [F=0.2000000000; CM=0.500; S=0.200(2015072901)] X-MAIL-FROM: X-SOURCE-IP: [193.34.186.16] Content-Type: text/plain; charset="windows-1252"; format=flowed Content-Transfer-Encoding: 7bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Subject: Re: [dpdk-dev] [PATCH 00/56] Solarflare libefx-based PMD 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: Fri, 25 Nov 2016 13:23:32 -0000 On 11/25/2016 04:00 PM, Thomas Monjalon wrote: > 2016-11-25 12:43, Ferruh Yigit: >> On 11/25/2016 12:02 PM, Andrew Rybchenko wrote: >>> On 11/25/2016 01:24 PM, Ferruh Yigit wrote: >>>> On 11/23/2016 7:49 AM, Andrew Rybchenko wrote: >>>>> On 11/23/2016 03:02 AM, Ferruh Yigit wrote: >>>>>> Also folder structure is drivers/net/sfc/efx/, why /sfc/ >>>>>> layer is created? >>>>>> sfc is company name (solarflare communications), right? Other driver >>>>>> folders not structured based on company, what about using >>>>>> drivers/net/efx/* ? >>>>> I've tried to explain it above in item (2): >>>>> >>>>> >>> >>>>> >>>>> 2. Another Solarflare PMD with in-kernel part (for control operations) >>>>> is considered and could be added in the future. Code for data path >>>>> should be shared by these two drivers. libefx-based PMD is put into >>>>> 'efx' subdirectory to have a space for another PMD and shared code. >>>>> >>>>> <<< >>>>> >>>>> So, main reason is to have location for the code shared by two Solarflare >>>>> network PMDs. May be it better to relocate when we really have it. >>>>> I'm open for other ideas/suggestions. >>>> If there will be another PMD that shares code with current one, the >>>> logic seems good, but I am not sure about start using company names, I >>>> am not against it, just I don't know. >>> I think that mlx4 and mlx5 are tightly bound to the company name (plus >>> adapter generation, I guess). >>> >>>> Let's relocate later, this buys some time to think / get feedback on issue. >>> Do I understand correctly that you suggest to avoid extra level inside >>> for now >>> and relocate later if required? If so, that's fine for me. >>> >>> As for naming, we think that just "efx" is a bad idea. The prefix is >>> occupied by >>> the libefx functions and driver should use something else. We have chosen >>> "sfc" mainly to follow naming used in Linux kernel for Solarflare driver >>> (the first level of Ethernet driver names is company bound in the Linux >>> kernel). >>> If we avoid extra level as discussed above, I think "sfc_efx", "sfcefx" >>> (may be it >>> will look better nearby other drivers) or just "sfc" are fine for us. >>> >> Thomas, Bruce, any comment on this? > You can add multiple drivers in the same library. As an example, ixgbe > directory have several drivers for PF/VF, scalar/vector, etc. > If you really want separate directories for your drivers while sharing some > code, you can link some files from the other directory. Thanks for ideas/examples. I'll remove extra level. > About the name of this directory, I have no strong opinion. > sfcefx looks good. If there is no strong opinion against "sfc" , we'd prefer to keep it: 1. sfc is used in Linux (and, hopefully, well know) 2. sfc is shorter and already used in sources 3. sfc seems to be more future proof if we would like to add more drivers inside (efx binds it to libefx) 4. sfc could be read as well as Solarflare Flareon Controller ;) Andrew.