On Mon, 28 Sep 2020 01:42:43 +0200 Thomas Monjalon <tho...@monjalon.net> wrote:
> The secondary processes are not allowed to release shared resources. > Only process-private ressources should be freed in a secondary process. > Most of the time, there is no process-private ressource, > 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> Since this is a common issue across many drivers, my preference would be that the check should be in common code of eth_dev_close. It looks like only a few drivers would allow close from secondary, and that would not be that helpful to a user.
