[PATCH-for-10.0,v3,0/8] hw/vfio: Build various objects once

Message

Philippe Mathieu-Daudé March 11, 2025, 8:57 a.m. UTC

Hi Cédric,

Here are the VFIO cleanup patches ready enough for 10.0,
with Richard and Eric comments from v2 addressed.

I'd prefer the rest (of v2) to wait for 10.1.

Thanks,

Phil.

Philippe Mathieu-Daudé (8):
  system: Declare qemu_[min/max]rampagesize() in 'system/hostmem.h'
  hw/vfio/spapr: Do not include <linux/kvm.h>
  hw/vfio/common: Include missing 'system/tcg.h' header
  hw/vfio/common: Get target page size using runtime helpers
  hw/vfio: Compile some common objects once
  hw/vfio: Compile more objects once
  hw/vfio: Compile iommufd.c once
  hw/vfio: Compile display.c once

 include/exec/ram_addr.h    |  3 ---
 include/system/hostmem.h   |  3 +++
 hw/ppc/spapr_caps.c        |  1 +
 hw/s390x/s390-virtio-ccw.c |  1 +
 hw/vfio/common.c           |  9 ++++++---
 hw/vfio/iommufd.c          |  1 -
 hw/vfio/migration.c        |  1 -
 hw/vfio/spapr.c            |  4 +---
 hw/vfio/meson.build        | 27 ++++++++++++++++-----------
 9 files changed, 28 insertions(+), 22 deletions(-)

Comments

Cédric Le Goater March 11, 2025, 3:48 p.m. UTC | #1

On 3/11/25 09:57, Philippe Mathieu-Daudé wrote:
> Hi Cédric,
> 
> Here are the VFIO cleanup patches ready enough for 10.0,
> with Richard and Eric comments from v2 addressed.

I dropped the previously applied patches from the v2 series and
applied this v3 in vfio-next instead.

> I'd prefer the rest (of v2) to wait for 10.1.

yes.


Thanks,

C.


  
> Thanks,
> 
> Phil.
> 
> Philippe Mathieu-Daudé (8):
>    system: Declare qemu_[min/max]rampagesize() in 'system/hostmem.h'
>    hw/vfio/spapr: Do not include <linux/kvm.h>
>    hw/vfio/common: Include missing 'system/tcg.h' header
>    hw/vfio/common: Get target page size using runtime helpers
>    hw/vfio: Compile some common objects once
>    hw/vfio: Compile more objects once
>    hw/vfio: Compile iommufd.c once
>    hw/vfio: Compile display.c once
> 
>   include/exec/ram_addr.h    |  3 ---
>   include/system/hostmem.h   |  3 +++
>   hw/ppc/spapr_caps.c        |  1 +
>   hw/s390x/s390-virtio-ccw.c |  1 +
>   hw/vfio/common.c           |  9 ++++++---
>   hw/vfio/iommufd.c          |  1 -
>   hw/vfio/migration.c        |  1 -
>   hw/vfio/spapr.c            |  4 +---
>   hw/vfio/meson.build        | 27 ++++++++++++++++-----------
>   9 files changed, 28 insertions(+), 22 deletions(-)
>