> -----Original Message----- > From: Burakov, Anatoly > Sent: Friday, April 13, 2018 12:24 AM > To: Wang, Xiao W <[email protected]>; Yigit, Ferruh > <[email protected]> > Cc: [email protected]; [email protected]; Wang, Zhihong > <[email protected]>; Bie, Tiwei <[email protected]>; Tan, Jianfeng > <[email protected]>; Liang, Cunming <[email protected]>; Daly, > Dan <[email protected]>; [email protected]; [email protected]; > [email protected]; Chen, Junjie J <[email protected]> > Subject: Re: [PATCH v6 1/4] eal/vfio: add multiple container support > > On 12-Apr-18 5:07 PM, Wang, Xiao W wrote: > > Hi Anatoly, > > > > <...> > > >> > >> Also, why providing DMA type? There's already a VFIO type pointer in > >> vfio_config - you can set this pointer for every new created container, > >> so the user wouldn't have to care about IOMMU type. Is it not possible > >> to figure out DMA type from within EAL VFIO? If not, maybe provide an > >> API to do so, e.g. rte_vfio_container_set_dma_type()? > > > > It's possible, EAL VFIO should be able to figure out a container's DMA type. > > You probably won't be able to do it until you add a group into the > container, so probably best place to do it would be on group_bind?
Yes, the IOMMU type pointer could be set when group binding. BRs, Xiao > > -- > Thanks, > Anatoly
