On Wed, Jan 27, 2016 at 9:26 PM, Santosh Shukla <sshukla at mvista.com> wrote: > On Wed, Jan 27, 2016 at 9:09 PM, Thomas Monjalon > <thomas.monjalon at 6wind.com> wrote: >> 2016-01-27 21:02, Santosh Shukla: >>> 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]..
