[dpdk-dev] [PATCH v2 7/7] virtio: add 1.0 support

Yuanhan Liu yuanhan.liu at linux.intel.com
Wed Jan 13 10:38:42 CET 2016


On Wed, Jan 13, 2016 at 12:31:43PM +0900, Tetsuya Mukawa wrote:
> On 2016/01/12 15:59, Yuanhan Liu wrote:
> > +static int
> > +virtio_read_caps(struct rte_pci_device *dev, struct virtio_hw *hw)
> > +{
> > +	uint8_t pos;
> > +	struct virtio_pci_cap cap;
> > +	int ret;
> > +
> > +	if (rte_eal_pci_map_device(dev) < 0) {
> > +		PMD_INIT_LOG(DEBUG, "failed to map pci device!");
> > +		return -1;
> > +	}
> > +
> 
> Do you need to call rte_eal_pci_unmap_device() in somewhere in this file?

Yes, we should. I will see where I can find a proper place for it in
next version; eth_virtio_dev_uninit sounds like a good option.

> Anyway, I've reviewed and tested your all patches.
> And it seems except for it, I guess your patches are good.

Thank you for reviewing and testing it!

	--yliu


More information about the dev mailing list