Re: [PATCH v5 kvmtool 00/13] Add vfio-pci support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jean-Philippe,

Jean-Philippe Brucker <jean-philippe.brucker@xxxxxxx> writes:

> Add vfio-pci support to kvmtool. Devices are assigned to the guest by
> passing "--vfio-pci [domain:]bus:dev.fn" to lkvm run, after having bound
> the device to the VFIO driver (see Documentation/vfio.txt)
>
> This version addresses Punit's comments on v4. Patch 7/13 is new; 8/13
> and 9/13 have fixes. You can find the full diff v4..v5 at the end of
> this cover letter.

The updated series looks good. For the remaining patches in the series
(7, 8, 9 and 11) -

Reviewed-by: Punit Agrawal <punit.agrawal@xxxxxxx>

Thanks,
Punit

>
> I'm testing the series on arm64 and x86, by passing an Intel x540 NIC to
> a guest on AMD Seattle, and an e1000e NIC on QEMU q35.
>
> v4: https://www.spinics.net/lists/kvm/msg159223.html
> v5: git://linux-arm.org/kvmtool-jpb.git vfio/v5
>
> Jean-Philippe Brucker (13):
>   pci: add config operations callbacks on the PCI header
>   pci: allow to specify IRQ type for PCI devices
>   irq: add irqfd helpers
>   Extend memory bank API with memory types
>   pci: add capability helpers
>   Import VFIO headers
>   Add fls_long and roundup_pow_of_two helpers
>   Add PCI device passthrough using VFIO
>   vfio-pci: add MSI-X support
>   vfio-pci: add MSI support
>   vfio: Support non-mmappable regions
>   Introduce reserved memory regions
>   vfio: check reserved regions before mapping DMA
>
>  Makefile                     |    2 +
>  arm/gic.c                    |   74 ++-
>  arm/include/arm-common/gic.h |    6 +
>  arm/kvm.c                    |    2 +-
>  arm/pci.c                    |    4 +-
>  builtin-run.c                |    5 +
>  hw/pci-shmem.c               |   12 +-
>  hw/vesa.c                    |    2 +-
>  include/kvm/irq.h            |   17 +
>  include/kvm/kvm-config.h     |    3 +
>  include/kvm/kvm.h            |   54 +-
>  include/kvm/pci.h            |  118 ++++-
>  include/kvm/util.h           |   14 +
>  include/kvm/vfio.h           |  127 +++++
>  include/linux/vfio.h         |  719 +++++++++++++++++++++++++
>  irq.c                        |   31 ++
>  kvm.c                        |   99 +++-
>  mips/kvm.c                   |    6 +-
>  pci.c                        |  105 ++--
>  powerpc/kvm.c                |    2 +-
>  vfio/core.c                  |  674 ++++++++++++++++++++++++
>  vfio/pci.c                   | 1193 ++++++++++++++++++++++++++++++++++++++++++
>  virtio/net.c                 |    9 +-
>  virtio/scsi.c                |   10 +-
>  x86/kvm.c                    |    6 +-
>  25 files changed, 3171 insertions(+), 123 deletions(-)
>  create mode 100644 include/kvm/vfio.h
>  create mode 100644 include/linux/vfio.h
>  create mode 100644 vfio/core.c
>  create mode 100644 vfio/pci.c

-- 
I don't like the Dutchman.  He's a crocodile.  He's sneaky.  I don't trust him.
		-- Jack "Legs" Diamond, just before a peace conference
		   with Dutch Schultz.

I don't trust Legs.  He's nuts.  He gets excited and starts pulling a
trigger like another guy wipes his nose.
		-- Dutch Schultz, just before a peace conference with
		   "Legs" Diamond.



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux