Re: [dpdk-dev] [PATCH v5 2/2] virtio: fix PCI config err handling

2018-08-28 Thread Luca Boccassi
On Tue, 2018-08-28 at 14:43 +0800, Tiwei Bie wrote: > I just noticed the title. It should be "net/virtio: xxx", > instead of "virtio: xxx". Fixed > On Mon, Aug 27, 2018 at 05:52:40PM +0100, Luca Boccassi wrote: > [...] > > + ret = rte_pci_read_config(dev, &flags, > > sizeof(flag

Re: [dpdk-dev] [PATCH v5 2/2] virtio: fix PCI config err handling

2018-08-27 Thread Tiwei Bie
I just noticed the title. It should be "net/virtio: xxx", instead of "virtio: xxx". On Mon, Aug 27, 2018 at 05:52:40PM +0100, Luca Boccassi wrote: [...] > + ret = rte_pci_read_config(dev, &flags, sizeof(flags), > + pos + sizeof(cap)); > +

[dpdk-dev] [PATCH v5 2/2] virtio: fix PCI config err handling

2018-08-27 Thread Luca Boccassi
From: Brian Russell In virtio_read_caps and vtpci_msix_detect, rte_pci_read_config returns the number of bytes read from PCI config or < 0 on error. If less than the expected number of bytes are read then log the failure and return rather than carrying on with garbage. Fixes: 6ba1f63b5ab0 ("virt