Re: [dpdk-dev] [PATCH] net/virtio: remove useless check on mempool

2019-06-05 Thread Maxime Coquelin
On 5/9/19 10:03 AM, David Marchand wrote: This .rx_queue_setup devop is called after ethdev already dereferenced the mempool pointer. No need to check and we can remove this rte_exit. Fixes: 48cec290a3d2 ("net/virtio: move queue configure code to proper place") Cc: sta...@dpdk.org Signed-off

Re: [dpdk-dev] [PATCH] net/virtio: remove useless check on mempool

2019-05-17 Thread Maxime Coquelin
On 5/9/19 10:03 AM, David Marchand wrote: This .rx_queue_setup devop is called after ethdev already dereferenced the mempool pointer. No need to check and we can remove this rte_exit. Fixes: 48cec290a3d2 ("net/virtio: move queue configure code to proper place") Cc: sta...@dpdk.org Signed-off

Re: [dpdk-dev] [PATCH] net/virtio: remove useless check on mempool

2019-05-17 Thread David Marchand
On Thu, May 9, 2019 at 3:29 PM Jens Freimann wrote: > On Thu, May 09, 2019 at 10:03:48AM +0200, David Marchand wrote: > >This .rx_queue_setup devop is called after ethdev already dereferenced > >the mempool pointer. > >No need to check and we can remove this rte_exit. > > > >Fixes: 48cec290a3d2 (

Re: [dpdk-dev] [PATCH] net/virtio: remove useless check on mempool

2019-05-09 Thread Jens Freimann
On Thu, May 09, 2019 at 10:03:48AM +0200, David Marchand wrote: This .rx_queue_setup devop is called after ethdev already dereferenced the mempool pointer. No need to check and we can remove this rte_exit. Fixes: 48cec290a3d2 ("net/virtio: move queue configure code to proper place") Cc: sta...@d

[dpdk-dev] [PATCH] net/virtio: remove useless check on mempool

2019-05-09 Thread David Marchand
This .rx_queue_setup devop is called after ethdev already dereferenced the mempool pointer. No need to check and we can remove this rte_exit. Fixes: 48cec290a3d2 ("net/virtio: move queue configure code to proper place") Cc: sta...@dpdk.org Signed-off-by: David Marchand --- drivers/net/virtio/vi