[PATCH 00/10] PCI passthrough support for ch guests

Praveen K Paladugu posted 10 patches 1 year, 3 months ago
Failed in applying to current master (apply log)
po/POTFILES                  |   1 +
src/ch/ch_conf.h             |   4 ++
src/ch/ch_domain.c           |   2 +-
src/ch/ch_driver.c           |   4 ++
src/ch/ch_hostdev.c          | 115 +++++++++++++++++++++++++++++++++++
src/ch/ch_hostdev.h          |  32 ++++++++++
src/ch/ch_monitor.c          |   1 +
src/ch/ch_process.c          |  74 +++++++++++++++++++++-
src/ch/meson.build           |   2 +
src/hypervisor/virhostdev.c  |  23 +++++++
src/hypervisor/virhostdev.h  |   5 ++
src/libvirt_private.syms     |   2 +
src/qemu/qemu_capabilities.c |   2 +-
src/qemu/qemu_cgroup.c       |   5 +-
src/qemu/qemu_domain.c       |   2 +-
src/qemu/qemu_driver.c       |   2 +-
src/qemu/qemu_hostdev.c      |  40 +-----------
src/qemu/qemu_hostdev.h      |  10 ---
src/qemu/qemu_hotplug.c      |   5 +-
src/qemu/qemu_namespace.c    |   2 +-
tests/domaincapstest.c       |   2 +-
21 files changed, 276 insertions(+), 59 deletions(-)
create mode 100644 src/ch/ch_hostdev.c
create mode 100644 src/ch/ch_hostdev.h
[PATCH 00/10] PCI passthrough support for ch guests
Posted by Praveen K Paladugu 1 year, 3 months ago
This patch series introduces PCI passthrough support for ch guests. While
enabling this feature I refactored a bunch of methods from qemu to hypervisor
to reduce duplication of logic between the drivers.

Praveen K Paladugu (7):
  hypervisor: move HostdevNeedsVFIO to hypervisor
  hypervisor: move HostdevHostSupportsPassthroughVFIO
  qemu: replace qemuHostdevPreparePCIDevices
  ch: prepare domain definition for pci passthrough
  ch: allow hostdev in domain definitions
  ch: reattach PCI devices to host while stopping guest
  ch: explicitly set INFILESIZE to 0

Wei Liu (3):
  ch: add host device manager to driver
  ch: add scaffolding for host devices management
  ch: prepare host for PCI passthrough

 po/POTFILES                  |   1 +
 src/ch/ch_conf.h             |   4 ++
 src/ch/ch_domain.c           |   2 +-
 src/ch/ch_driver.c           |   4 ++
 src/ch/ch_hostdev.c          | 115 +++++++++++++++++++++++++++++++++++
 src/ch/ch_hostdev.h          |  32 ++++++++++
 src/ch/ch_monitor.c          |   1 +
 src/ch/ch_process.c          |  74 +++++++++++++++++++++-
 src/ch/meson.build           |   2 +
 src/hypervisor/virhostdev.c  |  23 +++++++
 src/hypervisor/virhostdev.h  |   5 ++
 src/libvirt_private.syms     |   2 +
 src/qemu/qemu_capabilities.c |   2 +-
 src/qemu/qemu_cgroup.c       |   5 +-
 src/qemu/qemu_domain.c       |   2 +-
 src/qemu/qemu_driver.c       |   2 +-
 src/qemu/qemu_hostdev.c      |  40 +-----------
 src/qemu/qemu_hostdev.h      |  10 ---
 src/qemu/qemu_hotplug.c      |   5 +-
 src/qemu/qemu_namespace.c    |   2 +-
 tests/domaincapstest.c       |   2 +-
 21 files changed, 276 insertions(+), 59 deletions(-)
 create mode 100644 src/ch/ch_hostdev.c
 create mode 100644 src/ch/ch_hostdev.h

-- 
2.44.0
Re: [PATCH 00/10] PCI passthrough support for ch guests
Posted by Praveen K Paladugu 1 year, 3 months ago
ping.. for review.

Praveen

On 10/11/2024 1:13 PM, Praveen K Paladugu wrote:
> This patch series introduces PCI passthrough support for ch guests. While
> enabling this feature I refactored a bunch of methods from qemu to hypervisor
> to reduce duplication of logic between the drivers.
> 
> Praveen K Paladugu (7):
>    hypervisor: move HostdevNeedsVFIO to hypervisor
>    hypervisor: move HostdevHostSupportsPassthroughVFIO
>    qemu: replace qemuHostdevPreparePCIDevices
>    ch: prepare domain definition for pci passthrough
>    ch: allow hostdev in domain definitions
>    ch: reattach PCI devices to host while stopping guest
>    ch: explicitly set INFILESIZE to 0
> 
> Wei Liu (3):
>    ch: add host device manager to driver
>    ch: add scaffolding for host devices management
>    ch: prepare host for PCI passthrough
> 
>   po/POTFILES                  |   1 +
>   src/ch/ch_conf.h             |   4 ++
>   src/ch/ch_domain.c           |   2 +-
>   src/ch/ch_driver.c           |   4 ++
>   src/ch/ch_hostdev.c          | 115 +++++++++++++++++++++++++++++++++++
>   src/ch/ch_hostdev.h          |  32 ++++++++++
>   src/ch/ch_monitor.c          |   1 +
>   src/ch/ch_process.c          |  74 +++++++++++++++++++++-
>   src/ch/meson.build           |   2 +
>   src/hypervisor/virhostdev.c  |  23 +++++++
>   src/hypervisor/virhostdev.h  |   5 ++
>   src/libvirt_private.syms     |   2 +
>   src/qemu/qemu_capabilities.c |   2 +-
>   src/qemu/qemu_cgroup.c       |   5 +-
>   src/qemu/qemu_domain.c       |   2 +-
>   src/qemu/qemu_driver.c       |   2 +-
>   src/qemu/qemu_hostdev.c      |  40 +-----------
>   src/qemu/qemu_hostdev.h      |  10 ---
>   src/qemu/qemu_hotplug.c      |   5 +-
>   src/qemu/qemu_namespace.c    |   2 +-
>   tests/domaincapstest.c       |   2 +-
>   21 files changed, 276 insertions(+), 59 deletions(-)
>   create mode 100644 src/ch/ch_hostdev.c
>   create mode 100644 src/ch/ch_hostdev.h
> 

-- 
Regards,
Praveen K Paladugu
Re: [PATCH 00/10] PCI passthrough support for ch guests
Posted by Praveen K Paladugu 1 year, 2 months ago
bubbling this up for reviews

On 10/22/2024 10:39 AM, Praveen K Paladugu wrote:
> ping.. for review.
> 
> Praveen
> 
> On 10/11/2024 1:13 PM, Praveen K Paladugu wrote:
>> This patch series introduces PCI passthrough support for ch guests. While
>> enabling this feature I refactored a bunch of methods from qemu to 
>> hypervisor
>> to reduce duplication of logic between the drivers.
>>
>> Praveen K Paladugu (7):
>>    hypervisor: move HostdevNeedsVFIO to hypervisor
>>    hypervisor: move HostdevHostSupportsPassthroughVFIO
>>    qemu: replace qemuHostdevPreparePCIDevices
>>    ch: prepare domain definition for pci passthrough
>>    ch: allow hostdev in domain definitions
>>    ch: reattach PCI devices to host while stopping guest
>>    ch: explicitly set INFILESIZE to 0
>>
>> Wei Liu (3):
>>    ch: add host device manager to driver
>>    ch: add scaffolding for host devices management
>>    ch: prepare host for PCI passthrough
>>
>>   po/POTFILES                  |   1 +
>>   src/ch/ch_conf.h             |   4 ++
>>   src/ch/ch_domain.c           |   2 +-
>>   src/ch/ch_driver.c           |   4 ++
>>   src/ch/ch_hostdev.c          | 115 +++++++++++++++++++++++++++++++++++
>>   src/ch/ch_hostdev.h          |  32 ++++++++++
>>   src/ch/ch_monitor.c          |   1 +
>>   src/ch/ch_process.c          |  74 +++++++++++++++++++++-
>>   src/ch/meson.build           |   2 +
>>   src/hypervisor/virhostdev.c  |  23 +++++++
>>   src/hypervisor/virhostdev.h  |   5 ++
>>   src/libvirt_private.syms     |   2 +
>>   src/qemu/qemu_capabilities.c |   2 +-
>>   src/qemu/qemu_cgroup.c       |   5 +-
>>   src/qemu/qemu_domain.c       |   2 +-
>>   src/qemu/qemu_driver.c       |   2 +-
>>   src/qemu/qemu_hostdev.c      |  40 +-----------
>>   src/qemu/qemu_hostdev.h      |  10 ---
>>   src/qemu/qemu_hotplug.c      |   5 +-
>>   src/qemu/qemu_namespace.c    |   2 +-
>>   tests/domaincapstest.c       |   2 +-
>>   21 files changed, 276 insertions(+), 59 deletions(-)
>>   create mode 100644 src/ch/ch_hostdev.c
>>   create mode 100644 src/ch/ch_hostdev.h
>>
> 

-- 
Regards,
Praveen K Paladugu
Re: [PATCH 00/10] PCI passthrough support for ch guests
Posted by Michal Prívozník 1 year, 2 months ago
On 10/11/24 20:13, Praveen K Paladugu wrote:
> This patch series introduces PCI passthrough support for ch guests. While
> enabling this feature I refactored a bunch of methods from qemu to hypervisor
> to reduce duplication of logic between the drivers.
> 
> Praveen K Paladugu (7):
>   hypervisor: move HostdevNeedsVFIO to hypervisor
>   hypervisor: move HostdevHostSupportsPassthroughVFIO
>   qemu: replace qemuHostdevPreparePCIDevices
>   ch: prepare domain definition for pci passthrough
>   ch: allow hostdev in domain definitions
>   ch: reattach PCI devices to host while stopping guest
>   ch: explicitly set INFILESIZE to 0
> 
> Wei Liu (3):
>   ch: add host device manager to driver
>   ch: add scaffolding for host devices management
>   ch: prepare host for PCI passthrough
> 
>  po/POTFILES                  |   1 +
>  src/ch/ch_conf.h             |   4 ++
>  src/ch/ch_domain.c           |   2 +-
>  src/ch/ch_driver.c           |   4 ++
>  src/ch/ch_hostdev.c          | 115 +++++++++++++++++++++++++++++++++++
>  src/ch/ch_hostdev.h          |  32 ++++++++++
>  src/ch/ch_monitor.c          |   1 +
>  src/ch/ch_process.c          |  74 +++++++++++++++++++++-
>  src/ch/meson.build           |   2 +
>  src/hypervisor/virhostdev.c  |  23 +++++++
>  src/hypervisor/virhostdev.h  |   5 ++
>  src/libvirt_private.syms     |   2 +
>  src/qemu/qemu_capabilities.c |   2 +-
>  src/qemu/qemu_cgroup.c       |   5 +-
>  src/qemu/qemu_domain.c       |   2 +-
>  src/qemu/qemu_driver.c       |   2 +-
>  src/qemu/qemu_hostdev.c      |  40 +-----------
>  src/qemu/qemu_hostdev.h      |  10 ---
>  src/qemu/qemu_hotplug.c      |   5 +-
>  src/qemu/qemu_namespace.c    |   2 +-
>  tests/domaincapstest.c       |   2 +-
>  21 files changed, 276 insertions(+), 59 deletions(-)
>  create mode 100644 src/ch/ch_hostdev.c
>  create mode 100644 src/ch/ch_hostdev.h
> 

For all patches except 03/10:

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

and merged.

Michal