Re: [dpdk-dev] [RFC] remove unused functions

[Ferruh Yigit]

On 11/24/2020 11:01 AM, Zhang, Tianfei wrote:


-----Original Message-----
From: Ferruh Yigit <ferruh.yi...@intel.com>
Sent: 2020年11月24日 18:26
To: Zhang, Tianfei <tianfei.zh...@intel.com>; Jerin Jacob
<jer...@marvell.com>; Dumitrescu, Cristian <cristian.dumitre...@intel.com>;
Hemant Agrawal <hemant.agra...@nxp.com>; Sachin Saxena
<sachin.sax...@oss.nxp.com>; Ray Kinsella <m...@ashroe.eu>; Neil Horman
<nhor...@tuxdriver.com>; Xu, Rosen <rosen...@intel.com>; Wu, Jingjing
<jingjing...@intel.com>; Xing, Beilei <beilei.x...@intel.com>; Nithin
Dabilpuram <ndabilpu...@marvell.com>; Ajit Khaparde
<ajit.khapa...@broadcom.com>; Raveendra Padasalagi
<raveendra.padasal...@broadcom.com>; Vikas Gupta
<vikas.gu...@broadcom.com>; Gagandeep Singh <g.si...@nxp.com>;
Somalapuram Amaranath <asoma...@amd.com>; Akhil Goyal
<akhil.go...@nxp.com>; Jay Zhou <jianjay.z...@huawei.com>; McDaniel,
Timothy <timothy.mcdan...@intel.com>; Ma, Liang J <liang.j...@intel.com>;
Mccarthy, Peter <peter.mccar...@intel.com>; Shepard Siegel
<shepard.sie...@atomicrules.com>; Ed Czeck <ed.cz...@atomicrules.com>;
John Miller <john.mil...@atomicrules.com>; Igor Russkikh
<igor.russk...@aquantia.com>; Pavel Belous <pavel.bel...@aquantia.com>;
Rasesh Mody <rm...@marvell.com>; Shahed Shaikh
<shsha...@marvell.com>; Somnath Kotur <somnath.ko...@broadcom.com>;
Chas Williams <ch...@att.com>; Min Hu (Connor) <humi...@huawei.com>;
Rahul Lakkireddy <rahul.lakkire...@chelsio.com>; Guo, Jia
<jia....@intel.com>; Wang, Haiyue <haiyue.w...@intel.com>; Marcin
Wojtas <m...@semihalf.com>; Michal Krawczyk <m...@semihalf.com>; Guy
Tzalik <gtza...@amazon.com>; Evgeny Schemeilin <evge...@amazon.com>;
Igor Chauskin <igo...@amazon.com>; Zhang, Qi Z <qi.z.zh...@intel.com>;
Wang, Xiao W <xiao.w.w...@intel.com>; Yang, Qiming
<qiming.y...@intel.com>; Alfredo Cardigliano <cardigli...@ntop.org>;
Matan Azrad <ma...@nvidia.com>; Shahaf Shuler <shah...@nvidia.com>;
Viacheslav Ovsiienko <viachesl...@nvidia.com>; Zyta Szpak
<z...@semihalf.com>; Liron Himi <lir...@marvell.com>; Stephen Hemminger
<sthem...@microsoft.com>; K. Y. Srinivasan <k...@microsoft.com>; Haiyang
Zhang <haiya...@microsoft.com>; Long Li <lon...@microsoft.com>; Heinrich
Kuhn <heinrich.k...@netronome.com>; Harman Kalra
<hka...@marvell.com>; Kiran Kumar K <kirankum...@marvell.com>; Andrew
Rybchenko <andrew.rybche...@oktetlabs.ru>; Singh, Jasvinder
<jasvinder.si...@intel.com>; Jiawen Wu <jiawe...@trustnetic.com>; Jian
Wang <jianw...@trustnetic.com>; Ori Kam <or...@nvidia.com>; Guy Kaneti
<g...@marvell.com>; Burakov, Anatoly <anatoly.bura...@intel.com>;
Maxime Coquelin <maxime.coque...@redhat.com>; Xia, Chenbo
<chenbo....@intel.com>
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 <ferruh.yi...@intel.com>
Sent: 2020年11月23日 17:57
To: Zhang, Tianfei <tianfei.zh...@intel.com>; Jerin Jacob
<jer...@marvell.com>; Dumitrescu, Cristian
<cristian.dumitre...@intel.com>; Hemant Agrawal
<hemant.agra...@nxp.com>; Sachin Saxena
<sachin.sax...@oss.nxp.com>;
Ray Kinsella <m...@ashroe.eu>; Neil Horman <nhor...@tuxdriver.com>;
Xu, Rosen <rosen...@intel.com>; Wu, Jingjing <jingjing...@intel.com>;
Xing, Beilei <beilei.x...@intel.com>; Nithin Dabilpuram
<ndabilpu...@marvell.com>; Ajit Khaparde
<ajit.khapa...@broadcom.com>; Raveendra Padasalagi
<raveendra.padasal...@broadcom.com>; Vikas Gupta
<vikas.gu...@broadcom.com>; Gagandeep Singh <g.si...@nxp.com>;
Somalapuram Amaranath <asoma...@amd.com>; Akhil Goyal
<akhil.go...@nxp.com>; Jay Zhou <jianjay.z...@huawei.com>; McDaniel,
Timothy <timothy.mcdan...@intel.com>; Ma, Liang J
<liang.j...@intel.com>; Mccarthy, Peter <peter.mccar...@intel.com>;
Shepard Siegel <shepard.sie...@atomicrules.com>; Ed Czeck
<ed.cz...@atomicrules.com>; John Miller
<john.mil...@atomicrules.com>; Igor Russkikh
<igor.russk...@aquantia.com>; Pavel Belous
<pavel.bel...@aquantia.com>; Rasesh Mody <rm...@marvell.com>;
Shahed
Shaikh <shsha...@marvell.com>; Somnath Kotur
<somnath.ko...@broadcom.com>; Chas Williams <ch...@att.com>; Min
Hu
(Connor) <humi...@huawei.com>; Rahul Lakkireddy
<rahul.lakkire...@chelsio.com>; Guo, Jia <jia....@intel.com>; Wang,
Haiyue <haiyue.w...@intel.com>; Marcin Wojtas <m...@semihalf.com>;
Michal Krawczyk <m...@semihalf.com>; Guy Tzalik <gtza...@amazon.com>;
Evgeny Schemeilin <evge...@amazon.com>; Igor Chauskin
<igo...@amazon.com>; Zhang, Qi Z <qi.z.zh...@intel.com>; Wang, Xiao
W
<xiao.w.w...@intel.com>; Yang, Qiming <qiming.y...@intel.com>;
Alfredo Cardigliano <cardigli...@ntop.org>; Matan Azrad
<ma...@nvidia.com>; Shahaf Shuler <shah...@nvidia.com>; Viacheslav
Ovsiienko <viachesl...@nvidia.com>; Zyta Szpak <z...@semihalf.com>;
Liron Himi <lir...@marvell.com>; Stephen Hemminger
<sthem...@microsoft.com>; K. Y. Srinivasan <k...@microsoft.com>;
Haiyang Zhang <haiya...@microsoft.com>; Long Li
<lon...@microsoft.com>; Heinrich Kuhn
<heinrich.k...@netronome.com>;
Harman Kalra <hka...@marvell.com>; Kiran Kumar K
<kirankum...@marvell.com>; Andrew Rybchenko
<andrew.rybche...@oktetlabs.ru>; Singh, Jasvinder
<jasvinder.si...@intel.com>; Jiawen Wu <jiawe...@trustnetic.com>;
Jian Wang <jianw...@trustnetic.com>; Ori Kam <or...@nvidia.com>; Guy
Kaneti <g...@marvell.com>; Burakov, Anatoly
<anatoly.bura...@intel.com>; Maxime Coquelin
<maxime.coque...@redhat.com>; Xia, Chenbo <chenbo....@intel.com>
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?