> -----Original Message----- > From: Thomas Monjalon <tho...@monjalon.net> > Sent: Tuesday, September 29, 2020 07:15 > To: dev@dpdk.org > Cc: Yigit, Ferruh <ferruh.yi...@intel.com>; arybche...@solarflare.com; Xu, > Rosen <rosen...@intel.com>; > Sachin Saxena <sachin.sax...@oss.nxp.com>; Ajit Khaparde > <ajit.khapa...@broadcom.com>; Liron Himi > <lir...@marvell.com>; Loftus, Ciara <ciara.lof...@intel.com>; Zhang, Qi Z > <qi.z.zh...@intel.com>; > Shepard Siegel <shepard.sie...@atomicrules.com>; Ed Czeck > <ed.cz...@atomicrules.com>; John Miller > <john.mil...@atomicrules.com>; Steven Webster <steven.webs...@windriver.com>; > Matt Peters > <matt.pet...@windriver.com>; Somnath Kotur <somnath.ko...@broadcom.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>; > Gagandeep Singh > <g.si...@nxp.com>; John Daley <johnd...@cisco.com>; Hyong Youb Kim > <hyon...@cisco.com>; Wang, Xiao W > <xiao.w.w...@intel.com>; Ziyang Xuan <xuanziya...@huawei.com>; Xiaoyun Wang > <cloud.wangxiao...@huawei.com>; Guoyang Zhou <zhouguoy...@huawei.com>; Xing, > Beilei > <beilei.x...@intel.com>; Wu, Jingjing <jingjing...@intel.com>; Yang, Qiming > <qiming.y...@intel.com>; > Alfredo Cardigliano <cardigli...@ntop.org>; Shijith Thotton > <sthot...@marvell.com>; Srisivasubramanian > Srinivasan <sriniva...@marvell.com>; Matan Azrad <ma...@nvidia.com>; Shahaf > Shuler > <shah...@nvidia.com>; Zyta Szpak <z...@semihalf.com>; Stephen Hemminger > <sthem...@microsoft.com>; K. Y. > Srinivasan <k...@microsoft.com>; Haiyang Zhang <haiya...@microsoft.com>; Long > Li <lon...@microsoft.com>; > Martin Spinler <spin...@cesnet.cz>; Heinrich Kuhn > <heinrich.k...@netronome.com>; Harman Kalra > <hka...@marvell.com>; Akhil Goyal <akhil.go...@nxp.com>; Jerin Jacob > <jer...@marvell.com>; Maciej > Czekaj <mcze...@marvell.com>; Maxime Coquelin <maxime.coque...@redhat.com>; > Xia, Chenbo > <chenbo....@intel.com>; Wang, Zhihong <zhihong.w...@intel.com>; Yong Wang > <yongw...@vmware.com>; > Burakov, Anatoly <anatoly.bura...@intel.com> > Subject: [PATCH v3 23/29] drivers/net: check process type in close operation > > The secondary processes are not allowed to release shared resources. > Only process-private resources should be freed in a secondary process. > Most of the time, there is no process-private resource, > so the close operation is just forbidden in a secondary process. > > After adding proper check in the port close functions, > some redundant checks in the device remove functions are dropped. > > Signed-off-by: Thomas Monjalon <tho...@monjalon.net> > Reviewed-by: Rosen Xu <rosen...@intel.com> > Reviewed-by: Sachin Saxena <sachin.sax...@oss.nxp.com> > Reviewed-by: Ajit Khaparde <ajit.khapa...@broadcom.com> > Reviewed-by: Liron Himi <lir...@marvell.com> > ---
> drivers/net/e1000/em_ethdev.c | 3 +++ > drivers/net/e1000/igb_ethdev.c | 6 +++++ > drivers/net/igc/igc_ethdev.c | 6 ++--- > drivers/net/ixgbe/ixgbe_ethdev.c | 4 ++++ For e1000/igc/ixgbe Reviewed-by: Haiyue Wang <haiyue.w...@intel.com> > -- > 2.28.0
