> -----Original Message----- > From: Burakov, Anatoly > Sent: Friday, April 13, 2018 12:24 AM > To: Wang, Xiao W <xiao.w.w...@intel.com>; Yigit, Ferruh > <ferruh.yi...@intel.com> > Cc: email@example.com; maxime.coque...@redhat.com; Wang, Zhihong > <zhihong.w...@intel.com>; Bie, Tiwei <tiwei....@intel.com>; Tan, Jianfeng > <jianfeng....@intel.com>; Liang, Cunming <cunming.li...@intel.com>; Daly, > Dan <dan.d...@intel.com>; tho...@monjalon.net; gaetan.ri...@6wind.com; > hemant.agra...@nxp.com; Chen, Junjie J <junjie.j.c...@intel.com> > 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