From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id AB5B9A04B1; Tue, 24 Nov 2020 12:31:53 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 04D08C90E; Tue, 24 Nov 2020 12:31:51 +0100 (CET) Received: from mga04.intel.com (mga04.intel.com [192.55.52.120]) by dpdk.org (Postfix) with ESMTP id 54FEDC902 for ; Tue, 24 Nov 2020 12:31:48 +0100 (CET) IronPort-SDR: viyQMV57pg50S0NnXrvNuaaxF8wO+PDkjRRr1pzbmg2PJgkAlCeaLRMRHpZ3YSxwaiyY7n4muI Oh3b7XrVui3w== X-IronPort-AV: E=McAfee;i="6000,8403,9814"; a="169365668" X-IronPort-AV: E=Sophos;i="5.78,366,1599548400"; d="scan'208";a="169365668" X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from orsmga007.jf.intel.com ([10.7.209.58]) by fmsmga104.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Nov 2020 03:31:24 -0800 IronPort-SDR: uuG8MvZ3ToIwWjxkgYNZC8/ABG+vuXHqwojxL3w4CcPeRbAjlLOthEGBFP24XyuMvhbcPWCjOL DjtAozFif3Tw== X-IronPort-AV: E=Sophos;i="5.78,366,1599548400"; d="scan'208";a="370341273" Received: from fyigit-mobl1.ger.corp.intel.com (HELO [10.251.85.132]) ([10.251.85.132]) by orsmga007-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 24 Nov 2020 03:31:11 -0800 To: "Zhang, Tianfei" , Jerin Jacob , "Dumitrescu, Cristian" , Hemant Agrawal , Sachin Saxena , Ray Kinsella , Neil Horman , "Xu, Rosen" , "Wu, Jingjing" , "Xing, Beilei" , Nithin Dabilpuram , Ajit Khaparde , Raveendra Padasalagi , Vikas Gupta , Gagandeep Singh , Somalapuram Amaranath , Akhil Goyal , Jay Zhou , "McDaniel, Timothy" , "Ma, Liang J" , "Mccarthy, Peter" , Shepard Siegel , Ed Czeck , John Miller , Igor Russkikh , Pavel Belous , Rasesh Mody , Shahed Shaikh , Somnath Kotur , Chas Williams , "Min Hu (Connor)" , Rahul Lakkireddy , "Guo, Jia" , "Wang, Haiyue" , Marcin Wojtas , Michal Krawczyk , Guy Tzalik , Evgeny Schemeilin , Igor Chauskin , "Zhang, Qi Z" , "Wang, Xiao W" , "Yang, Qiming" , Alfredo Cardigliano , Matan Azrad , Shahaf Shuler , Viacheslav Ovsiienko , Zyta Szpak , Liron Himi , Stephen Hemminger , "K. Y. Srinivasan" , Haiyang Zhang , Long Li , Heinrich Kuhn , Harman Kalra , Kiran Kumar K , Andrew Rybchenko , "Singh, Jasvinder" , Jiawen Wu , Jian Wang , Ori Kam , Guy Kaneti , "Burakov, Anatoly" , Maxime Coquelin , "Xia, Chenbo" Cc: "dev@dpdk.org" References: <20201119035238.3653702-1-ferruh.yigit@intel.com> <692cbe0f-618e-ad65-427a-0309c49bc424@intel.com> From: Ferruh Yigit Message-ID: <4613b0c1-c7fe-0f89-7cf6-a9acd3a3cd84@intel.com> Date: Tue, 24 Nov 2020 11:31:07 +0000 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 8bit Subject: Re: [dpdk-dev] [RFC] remove unused functions 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: , Errors-To: dev-bounces@dpdk.org Sender: "dev" On 11/24/2020 11:01 AM, Zhang, Tianfei wrote: > > >> -----Original Message----- >> From: Ferruh Yigit >> Sent: 2020年11月24日 18:26 >> To: Zhang, Tianfei ; Jerin Jacob >> ; Dumitrescu, Cristian ; >> Hemant Agrawal ; Sachin Saxena >> ; Ray Kinsella ; Neil Horman >> ; Xu, Rosen ; Wu, Jingjing >> ; Xing, Beilei ; Nithin >> Dabilpuram ; Ajit Khaparde >> ; Raveendra Padasalagi >> ; Vikas Gupta >> ; Gagandeep Singh ; >> Somalapuram Amaranath ; Akhil Goyal >> ; Jay Zhou ; McDaniel, >> Timothy ; Ma, Liang J ; >> Mccarthy, Peter ; Shepard Siegel >> ; Ed Czeck ; >> John Miller ; Igor Russkikh >> ; Pavel Belous ; >> Rasesh Mody ; Shahed Shaikh >> ; Somnath Kotur ; >> Chas Williams ; Min Hu (Connor) ; >> Rahul Lakkireddy ; Guo, Jia >> ; Wang, Haiyue ; Marcin >> Wojtas ; Michal Krawczyk ; Guy >> Tzalik ; Evgeny Schemeilin ; >> Igor Chauskin ; Zhang, Qi Z ; >> Wang, Xiao W ; Yang, Qiming >> ; Alfredo Cardigliano ; >> Matan Azrad ; Shahaf Shuler ; >> Viacheslav Ovsiienko ; Zyta Szpak >> ; Liron Himi ; Stephen Hemminger >> ; K. Y. Srinivasan ; Haiyang >> Zhang ; Long Li ; Heinrich >> Kuhn ; Harman Kalra >> ; Kiran Kumar K ; Andrew >> Rybchenko ; Singh, Jasvinder >> ; Jiawen Wu ; Jian >> Wang ; Ori Kam ; Guy Kaneti >> ; Burakov, Anatoly ; >> Maxime Coquelin ; Xia, Chenbo >> >> Cc: dev@dpdk.org >> Subject: Re: [RFC] remove unused functions >> >> On 11/24/2020 10:00 AM, Zhang, Tianfei wrote: >>> >>> >>>> -----Original Message----- >>>> From: Ferruh Yigit >>>> Sent: 2020年11月23日 17:57 >>>> To: Zhang, Tianfei ; Jerin Jacob >>>> ; Dumitrescu, Cristian >>>> ; Hemant Agrawal >>>> ; Sachin Saxena >> ; >>>> Ray Kinsella ; Neil Horman ; >>>> Xu, Rosen ; Wu, Jingjing ; >>>> Xing, Beilei ; Nithin Dabilpuram >>>> ; Ajit Khaparde >>>> ; Raveendra Padasalagi >>>> ; Vikas Gupta >>>> ; Gagandeep Singh ; >>>> Somalapuram Amaranath ; Akhil Goyal >>>> ; Jay Zhou ; McDaniel, >>>> Timothy ; Ma, Liang J >>>> ; Mccarthy, Peter ; >>>> Shepard Siegel ; Ed Czeck >>>> ; John Miller >>>> ; Igor Russkikh >>>> ; Pavel Belous >>>> ; Rasesh Mody ; >> Shahed >>>> Shaikh ; Somnath Kotur >>>> ; Chas Williams ; Min >> Hu >>>> (Connor) ; Rahul Lakkireddy >>>> ; Guo, Jia ; Wang, >>>> Haiyue ; Marcin Wojtas ; >>>> Michal Krawczyk ; Guy Tzalik ; >>>> Evgeny Schemeilin ; Igor Chauskin >>>> ; Zhang, Qi Z ; Wang, Xiao >> W >>>> ; Yang, Qiming ; >>>> Alfredo Cardigliano ; Matan Azrad >>>> ; Shahaf Shuler ; Viacheslav >>>> Ovsiienko ; Zyta Szpak ; >>>> Liron Himi ; Stephen Hemminger >>>> ; K. Y. Srinivasan ; >>>> Haiyang Zhang ; Long Li >>>> ; Heinrich Kuhn >> ; >>>> Harman Kalra ; Kiran Kumar K >>>> ; Andrew Rybchenko >>>> ; Singh, Jasvinder >>>> ; Jiawen Wu ; >>>> Jian Wang ; Ori Kam ; Guy >>>> Kaneti ; Burakov, Anatoly >>>> ; Maxime Coquelin >>>> ; Xia, Chenbo >>>> Cc: dev@dpdk.org >>>> Subject: Re: [RFC] remove unused functions >>>> >>>> On 11/23/2020 2:55 AM, Zhang, Tianfei wrote: >>>>>> drivers/raw/ifpga/base/opae_eth_group.c | 25 - >>>>>> drivers/raw/ifpga/base/opae_eth_group.h | 1 - > > For those unused functions on opae_eth_group.c are release functions, like eth_group_release(), it should be called by fme_eth_group_uinit(), > I will send a patch to fix it. > >>>>>> drivers/raw/ifpga/base/opae_hw_api.c | 212 -- >>>>>> drivers/raw/ifpga/base/opae_hw_api.h | 36 - >>>>>> drivers/raw/ifpga/base/opae_i2c.c | 12 - >>>>>> drivers/raw/ifpga/base/opae_i2c.h | 4 - >>>>>> drivers/raw/ifpga/base/opae_ifpga_hw_api.c | 99 - >>>>>> drivers/raw/ifpga/base/opae_ifpga_hw_api.h | 15 - >>>>> >>>>> The ifpga base code is want to provide fully functionality low level >>>>> hardware >>>> support for Intel FPGA card like N3000, even though some APIs are not >>>> used by DPDK framework now，but it will useful for end-users or other >>>> customers developing their own productions , my opinion is keeping >>>> those APIs of ifpga base code in DPDK. >>>>> >>>> >>>> The code is hidden behind the driver code, it is not accessible to user >> directly. >>>> >>>> If the idea is the code can be used later, it can be added when needed. >>> >>> Sound good! >>>> >>>> If the idea is the code can be needed in the field by users, can you >>>> please give example on the use case? >>> >>> Do you mean write some test cases in examples folder? >>> >> >> You mentioned end user may need some of the removed functions, can you >> give an example how an end user may need them? > > Currently, there is only one DPDK PMD driver using the ifpga base code, it is called ipn3ke, which is target for VBNG applications. > The ipn3ke is not using all of our APIs which provided by ifpga base code. I am agree remove the unused functions firstly. And I plan to write > an test application to test all of APIs ( including those unused functions ) and add those functions back in future, and I think the test applications should be a good example for the end-user how to use those APIs. > I am not clear how a test application can help to test driver base code functions. Driver should call them to configure the HW, if there is no DPDK API to request that configuration, or current device doesn't support it, what application can do?