> -----Original Message----- > From: Thomas Monjalon <[email protected]> > Sent: Tuesday, September 29, 2020 07:15 > To: [email protected] > Cc: Yigit, Ferruh <[email protected]>; [email protected]; Xu, > Rosen <[email protected]>; > Sachin Saxena <[email protected]>; Ajit Khaparde > <[email protected]>; Liron Himi > <[email protected]>; Loftus, Ciara <[email protected]>; Zhang, Qi Z > <[email protected]>; > Shepard Siegel <[email protected]>; Ed Czeck > <[email protected]>; John Miller > <[email protected]>; Steven Webster <[email protected]>; > Matt Peters > <[email protected]>; Somnath Kotur <[email protected]>; > Rahul Lakkireddy > <[email protected]>; Guo, Jia <[email protected]>; Wang, Haiyue > <[email protected]>; > Marcin Wojtas <[email protected]>; Michal Krawczyk <[email protected]>; Guy > Tzalik <[email protected]>; > Evgeny Schemeilin <[email protected]>; Igor Chauskin <[email protected]>; > Gagandeep Singh > <[email protected]>; John Daley <[email protected]>; Hyong Youb Kim > <[email protected]>; Wang, Xiao W > <[email protected]>; Ziyang Xuan <[email protected]>; Xiaoyun Wang > <[email protected]>; Guoyang Zhou <[email protected]>; Xing, > Beilei > <[email protected]>; Wu, Jingjing <[email protected]>; Yang, Qiming > <[email protected]>; > Alfredo Cardigliano <[email protected]>; Shijith Thotton > <[email protected]>; Srisivasubramanian > Srinivasan <[email protected]>; Matan Azrad <[email protected]>; Shahaf > Shuler > <[email protected]>; Zyta Szpak <[email protected]>; Stephen Hemminger > <[email protected]>; K. Y. > Srinivasan <[email protected]>; Haiyang Zhang <[email protected]>; Long > Li <[email protected]>; > Martin Spinler <[email protected]>; Heinrich Kuhn > <[email protected]>; Harman Kalra > <[email protected]>; Akhil Goyal <[email protected]>; Jerin Jacob > <[email protected]>; Maciej > Czekaj <[email protected]>; Maxime Coquelin <[email protected]>; > Xia, Chenbo > <[email protected]>; Wang, Zhihong <[email protected]>; Yong Wang > <[email protected]>; > Burakov, Anatoly <[email protected]> > 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 <[email protected]> > Reviewed-by: Rosen Xu <[email protected]> > Reviewed-by: Sachin Saxena <[email protected]> > Reviewed-by: Ajit Khaparde <[email protected]> > Reviewed-by: Liron Himi <[email protected]> > ---
> 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 <[email protected]> > -- > 2.28.0
