Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-30 Thread Thomas Monjalon
28/04/2017 15:24, Burakov, Anatoly: > > From: Andrew Rybchenko [mailto:arybche...@solarflare.com] > > Sent: Tuesday, April 4, 2017 5:06 PM > > To: dev@dpdk.org > > Cc: Burakov, Anatoly ; Alejandro Lucero > > > > Subject: [PATCH] vfio: set IOMMU type for the container once > > > > If more than one

Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-28 Thread Burakov, Anatoly
> From: Andrew Rybchenko [mailto:arybche...@solarflare.com] > Sent: Tuesday, April 4, 2017 5:06 PM > To: dev@dpdk.org > Cc: Burakov, Anatoly ; Alejandro Lucero > > Subject: [PATCH] vfio: set IOMMU type for the container once > > If more than one used PCI device belongs to one IOMMU group, it is s

Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-26 Thread gowrishankar muthukrishnan
Thanks Alajandro. Could this patch be merged before you make further cases considered ?. Reason is that, we need this fix in 17.05 for ppc64le as I pointed earlier (and of course any changes that would make the fix better, if possible before RCs closed). Thanks, Gowrishankar On Friday 21 April

Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-26 Thread Alejandro Lucero
Hi Gowrishankar, The VFIO maintainer is going to accept your patch. I have just sent a patch about the unplug issue: http://www.dpdk.org/ml/archives/dev/2017-April/064737.html On Wed, Apr 26, 2017 at 11:36 AM, gowrishankar muthukrishnan < gowrishanka...@linux.vnet.ibm.com> wrote: > Thanks Al

Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-21 Thread Alejandro Lucero
The patch is OK for me. I have run tests with that change with no problems. But I can just run tests with devices where each one got its own IOMMU group, so my ack is just for that case (although theoretically this patch fixes the other case). I'm looking at doing some changes to the kernel for bei

Re: [dpdk-dev] [PATCH] vfio: set IOMMU type for the container once

2017-04-21 Thread gowrishankar muthukrishnan
Could this patch be reviewed and merged for 17.05 rc3 ?. It solved regression with i40e pmd bring up in one of our ppc64le server models through vfio-pci pci module. Thanks Alexey pointing out this in one of your patches. Regards, Gowrishankar On Tuesday 04 April 2017 09:36 PM, Andrew Rybchenk