Re: [PATCH 0/5 v2] VFIO PPC64: add VFIO support on POWERPC64

From: Alex Williamson
Date: Thu May 23 2013 - 10:56:57 EST


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

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/