[dpdk-dev] [PATCH 2/3] vhost: check that a device exists during reset_owner

2015-10-19 Thread Yuanhan Liu
On Fri, Oct 16, 2015 at 11:08:49AM +0200, Jerome Jutteau wrote: > virtio-net search for it's device in reset_owner. > The function don't check the return result of get_config_ll_entry > which can be NULL. > > Signed-off-by: Jerome Jutteau > --- > lib/librte_vhost/virtio-net.c | 2 ++ > 1 file ch

[dpdk-dev] [PATCH 2/3] vhost: check that a device exists during reset_owner

2015-10-19 Thread Jérôme Jutteau
Hi Yuanhan, 2015-10-19 6:35 GMT+02:00 Yuanhan Liu : > I would call get_device() here, which dumps an error message when > it's NULL so that user will get noticed. > > Besides that, the 3 patches look good to me, and, > > Acked-by: Yuanhan Liu Thanks for your review, a v2 is coming soon with get_

[dpdk-dev] [PATCH 2/3] vhost: check that a device exists during reset_owner

2015-10-16 Thread Jerome Jutteau
virtio-net search for it's device in reset_owner. The function don't check the return result of get_config_ll_entry which can be NULL. Signed-off-by: Jerome Jutteau --- lib/librte_vhost/virtio-net.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/lib/librte_vhost/virtio-net.c b/lib/librte_