[dpdk-dev] [PATCH] virtio: check if devargs is NULL before checking its value

2016-06-01 Thread Xie, Huawei
On 5/27/2016 10:08 AM, Yuanhan Liu wrote: > On Wed, May 25, 2016 at 12:47:30PM +0200, Thomas Monjalon wrote: >>> - dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI) { >>> + (!dev->devargs || >>> +dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI)) { >> S

[dpdk-dev] [PATCH] virtio: check if devargs is NULL before checking its value

2016-05-27 Thread Yuanhan Liu
On Wed, May 25, 2016 at 12:47:30PM +0200, Thomas Monjalon wrote: > > - dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI) { > > + (!dev->devargs || > > +dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI)) { > > Should the title be something like "fix cra

[dpdk-dev] [PATCH] virtio: check if devargs is NULL before checking its value

2016-05-25 Thread Thomas Monjalon
> - dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI) { > + (!dev->devargs || > + dev->devargs->type != RTE_DEVTYPE_WHITELISTED_PCI)) { Should the title be something like "fix crash ..."? I would also add Reported-by: Vincent Li

[dpdk-dev] [PATCH] virtio: check if devargs is NULL before checking its value

2016-05-24 Thread Huawei Xie
Fixes: ac5e1d838dc1 ("virtio: skip error when probing kernel managed device") Signed-off-by: Huawei Xie --- drivers/net/virtio/virtio_pci.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/net/virtio/virtio_pci.c b/drivers/net/virtio/virtio_pci.c index 9cdca06..0e76