[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 06:03:18 -0700 |
Branch: refs/heads/staging
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
- [Qemu-commits] [qemu/qemu] 836bb3: vfio: Add Error** argument to .set_dirty_page_trac...,
Richard Henderson <=