Re: [PATCH v6 08/11] eal: pci: introduce RTE_KDRV_VFIO_NOIOMMUi driver mode

On Wed, Jan 27, 2016 at 9:26 PM, Santosh Shukla [off-list ref] wrote:

On Wed, Jan 27, 2016 at 9:09 PM, Thomas Monjalon
[off-list ref] wrote:

quoted

2016-01-27 21:02, Santosh Shukla:

quoted

1. virtio currently works for vfio+noiommu and likely will work for
vfio+iommu in near future.
2. So remove __noiommu suffix and always use default.
3. Introduce vfio resource parsing global function, That function
suppose to do parsing for default vfio case and for vfio-noiommu case.
This function will be used by pmd drivers for resource parsing purpose
example virtio.

Yuan won't be happy with 3) I guess, because he wanted to get rid of
interface parsing from pmd driver.

Thomas, if 1/2/3/ addresses your concern then I'll spin the series,

I agree with 1/ and 2/.
Please, could you explain why 3/ is needed?

Because someone should do resource parsing / validation before driver
does resource mapping/initialization. That someone could be either EAL
layer or driver itself.

In my case;
- driver is virtio
- resource is vfio interface

FWIW, removed 3) / Removed this patch entirely from this series,
Sending v6 version for effected patch [09/11]..