[dpdk-dev] [PATCH v2] ethdev: add namespace

[Xu, Rosen]

Hi,

> -----Original Message-----
> From: Wisam Monther <wisamm at nvidia.com>
> Sent: Sunday, August 29, 2021 16:18
> To: Ajit Khaparde <ajit.khaparde at broadcom.com>; Andrew Rybchenko
> <andrew.rybchenko at oktetlabs.ru>
> Cc: Yigit, Ferruh <ferruh.yigit at intel.com>; Tahhan, Maryam
> <maryam.tahhan at intel.com>; Pattan, Reshma <reshma.pattan at intel.com>;
> Jerin Jacob <jerinj at marvell.com>; Dumitrescu, Cristian
> <cristian.dumitrescu at intel.com>; Li, Xiaoyun <xiaoyun.li at intel.com>; NBU-
> Contact-Thomas Monjalon <thomas at monjalon.net>; Jayatheerthan, Jay
> <jay.jayatheerthan at intel.com>; Chas Williams <chas3 at att.com>; Min Hu
> (Connor) <humin29 at huawei.com>; Pavan Nikhilesh
> <pbhagavatula at marvell.com>; Shijith Thotton <sthotton at marvell.com>;
> Somnath Kotur <somnath.kotur at broadcom.com>; Daley, John
> <johndale at cisco.com>; Hyong Youb Kim <hyonkim at cisco.com>; Zhang, Qi Z
> <qi.z.zhang at intel.com>; Wang, Xiao W <xiao.w.wang at intel.com>; Xing,
> Beilei <beilei.xing at intel.com>; Wang, Haiyue <haiyue.wang at intel.com>;
> Matan Azrad <matan at nvidia.com>; Shahaf Shuler <shahafs at nvidia.com>;
> Slava Ovsiienko <viacheslavo at nvidia.com>; Wiles, Keith
> <keith.wiles at intel.com>; Hu, Jiayu <jiayu.hu at intel.com>; Olivier Matz
> <olivier.matz at 6wind.com>; Ori Kam <orika at nvidia.com>; Akhil Goyal
> <gakhil at marvell.com>; Doherty, Declan <declan.doherty at intel.com>; Ray
> Kinsella <mdr at ashroe.eu>; Nicolau, Radu <radu.nicolau at intel.com>;
> Hemant Agrawal <hemant.agrawal at nxp.com>; Sachin Saxena
> <sachin.saxena at oss.nxp.com>; Nithin Dabilpuram
> <ndabilpuram at marvell.com>; Kiran Kumar K <kirankumark at marvell.com>;
> Sunil Kumar Kori <skori at marvell.com>; Satha Rao
> <skoteshwar at marvell.com>; John W. Linville <linville at tuxdriver.com>;
> Loftus, Ciara <ciara.loftus at intel.com>; Shepard Siegel
> <shepard.siegel at atomicrules.com>; Ed Czeck <ed.czeck at atomicrules.com>;
> John Miller <john.miller at atomicrules.com>; Igor Russkikh
> <irusskikh at marvell.com>; Steven Webster
> <steven.webster at windriver.com>; Matt Peters
> <matt.peters at windriver.com>; Somalapuram Amaranath
> <asomalap at amd.com>; Rasesh Mody <rmody at marvell.com>; Shahed
> Shaikh <shshaikh at marvell.com>; Richardson, Bruce
> <bruce.richardson at intel.com>; Ananyev, Konstantin
> <konstantin.ananyev at intel.com>; Ruifeng Wang <ruifeng.wang at arm.com>;
> Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>; Marcin Wojtas
> <mw at semihalf.com>; Michal Krawczyk <mk at semihalf.com>; Shai Brandes
> <shaibran at amazon.com>; Evgeny Schemeilin <evgenys at amazon.com>;
> Igor Chauskin <igorch at amazon.com>; Gagandeep Singh
> <g.singh at nxp.com>; Gaetan Rivet <grive at u256.net>; Ziyang Xuan
> <xuanziyang2 at huawei.com>; Xiaoyun Wang
> <cloud.wangxiaoyun at huawei.com>; Guoyang Zhou
> <zhouguoyang at huawei.com>; Yisen Zhuang <yisen.zhuang at huawei.com>;
> Lijun Ou <oulijun at huawei.com>; Wu, Jingjing <jingjing.wu at intel.com>;
> Yang, Qiming <qiming.yang at intel.com>; Andrew Boyer
> <aboyer at pensando.io>; Xu, Rosen <rosen.xu at intel.com>;
> Srisivasubramanian Srinivasan <srinivasan at marvell.com>; Jakub Grajciar
> <jgrajcia at cisco.com>; Zyta Szpak <zr at semihalf.com>; Liron Himi
> <lironh at marvell.com>; Stephen Hemminger <sthemmin at microsoft.com>;
> NBU-Contact-longli <longli at microsoft.com>; Martin Spinler
> <spinler at cesnet.cz>; Heinrich Kuhn <heinrich.kuhn at netronome.com>;
> Jiawen Wu <jiawenwu at trustnetic.com>; Tetsuya Mukawa
> <mtetsuyah at gmail.com>; Harman Kalra <hkalra at marvell.com>; Anoob
> Joseph <anoobj at marvell.com>; Nalla Pradeep <pnalla at marvell.com>;
> Radha Mohan Chintakuntla <radhac at marvell.com>; Veerasenareddy Burru
> <vburru at marvell.com>; Devendra Singh Rawat
> <dsinghrawat at marvell.com>; Singh, Jasvinder <jasvinder.singh at intel.com>;
> Maciej Czekaj <mczekaj at marvell.com>; Jian Wang
> <jianwang at trustnetic.com>; Maxime Coquelin
> <maxime.coquelin at redhat.com>; Xia, Chenbo <chenbo.xia at intel.com>;
> Yong Wang <yongwang at vmware.com>; Chautru, Nicolas
> <nicolas.chautru at intel.com>; Hunt, David <david.hunt at intel.com>; Van
> Haaren, Harry <harry.van.haaren at intel.com>; Iremonger, Bernard
> <bernard.iremonger at intel.com>; Burakov, Anatoly
> <anatoly.burakov at intel.com>; Mcnamara, John
> <john.mcnamara at intel.com>; Rybalchenko, Kirill
> <kirill.rybalchenko at intel.com>; Marohn, Byron <byron.marohn at intel.com>;
> Wang, Yipeng1 <yipeng1.wang at intel.com>; dpdk-dev <dev at dpdk.org>;
> Tyler Retzlaff <roretzla at linux.microsoft.com>
> Subject: RE: [PATCH v2] ethdev: add namespace
> 
> Hi,
> 
> > -----Original Message-----
> > From: Ajit Khaparde <ajit.khaparde at broadcom.com>
> > Sent: Saturday, August 28, 2021 5:26 PM
> > To: Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
> > Cc: Ferruh Yigit <ferruh.yigit at intel.com>; Maryam Tahhan
> > <maryam.tahhan at intel.com>; Reshma Pattan
> <reshma.pattan at intel.com>;
> > Jerin Jacob <jerinj at marvell.com>; Wisam Monther
> <wisamm at nvidia.com>;
> > Cristian Dumitrescu <cristian.dumitrescu at intel.com>; Xiaoyun Li
> > <xiaoyun.li at intel.com>; NBU-Contact-Thomas Monjalon
> > <thomas at monjalon.net>; Jay Jayatheerthan
> > <jay.jayatheerthan at intel.com>; Chas Williams <chas3 at att.com>; Min Hu
> > (Connor) <humin29 at huawei.com>; Pavan Nikhilesh
> > <pbhagavatula at marvell.com>; Shijith Thotton <sthotton at marvell.com>;
> > Somnath Kotur <somnath.kotur at broadcom.com>; John Daley
> > <johndale at cisco.com>; Hyong Youb Kim <hyonkim at cisco.com>; Qi Zhang
> > <qi.z.zhang at intel.com>; Xiao Wang <xiao.w.wang at intel.com>; Beilei Xing
> > <beilei.xing at intel.com>; Haiyue Wang <haiyue.wang at intel.com>; Matan
> > Azrad <matan at nvidia.com>; Shahaf Shuler <shahafs at nvidia.com>; Slava
> > Ovsiienko <viacheslavo at nvidia.com>; Keith Wiles
> > <keith.wiles at intel.com>; Jiayu Hu <jiayu.hu at intel.com>; Olivier Matz
> > <olivier.matz at 6wind.com>; Ori Kam <orika at nvidia.com>; Akhil Goyal
> > <gakhil at marvell.com>; Declan Doherty <declan.doherty at intel.com>; Ray
> > Kinsella <mdr at ashroe.eu>; Radu Nicolau <radu.nicolau at intel.com>;
> > Hemant Agrawal <hemant.agrawal at nxp.com>; Sachin Saxena
> > <sachin.saxena at oss.nxp.com>; Nithin Dabilpuram
> > <ndabilpuram at marvell.com>; Kiran Kumar K <kirankumark at marvell.com>;
> > Sunil Kumar Kori <skori at marvell.com>; Satha Rao
> > <skoteshwar at marvell.com>; John W. Linville <linville at tuxdriver.com>;
> > Ciara Loftus <ciara.loftus at intel.com>; Shepard Siegel
> > <shepard.siegel at atomicrules.com>; Ed Czeck
> <ed.czeck at atomicrules.com>;
> > John Miller <john.miller at atomicrules.com>; Igor Russkikh
> > <irusskikh at marvell.com>; Steven Webster
> > <steven.webster at windriver.com>; Matt Peters
> > <matt.peters at windriver.com>; Somalapuram Amaranath
> <asomalap at amd.com>;
> > Rasesh Mody <rmody at marvell.com>; Shahed Shaikh
> <shshaikh at marvell.com>;
> > Bruce Richardson <bruce.richardson at intel.com>; Konstantin Ananyev
> > <konstantin.ananyev at intel.com>; Ruifeng Wang
> <ruifeng.wang at arm.com>;
> > Rahul Lakkireddy <rahul.lakkireddy at chelsio.com>; Marcin Wojtas
> > <mw at semihalf.com>; Michal Krawczyk <mk at semihalf.com>; Shai Brandes
> > <shaibran at amazon.com>; Evgeny Schemeilin <evgenys at amazon.com>;
> Igor
> > Chauskin <igorch at amazon.com>; Gagandeep Singh <g.singh at nxp.com>;
> > Gaetan Rivet <grive at u256.net>; Ziyang Xuan <xuanziyang2 at huawei.com>;
> > Xiaoyun Wang <cloud.wangxiaoyun at huawei.com>; Guoyang Zhou
> > <zhouguoyang at huawei.com>; Yisen Zhuang
> <yisen.zhuang at huawei.com>;
> > Lijun Ou <oulijun at huawei.com>; Jingjing Wu <jingjing.wu at intel.com>;
> > Qiming Yang <qiming.yang at intel.com>; Andrew Boyer
> > <aboyer at pensando.io>; Rosen Xu <rosen.xu at intel.com>;
> > Srisivasubramanian Srinivasan <srinivasan at marvell.com>; Jakub Grajciar
> > <jgrajcia at cisco.com>; Zyta Szpak <zr at semihalf.com>; Liron Himi
> > <lironh at marvell.com>; Stephen Hemminger <sthemmin at microsoft.com>;
> > NBU-Contact-longli <longli at microsoft.com>; Martin Spinler
> > <spinler at cesnet.cz>; Heinrich Kuhn <heinrich.kuhn at netronome.com>;
> > Jiawen Wu <jiawenwu at trustnetic.com>; Tetsuya Mukawa
> > <mtetsuyah at gmail.com>; Harman Kalra <hkalra at marvell.com>; Anoob
> Joseph
> > <anoobj at marvell.com>; Nalla Pradeep <pnalla at marvell.com>; Radha
> Mohan
> > Chintakuntla <radhac at marvell.com>; Veerasenareddy Burru
> > <vburru at marvell.com>; Devendra Singh Rawat
> <dsinghrawat at marvell.com>;
> > Jasvinder Singh <jasvinder.singh at intel.com>; Maciej Czekaj
> > <mczekaj at marvell.com>; Jian Wang <jianwang at trustnetic.com>; Maxime
> > Coquelin <maxime.coquelin at redhat.com>; Chenbo Xia
> > <chenbo.xia at intel.com>; Yong Wang <yongwang at vmware.com>; Nicolas
> > Chautru <nicolas.chautru at intel.com>; David Hunt
> > <david.hunt at intel.com>; Harry van Haaren <harry.van.haaren at intel.com>;
> > Bernard Iremonger <bernard.iremonger at intel.com>; Anatoly Burakov
> > <anatoly.burakov at intel.com>; John McNamara
> <john.mcnamara at intel.com>;
> > Kirill Rybalchenko <kirill.rybalchenko at intel.com>; Byron Marohn
> > <byron.marohn at intel.com>; Yipeng Wang <yipeng1.wang at intel.com>;
> > dpdk-dev <dev at dpdk.org>; Tyler Retzlaff <roretzla at linux.microsoft.com>
> > Subject: Re: [PATCH v2] ethdev: add namespace
> >
> > On Fri, Aug 27, 2021 at 12:59 AM Andrew Rybchenko
> > <andrew.rybchenko at oktetlabs.ru> wrote:
> > >
> > > On 8/27/21 4:19 AM, Ferruh Yigit wrote:
> > > > Add 'RTE_ETH' namespace to all enums & macros in a backward
> > compatible
> > > > way. The macros for backward compatibility can be removed in next LTS.
> > > >
> > > > Internal components switched to new enum & macro names.
> > > >
> > > > Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
> > > > Acked-By: Tyler Retzlaff <roretzla at linux.microsoft.com>
> > >
> > > Acked-by: Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>
> > Acked-by: Ajit Khaparde <ajit.khaparde at broadcom.com>
> 
> Acked-by: Wisam Jaddo <wisamm at nvidia.com>

Acked-by: Rosen Xu <rosen.xu at intel.com>

> >
> > >
> > > I think that it should be pushed as early as possible in the release
> > > cycle.
> > >
> > > The changeset definitely deserves entry in release notes.