From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by ozlabs.org (Postfix) with ESMTP id B79102C016A for ; Fri, 24 May 2013 00:56:32 +1000 (EST) Message-ID: <1369320984.2646.135.camel@ul30vt.home> Subject: Re: [PATCH 0/5 v2] VFIO PPC64: add VFIO support on POWERPC64 From: Alex Williamson To: Alexey Kardashevskiy Date: Thu, 23 May 2013 08:56:24 -0600 In-Reply-To: <1369107191-28547-1-git-send-email-aik@ozlabs.ru> References: <1369107191-28547-1-git-send-email-aik@ozlabs.ru> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: kvm@vger.kernel.org, linux-kernel@vger.kernel.org, Paul Mackerras , linuxppc-dev@lists.ozlabs.org, David Gibson List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2013-05-21 at 13:33 +1000, Alexey Kardashevskiy wrote: > The series adds support for VFIO on POWERPC in user space (such as QEMU). > The in-kernel real mode IOMMU support is added by another series posted > separately. > > As the first and main aim of this series is the POWERNV platform support, > the "Enable on POWERNV platform" patch goes first and introduces an API > to be used by the VFIO IOMMU driver. The "Enable on pSeries platform" patch > simply registers PHBs in the IOMMU subsystem and expects the API to be present, > it enables VFIO support in fully emulated QEMU guests. > > The main change is that this series was changed and tested against v3.10-rc1. > It also contains some bugfixes which are mentioned (if any) in the patch messages. > > Alexey Kardashevskiy (3): > powerpc/vfio: Enable on POWERNV platform > powerpc/vfio: Implement IOMMU driver for VFIO > powerpc/vfio: Enable on pSeries platform > > Documentation/vfio.txt | 63 +++++ > arch/powerpc/include/asm/iommu.h | 26 ++ > arch/powerpc/kernel/iommu.c | 323 +++++++++++++++++++++++ > arch/powerpc/platforms/powernv/pci-ioda.c | 1 + > arch/powerpc/platforms/powernv/pci-p5ioc2.c | 5 +- > arch/powerpc/platforms/powernv/pci.c | 2 + > arch/powerpc/platforms/pseries/iommu.c | 4 + > drivers/iommu/Kconfig | 8 + > drivers/vfio/Kconfig | 6 + > drivers/vfio/Makefile | 1 + > drivers/vfio/vfio.c | 1 + > drivers/vfio/vfio_iommu_spapr_tce.c | 377 +++++++++++++++++++++++++++ > include/uapi/linux/vfio.h | 34 +++ > 13 files changed, 850 insertions(+), 1 deletion(-) > create mode 100644 drivers/vfio/vfio_iommu_spapr_tce.c > These look ok to me, how do you want to integrate them? Should I provide Acks on patches 2 & 3 and let them get pushed through the ppc tree or should I wait for patch 1 then push 2 & 3 through my tree? Thanks, Alex