Re: [PATCH] net/memif: allow stopping and closing device

2021-11-24 Thread Ferruh Yigit
On 11/18/2021 5:33 PM, Junxiao Shi wrote: Bugzilla ID: 888 Fixes: febc855b358e ("ethdev: forbid closing started device") Signed-off-by: Junxiao Shi Reviewed-by: Ferruh Yigit Applied to dpdk-next-net/main, thanks.

Re: [PATCH] net/memif: allow stopping and closing device

2021-11-24 Thread Ferruh Yigit
On 11/24/2021 2:01 PM, yoursunny wrote: Hi Junxiao, comment moved down, please don't top post, it makes following the discussion very hard in the archive. Yours, Junxiao On Wed, Nov 24, 2021, 06:02 Ferruh Yigit mailto:ferruh.yi...@intel.com>> wrote: On 11/18/2021 5:33 PM, Junxiao Shi wrot

Re: [PATCH] net/memif: allow stopping and closing device

2021-11-24 Thread yoursunny
Hi Ferruh You have to rely on user to call stop before calling close/remove. This is mandated in ethdev library, as implemented in: febc855b358e ("ethdev: forbid closing started device") Yours, Junxiao On Wed, Nov 24, 2021, 06:02 Ferruh Yigit wrote: > On 11/18/2021 5:33 PM, Junxiao Shi wrote:

Re: [PATCH] net/memif: allow stopping and closing device

2021-11-24 Thread Ferruh Yigit
On 11/18/2021 5:33 PM, Junxiao Shi wrote: Bugzilla ID: 888 Fixes: febc855b358e ("ethdev: forbid closing started device") Signed-off-by: Junxiao Shi Thanks Junxiao, +1 to this fix, cc'ed memif maintainer Jakub. --- drivers/net/memif/rte_eth_memif.c | 11 --- 1 file changed, 8 inse

[PATCH] net/memif: allow stopping and closing device

2021-11-18 Thread Junxiao Shi
Bugzilla ID: 888 Fixes: febc855b358e ("ethdev: forbid closing started device") Signed-off-by: Junxiao Shi --- drivers/net/memif/rte_eth_memif.c | 11 --- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/drivers/net/memif/rte_eth_memif.c b/drivers/net/memif/rte_eth_memif.c i