qemu-commits
[Top][All Lists]
Advanced

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

[Qemu-commits] [qemu/qemu] 836bb3: vfio: Add Error** argument to .set_di


From: Richard Henderson
Subject: [Qemu-commits] [qemu/qemu] 836bb3: vfio: Add Error** argument to .set_dirty_page_trac...
Date: Wed, 22 May 2024 15:31:48 -0700

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 836bb308685007ca7cccb891ec213a33f478af18
      
https://github.com/qemu/qemu/commit/836bb308685007ca7cccb891ec213a33f478af18
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c
    M hw/vfio/container-base.c
    M hw/vfio/container.c
    M include/hw/vfio/vfio-container-base.h

  Log Message:
  -----------
  vfio: Add Error** argument to .set_dirty_page_tracking() handler

We will use the Error object to improve error reporting in the
.log_global*() handlers of VFIO. Add documentation while at it.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 0f21358f33f0b9aa1b8dd5e33de92118186999db
      
https://github.com/qemu/qemu/commit/0f21358f33f0b9aa1b8dd5e33de92118186999db
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c

  Log Message:
  -----------
  vfio: Add Error** argument to vfio_devices_dma_logging_start()

This allows to update the Error argument of the VFIO log_global_start()
handler. Errors for container based logging will also be propagated to
qemu_savevm_state_setup() when the ram save_setup() handler is executed.
Also, errors from vfio_container_set_dirty_page_tracking() are now
collected and reported.

The vfio_set_migration_error() call becomes redundant in
vfio_listener_log_global_start(). Remove it.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 019d9e6cc408f402dd27d6884ac7b742e0e4f99e
      
https://github.com/qemu/qemu/commit/019d9e6cc408f402dd27d6884ac7b742e0e4f99e
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c
    M hw/vfio/migration.c
    M include/migration/misc.h
    M migration/migration.c

  Log Message:
  -----------
  migration: Extend migration_file_set_error() with Error* argument

Use it to update the current error of the migration stream if
available and if not, simply print out the error. Next changes will
update with an error to report.

Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Acked-by: Fabiano Rosas <farosas@suse.de>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: fbd2469a66e3fd3f3457170928daf8098f207427
      
https://github.com/qemu/qemu/commit/fbd2469a66e3fd3f3457170928daf8098f207427
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c

  Log Message:
  -----------
  vfio/migration: Add an Error** argument to vfio_migration_set_state()

Add an Error** argument to vfio_migration_set_state() and adjust
callers, including vfio_save_setup(). The error will be propagated up
to qemu_savevm_state_setup() where the save_setup() handler is
executed.

Modify vfio_vmstate_change_prepare() and vfio_vmstate_change() to
store a reported error under the migration stream if a migration is in
progress.

Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 3783f814e7e2e11cbd4e18e51e710db37900a1b8
      
https://github.com/qemu/qemu/commit/3783f814e7e2e11cbd4e18e51e710db37900a1b8
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c
    M hw/vfio/pci.c
    M include/hw/vfio/vfio-common.h

  Log Message:
  -----------
  vfio/migration: Add Error** argument to .vfio_save_config() handler

Use vmstate_save_state_with_err() to improve error reporting in the
callers and store a reported error under the migration stream. Add
documentation while at it.

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 94d12088409b9544dd4e191e2971c47ecec4c315
      
https://github.com/qemu/qemu/commit/94d12088409b9544dd4e191e2971c47ecec4c315
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c

  Log Message:
  -----------
  vfio: Reverse test on vfio_get_xlat_addr()

It will simplify the changes coming after.

Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: ebb481c03c22eaf052b02c37ddd53989a078b771
      
https://github.com/qemu/qemu/commit/ebb481c03c22eaf052b02c37ddd53989a078b771
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c
    M hw/virtio/vhost-vdpa.c
    M include/exec/memory.h
    M system/memory.c

  Log Message:
  -----------
  memory: Add Error** argument to memory_get_xlat_addr()

Let the callers do the reporting. This will be useful in
vfio_iommu_map_dirty_notify().

Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Paolo Bonzini <pbonzini@redhat.com>
Cc: David Hildenbrand <david@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 2da5f9e4d86c1a6cf5ff35cdb7b87e993e947fe0
      
https://github.com/qemu/qemu/commit/2da5f9e4d86c1a6cf5ff35cdb7b87e993e947fe0
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c
    M hw/vfio/container-base.c
    M hw/vfio/container.c
    M include/hw/vfio/vfio-common.h
    M include/hw/vfio/vfio-container-base.h

  Log Message:
  -----------
  vfio: Add Error** argument to .get_dirty_bitmap() handler

Let the callers do the error reporting. Add documentation while at it.

Reviewed-by: Eric Auger <eric.auger@redhat.com>
Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 33dc04b0722974ae140527d9c44eb442f8ac463e
      
https://github.com/qemu/qemu/commit/33dc04b0722974ae140527d9c44eb442f8ac463e
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c

  Log Message:
  -----------
  vfio: Also trace event failures in vfio_save_complete_precopy()

vfio_save_complete_precopy() currently returns before doing the trace
event. Change that.

Reviewed-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Eric Auger <eric.auger@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: ae7aca14bd531922e27767e90902b673f2cf786a
      
https://github.com/qemu/qemu/commit/ae7aca14bd531922e27767e90902b673f2cf786a
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/ap.c

  Log Message:
  -----------
  vfio/ap: Use g_autofree variable in vfio_ap_register_irq_notifier()

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Anthony Krowiak <akrowiak@linux.ibm.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: cbd470f0aac51103e010e45e1b10cb3bfedb4f99
      
https://github.com/qemu/qemu/commit/cbd470f0aac51103e010e45e1b10cb3bfedb4f99
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/ap.c

  Log Message:
  -----------
  vfio/ap: Make vfio_ap_register_irq_notifier() return a bool

Since vfio_ap_register_irq_notifier() takes and 'Error **' argument,
best practices suggest to return a bool. See the qapi/error.h Rules
section.

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 04f8e4f29b5264f855be015852497b3c1dcdb3c4
      
https://github.com/qemu/qemu/commit/04f8e4f29b5264f855be015852497b3c1dcdb3c4
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/ccw.c

  Log Message:
  -----------
  vfio/ccw: Use g_autofree variable in vfio_ccw_register_irq_notifier()

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 8aaeff97acee1ad0c96b6c229cd02c8f3e96dcda
      
https://github.com/qemu/qemu/commit/8aaeff97acee1ad0c96b6c229cd02c8f3e96dcda
  Author: Cédric Le Goater <clg@redhat.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/ccw.c

  Log Message:
  -----------
  vfio/ccw: Make vfio_ccw_register_irq_notifier() return a bool

Since vfio_ccw_register_irq_notifier() takes an 'Error **' argument,
best practices suggest to return a bool. See the qapi/error.h Rules
section.

Reviewed-by: Markus Armbruster <armbru@redhat.com>
Reviewed-by: Eric Farman <farman@linux.ibm.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 187716feeba406b5a3879db66a7bafd687472a1f
      
https://github.com/qemu/qemu/commit/187716feeba406b5a3879db66a7bafd687472a1f
  Author: Vinayak Kale <vkale@nvidia.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/core/machine.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h

  Log Message:
  -----------
  vfio/pci: migration: Skip config space check for Vendor Specific Information 
in VSC during restore/load

In case of migration, during restore operation, qemu checks config space of the
pci device with the config space in the migration stream captured during save
operation. In case of config space data mismatch, restore operation is failed.

config space check is done in function get_pci_config_device(). By default VSC
(vendor-specific-capability) in config space is checked.

Due to qemu's config space check for VSC, live migration is broken across NVIDIA
vGPU devices in situation where source and destination host driver is different.
In this situation, Vendor Specific Information in VSC varies on the destination
to ensure vGPU feature capabilities exposed to the guest driver are compatible
with destination host.

If a vfio-pci device is migration capable and vfio-pci vendor driver is OK with
volatile Vendor Specific Info in VSC then qemu should exempt config space check
for Vendor Specific Info. It is vendor driver's responsibility to ensure that
VSC is consistent across migration. Here consistency could mean that VSC format
should be same on source and destination, however actual Vendor Specific Info
may not be byte-to-byte identical.

This patch skips the check for Vendor Specific Information in VSC for VFIO-PCI
device by clearing pdev->cmask[] offsets. Config space check is still enforced
for 3 byte VSC header. If cmask[] is not set for an offset, then qemu skips
config space check for that offset.

VSC check is skipped for machine types >= 9.1. The check would be enforced on
older machine types (<= 9.0).

Cc: Alex Williamson <alex.williamson@redhat.com>
Cc: Michael S. Tsirkin <mst@redhat.com>
Cc: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Vinayak Kale <vkale@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: a0359b56ecb2002fbecd249b1ea71df618dbe4a2
      
https://github.com/qemu/qemu/commit/a0359b56ecb2002fbecd249b1ea71df618dbe4a2
  Author: Avihai Horon <avihaih@nvidia.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M MAINTAINERS
    M qapi/meson.build
    M qapi/qapi-schema.json
    A qapi/vfio.json

  Log Message:
  -----------
  qapi/vfio: Add VFIO migration QAPI event

Add a new QAPI event for VFIO migration. This event will be emitted when
a VFIO device changes its migration state, for example, during migration
or when stopping/starting the guest.

This event can be used by management applications to get updates on the
current state of the VFIO device for their own purposes.

Note that this new event is introduced since VFIO devices have a unique
set of migration states which cannot be described as accurately by other
existing events such as run state or migration status.

Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 5e1f8905ca72775452f714008c3906c9d86158ae
      
https://github.com/qemu/qemu/commit/5e1f8905ca72775452f714008c3906c9d86158ae
  Author: Avihai Horon <avihaih@nvidia.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c
    M hw/vfio/pci.c
    M include/hw/vfio/vfio-common.h

  Log Message:
  -----------
  vfio/migration: Emit VFIO migration QAPI event

Emit VFIO migration QAPI event when a VFIO device changes its migration
state. This can be used by management applications to get updates on the
current state of the VFIO device for their own purposes.

A new per VFIO device capability, "migration-events", is added so events
can be enabled only for the required devices. It is disabled by default.

Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 64366eddf15a6224263c1152f9b0a1a97965e932
      
https://github.com/qemu/qemu/commit/64366eddf15a6224263c1152f9b0a1a97965e932
  Author: Avihai Horon <avihaih@nvidia.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c

  Log Message:
  -----------
  vfio/migration: Don't emit STOP_COPY VFIO migration QAPI event twice

When migrating a VFIO device that supports pre-copy, it is transitioned
to STOP_COPY twice: once in vfio_vmstate_change() and second time in
vfio_save_complete_precopy().

The second transition is harmless, as it's a STOP_COPY->STOP_COPY no-op
transition. However, with the newly added VFIO migration QAPI event, the
STOP_COPY event is undesirably emitted twice.

Prevent this by returning early in vfio_migration_set_state() if
new_state is the same as current device state.

Note that the STOP_COPY transition in vfio_save_complete_precopy() is
essential for VFIO devices that don't support pre-copy, for migrating an
already stopped guest and for snapshots.

Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 11ebce2a5544cacab31e8ff169c6f0ceef37f4e8
      
https://github.com/qemu/qemu/commit/11ebce2a5544cacab31e8ff169c6f0ceef37f4e8
  Author: Avihai Horon <avihaih@nvidia.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/migration.c
    M hw/vfio/trace-events

  Log Message:
  -----------
  vfio/migration: Enhance VFIO migration state tracing

Move trace_vfio_migration_set_state() to the top of the function, add
recover_state to it, and add a new trace event to
vfio_migration_set_device_state().

This improves tracing of device state changes as state changes are now
also logged when vfio_migration_set_state() fails (covering recover
state and device reset transitions) and in no-op state transitions to
the same state.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Avihai Horon <avihaih@nvidia.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 81987bd58b558be71dc4d8adf163f8c787a1c3e9
      
https://github.com/qemu/qemu/commit/81987bd58b558be71dc4d8adf163f8c787a1c3e9
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Use g_autofree in vfio_realize

Local pointer name is allocated before vfio_attach_device() call
and freed after the call.

Same for tmp when calling realpath().

Use 'g_autofree' to avoid the g_free() calls.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: f3758413b77d396dd4db68643f7068fb49ec77d9
      
https://github.com/qemu/qemu/commit/f3758413b77d396dd4db68643f7068fb49ec77d9
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/iommufd.c

  Log Message:
  -----------
  vfio/pci: Use g_autofree in iommufd_cdev_get_info_iova_range()

Local pointer info is freed before return from
iommufd_cdev_get_info_iova_range().

Use 'g_autofree' to avoid the g_free() calls.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: b77548355ab5d8c8377ba8f981c7c597507de37a
      
https://github.com/qemu/qemu/commit/b77548355ab5d8c8377ba8f981c7c597507de37a
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/ap.c
    M hw/vfio/ccw.c
    M hw/vfio/common.c
    M hw/vfio/container.c
    M hw/vfio/iommufd.c
    M hw/vfio/pci.c
    M hw/vfio/platform.c
    M include/hw/vfio/vfio-common.h
    M include/hw/vfio/vfio-container-base.h

  Log Message:
  -----------
  vfio: Make VFIOIOMMUClass::attach_device() and its wrapper return bool

Make VFIOIOMMUClass::attach_device() and its wrapper function
vfio_attach_device() return bool.

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 35b25cf40e4228e922cc9830bd61a7341623023c
      
https://github.com/qemu/qemu/commit/35b25cf40e4228e922cc9830bd61a7341623023c
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/container.c
    M hw/vfio/spapr.c
    M include/hw/vfio/vfio-container-base.h

  Log Message:
  -----------
  vfio: Make VFIOIOMMUClass::setup() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 33e4c22fd1dd2aa1e1e6f45e8e89c9b961fb72d1
      
https://github.com/qemu/qemu/commit/33e4c22fd1dd2aa1e1e6f45e8e89c9b961fb72d1
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/common.c
    M hw/vfio/container-base.c
    M hw/vfio/spapr.c
    M include/hw/vfio/vfio-container-base.h

  Log Message:
  -----------
  vfio: Make VFIOIOMMUClass::add_window() and its wrapper return bool

Make VFIOIOMMUClass::add_window() and its wrapper function
vfio_container_add_section_window() return bool.

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: f6c12eaca52ca014ae1ec57ecee51c57187c2ef6
      
https://github.com/qemu/qemu/commit/f6c12eaca52ca014ae1ec57ecee51c57187c2ef6
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/container.c

  Log Message:
  -----------
  vfio/container: Make vfio_connect_container() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 534ed2e4725225e522bc5d69c219b574c9f164de
      
https://github.com/qemu/qemu/commit/534ed2e4725225e522bc5d69c219b574c9f164de
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/container.c

  Log Message:
  -----------
  vfio/container: Make vfio_set_iommu() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: be1ff306bb31fbecfe3593a2a37493407cee87d8
      
https://github.com/qemu/qemu/commit/be1ff306bb31fbecfe3593a2a37493407cee87d8
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/container.c

  Log Message:
  -----------
  vfio/container: Make vfio_get_device() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 45d0d8c4044d330225ef684894e2cdcc05ec0823
      
https://github.com/qemu/qemu/commit/45d0d8c4044d330225ef684894e2cdcc05ec0823
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/iommufd.c

  Log Message:
  -----------
  vfio/iommufd: Make iommufd_cdev_*() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

The changed functions include:

iommufd_cdev_kvm_device_add
iommufd_cdev_connect_and_bind
iommufd_cdev_attach_ioas_hwpt
iommufd_cdev_detach_ioas_hwpt
iommufd_cdev_attach_container
iommufd_cdev_get_info_iova_range

After the change, all functions in hw/vfio/iommufd.c follows the
standand.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: f38f5dd1d454e41def9cdba899d0b059dd8855d9
      
https://github.com/qemu/qemu/commit/f38f5dd1d454e41def9cdba899d0b059dd8855d9
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M hw/vfio/container.c
    M hw/vfio/cpr.c
    M hw/vfio/iommufd.c
    M include/hw/vfio/vfio-common.h

  Log Message:
  -----------
  vfio/cpr: Make vfio_cpr_register_container() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 9067d50dff29b1b96ef0d4ab7448dbd7b636e55c
      
https://github.com/qemu/qemu/commit/9067d50dff29b1b96ef0d4ab7448dbd7b636e55c
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-16 (Thu, 16 May 2024)

  Changed paths:
    M backends/iommufd.c
    M backends/trace-events
    M hw/vfio/iommufd.c
    M include/sysemu/iommufd.h

  Log Message:
  -----------
  backends/iommufd: Make iommufd_backend_*() return bool

This is to follow the coding standand to return bool if 'Error **'
is used to pass error.

The changed functions include:

iommufd_backend_connect
iommufd_backend_alloc_ioas

By this chance, simplify the functions a bit by avoiding duplicate
recordings, e.g., log through either error interface or trace, not
both.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 9442d8af674c80a2f8a7358977e1fc7ed43d2776
      
https://github.com/qemu/qemu/commit/9442d8af674c80a2f8a7358977e1fc7ed43d2776
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/display.c

  Log Message:
  -----------
  vfio/display: Fix error path in call site of ramfb_setup()

vfio_display_dmabuf_init() and vfio_display_region_init() calls
ramfb_setup() without checking its return value.

So we may run into a situation that vfio_display_probe() succeed
but errp is set. This is risky and may lead to assert failure in
error_setv().

Cc: Gerd Hoffmann <kraxel@redhat.com>
Fixes: b290659fc3d ("hw/vfio/display: add ramfb support")
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 455c009dc4ec13dab51c3764332433013b5cf3cb
      
https://github.com/qemu/qemu/commit/455c009dc4ec13dab51c3764332433013b5cf3cb
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/display.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h

  Log Message:
  -----------
  vfio/display: Make vfio_display_*() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 50b632b64ce460844089ddd5061460d8b119df5d
      
https://github.com/qemu/qemu/commit/50b632b64ce460844089ddd5061460d8b119df5d
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/helpers.c

  Log Message:
  -----------
  vfio/helpers: Use g_autofree in vfio_set_irq_signaling()

Local pointer irq_set is freed before return from
vfio_set_irq_signaling().

Use 'g_autofree' to avoid the g_free() calls.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 84e37d02969ca1c7a6a8670e7d1da8e4ca5d56b9
      
https://github.com/qemu/qemu/commit/84e37d02969ca1c7a6a8670e7d1da8e4ca5d56b9
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/ap.c
    M hw/vfio/ccw.c
    M hw/vfio/helpers.c
    M hw/vfio/pci.c
    M hw/vfio/platform.c
    M include/hw/vfio/vfio-common.h

  Log Message:
  -----------
  vfio/helpers: Make vfio_set_irq_signaling() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: c6c6cf91c0ad2a98d2566de162c3e089b75c2fc4
      
https://github.com/qemu/qemu/commit/c6c6cf91c0ad2a98d2566de162c3e089b75c2fc4
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/ap.c
    M hw/vfio/ccw.c
    M hw/vfio/helpers.c
    M hw/vfio/pci.c
    M hw/vfio/platform.c
    M include/hw/vfio/vfio-common.h

  Log Message:
  -----------
  vfio/helpers: Make vfio_device_get_name() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 958609cfeb0502d3e0a59921b5d039c5c77d57ff
      
https://github.com/qemu/qemu/commit/958609cfeb0502d3e0a59921b5d039c5c77d57ff
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/platform.c

  Log Message:
  -----------
  vfio/platform: Make vfio_populate_device() and vfio_base_device_init() return 
bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 040f8d105057383b42c162424a99a468f67a97f4
      
https://github.com/qemu/qemu/commit/040f8d105057383b42c162424a99a468f67a97f4
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/ccw.c

  Log Message:
  -----------
  vfio/ccw: Make vfio_ccw_get_region() return a bool

Since vfio_populate_device() takes an 'Error **' argument,
best practices suggest to return a bool. See the qapi/error.h
Rules section.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 44cd660a992df4bee1723fb8899c1d9bb9f8de0b
      
https://github.com/qemu/qemu/commit/44cd660a992df4bee1723fb8899c1d9bb9f8de0b
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Make vfio_intx_enable_kvm() return a bool

Since vfio_intx_enable_kvm() takes an 'Error **' argument,
best practices suggest to return a bool. See the qapi/error.h
Rules section.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 713b59a674b407b894161ada0b6541ce4e0f68d7
      
https://github.com/qemu/qemu/commit/713b59a674b407b894161ada0b6541ce4e0f68d7
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Make vfio_pci_relocate_msix() and vfio_msix_early_setup() return a 
bool

Since vfio_pci_relocate_msix() and vfio_msix_early_setup() takes
an 'Error **' argument, best practices suggest to return a bool.
See the qapi/error.h Rules section.

By this chance, pass errp directly to vfio_msix_early_setup() to avoid
calling error_propagate().

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: e942d8f08dd54c5b4ca309e8ccd22193192543a5
      
https://github.com/qemu/qemu/commit/e942d8f08dd54c5b4ca309e8ccd22193192543a5
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Make vfio_populate_device() return a bool

Since vfio_populate_device() takes an 'Error **' argument,
best practices suggest to return a bool. See the qapi/error.h
Rules section.

By this chance, pass errp directly to vfio_populate_device() to
avoid calling error_propagate().

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: c32bab074ebac4dc872dabe801efe6581f73daba
      
https://github.com/qemu/qemu/commit/c32bab074ebac4dc872dabe801efe6581f73daba
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Make vfio_intx_enable() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 64410a741d57b62f736562a89b89d0497b3033ce
      
https://github.com/qemu/qemu/commit/64410a741d57b62f736562a89b89d0497b3033ce
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/igd.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h

  Log Message:
  -----------
  vfio/pci: Make vfio_populate_vga() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: b771a40f9eecbffee70a7425e5a96feb58f85d99
      
https://github.com/qemu/qemu/commit/b771a40f9eecbffee70a7425e5a96feb58f85d99
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Make capability related functions return bool

The functions operating on capability don't have a consistent return style.

Below functions are in bool-valued functions style:
vfio_msi_setup()
vfio_msix_setup()
vfio_add_std_cap()
vfio_add_capabilities()

Below two are integer-valued functions:
vfio_add_vendor_specific_cap()
vfio_setup_pcie_cap()

But the returned integer is only used for check succeed/failure.
Change them all to return bool so now all capability related
functions follow the coding standand in qapi/error.h to return
bool.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 514855e18fcdc3378fb925046d89a0f1493336db
      
https://github.com/qemu/qemu/commit/514855e18fcdc3378fb925046d89a0f1493336db
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio/pci: Use g_autofree for vfio_region_info pointer

Pointer opregion is freed after vfio_pci_igd_opregion_init().
Use 'g_autofree' to avoid the g_free() calls.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: d3c6a18bc7affd3b89903d98d8a3125a3e1c9861
      
https://github.com/qemu/qemu/commit/d3c6a18bc7affd3b89903d98d8a3125a3e1c9861
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/igd.c
    M hw/vfio/pci-quirks.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h

  Log Message:
  -----------
  vfio/pci-quirks: Make vfio_pci_igd_opregion_init() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 0a0bda0acd529161d5cbed6c603ab89674a9af8c
      
https://github.com/qemu/qemu/commit/0a0bda0acd529161d5cbed6c603ab89674a9af8c
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/pci-quirks.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h

  Log Message:
  -----------
  vfio/pci-quirks: Make vfio_add_*_cap() return bool

This is to follow the coding standand in qapi/error.h to return bool
for bool-valued functions.

Include below functions:
vfio_add_virt_caps()
vfio_add_nv_gpudirect_cap()
vfio_add_vmd_shadow_cap()

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 0d3e89bea8ff2fb74b829d80a8e3ebe05b80614a
      
https://github.com/qemu/qemu/commit/0d3e89bea8ff2fb74b829d80a8e3ebe05b80614a
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/helpers.c
    M hw/vfio/igd.c
    M hw/vfio/pci.c

  Log Message:
  -----------
  vfio: Use g_autofree in all call site of vfio_get_region_info()

There are some exceptions when pointer to vfio_region_info is reused.
In that case, the pointed memory is freed manually.

Suggested-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: b4e1670c494165d4186930d56f692857f4fec89b
      
https://github.com/qemu/qemu/commit/b4e1670c494165d4186930d56f692857f4fec89b
  Author: Zhenzhong Duan <zhenzhong.duan@intel.com>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M hw/vfio/igd.c

  Log Message:
  -----------
  vfio/igd: Use g_autofree in vfio_probe_igd_bar4_quirk()

Pointer opregion, host and lpc are allocated and freed in
vfio_probe_igd_bar4_quirk(). Use g_autofree to automatically
free them.

Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>


  Commit: 6af8037c42fdc3d20d5aa2686799ab356a9ee1a9
      
https://github.com/qemu/qemu/commit/6af8037c42fdc3d20d5aa2686799ab356a9ee1a9
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2024-05-22 (Wed, 22 May 2024)

  Changed paths:
    M MAINTAINERS
    M backends/iommufd.c
    M backends/trace-events
    M hw/core/machine.c
    M hw/vfio/ap.c
    M hw/vfio/ccw.c
    M hw/vfio/common.c
    M hw/vfio/container-base.c
    M hw/vfio/container.c
    M hw/vfio/cpr.c
    M hw/vfio/display.c
    M hw/vfio/helpers.c
    M hw/vfio/igd.c
    M hw/vfio/iommufd.c
    M hw/vfio/migration.c
    M hw/vfio/pci-quirks.c
    M hw/vfio/pci.c
    M hw/vfio/pci.h
    M hw/vfio/platform.c
    M hw/vfio/spapr.c
    M hw/vfio/trace-events
    M hw/virtio/vhost-vdpa.c
    M include/exec/memory.h
    M include/hw/vfio/vfio-common.h
    M include/hw/vfio/vfio-container-base.h
    M include/migration/misc.h
    M include/sysemu/iommufd.h
    M migration/migration.c
    M qapi/meson.build
    M qapi/qapi-schema.json
    A qapi/vfio.json
    M system/memory.c

  Log Message:
  -----------
  Merge tag 'pull-vfio-20240522' of https://github.com/legoater/qemu into 
staging

vfio queue:

* Improvement of error reporting during migration
* Removed Vendor Specific Capability check on newer machine
* Addition of a VFIO migration QAPI event
* Changed prototype of routines using an error parameter to return bool
* Several cleanups regarding autofree variables

# -----BEGIN PGP SIGNATURE-----
#
# iQIzBAABCAAdFiEEoPZlSPBIlev+awtgUaNDx8/77KEFAmZNwDEACgkQUaNDx8/7
# 7KHaYQ/+MUFOiWEiAwJdP8I1DkY6mJV3ZDixKMHLmr8xH6fAkR2htEw6UUcYijcn
# Z0wVvcB7A1wetgIAB2EPc2o6JtRD1uEW2pPq3SVpdWO2rWYa4QLvldOiJ8A+Kvss
# 0ZugWirgZsM7+ka9TCuysmqWdQD+P6z2RURMSwiPi6QPHwv1Tt69gLSxFeV5WWai
# +mS6wUbaU3LSt6yRhORRvFkCss4je3D3YR73ivholGHANxi/7C5T22KwOHrW6Qzf
# uk3W/zq1yL1YLXSu6WoKPw0mMCvNtGyKK2oAlhG3Ln1tPYnctNrlfXlApqxEOGl3
# adGtwd6fyg6UTRR+vOXEy1QPCGcHtKWc5SuV5E677JftARJMwzbXrJw9Y9xS2RCQ
# oRYS5814k9RdubTxu+/l8NLICMdox7dNy//QLyrIdD7nJKYhFODkV1giWh4NWkt6
# m0T3PGLlUJ/V2ngWQu9Aw150m3lCPEKt+Nv/mGOEFDRu9dv55Vb7oJwr1dBB/n+e
# 1lNNpDmV0YipoKYMzrlBwNwxhXGJOtNPwHtw/vZuiy70CXUwo0t4XLMpWbWasxZc
# 0yz4O9RLRJEhPtPqv54aLsE2kNY10I8vwHBlhyNgIEsA7eCDduA+65aPBaqIF7z6
# GjvYdixF+vAZFexn0mDi1gtM3Yh60Hiiq1j7kKyyti/q0WUQzIc=
# =awMc
# -----END PGP SIGNATURE-----
# gpg: Signature made Wed 22 May 2024 02:51:45 AM PDT
# gpg:                using RSA key A0F66548F04895EBFE6B0B6051A343C7CFFBECA1
# gpg: Good signature from "Cédric Le Goater <clg@kaod.org>" [undefined]
# gpg: WARNING: This key is not certified with a trusted signature!
# gpg:          There is no indication that the signature belongs to the owner.
# Primary key fingerprint: A0F6 6548 F048 95EB FE6B  0B60 51A3 43C7 CFFB ECA1

* tag 'pull-vfio-20240522' of https://github.com/legoater/qemu: (47 commits)
  vfio/igd: Use g_autofree in vfio_probe_igd_bar4_quirk()
  vfio: Use g_autofree in all call site of vfio_get_region_info()
  vfio/pci-quirks: Make vfio_add_*_cap() return bool
  vfio/pci-quirks: Make vfio_pci_igd_opregion_init() return bool
  vfio/pci: Use g_autofree for vfio_region_info pointer
  vfio/pci: Make capability related functions return bool
  vfio/pci: Make vfio_populate_vga() return bool
  vfio/pci: Make vfio_intx_enable() return bool
  vfio/pci: Make vfio_populate_device() return a bool
  vfio/pci: Make vfio_pci_relocate_msix() and vfio_msix_early_setup() return a 
bool
  vfio/pci: Make vfio_intx_enable_kvm() return a bool
  vfio/ccw: Make vfio_ccw_get_region() return a bool
  vfio/platform: Make vfio_populate_device() and vfio_base_device_init() return 
bool
  vfio/helpers: Make vfio_device_get_name() return bool
  vfio/helpers: Make vfio_set_irq_signaling() return bool
  vfio/helpers: Use g_autofree in vfio_set_irq_signaling()
  vfio/display: Make vfio_display_*() return bool
  vfio/display: Fix error path in call site of ramfb_setup()
  backends/iommufd: Make iommufd_backend_*() return bool
  vfio/cpr: Make vfio_cpr_register_container() return bool
  ...

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Compare: https://github.com/qemu/qemu/compare/01782d6b294f...6af8037c42fd

To unsubscribe from these emails, change your notification settings at 
https://github.com/qemu/qemu/settings/notifications



reply via email to

[Prev in Thread] Current Thread [Next in Thread]