[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 29/47] backends/iommufd: Make iommufd_backend_*() return bool
From: |
Cédric Le Goater |
Subject: |
[PULL 29/47] backends/iommufd: Make iommufd_backend_*() return bool |
Date: |
Wed, 22 May 2024 11:54:24 +0200 |
From: Zhenzhong Duan <zhenzhong.duan@intel.com>
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>
---
include/sysemu/iommufd.h | 6 +++---
backends/iommufd.c | 29 +++++++++++++----------------
hw/vfio/iommufd.c | 5 ++---
backends/trace-events | 4 ++--
4 files changed, 20 insertions(+), 24 deletions(-)
diff --git a/include/sysemu/iommufd.h b/include/sysemu/iommufd.h
index
9af27ebd6ccb78ca8e16aa3c62629aab9f7f31e4..293bfbe967215381532b8267227dde61fa9157b7
100644
--- a/include/sysemu/iommufd.h
+++ b/include/sysemu/iommufd.h
@@ -23,11 +23,11 @@ struct IOMMUFDBackend {
/*< public >*/
};
-int iommufd_backend_connect(IOMMUFDBackend *be, Error **errp);
+bool iommufd_backend_connect(IOMMUFDBackend *be, Error **errp);
void iommufd_backend_disconnect(IOMMUFDBackend *be);
-int iommufd_backend_alloc_ioas(IOMMUFDBackend *be, uint32_t *ioas_id,
- Error **errp);
+bool iommufd_backend_alloc_ioas(IOMMUFDBackend *be, uint32_t *ioas_id,
+ Error **errp);
void iommufd_backend_free_id(IOMMUFDBackend *be, uint32_t id);
int iommufd_backend_map_dma(IOMMUFDBackend *be, uint32_t ioas_id, hwaddr iova,
ram_addr_t size, void *vaddr, bool readonly);
diff --git a/backends/iommufd.c b/backends/iommufd.c
index
76a0204852f73466a9f31492a08d89d468bff7c0..c506afbdac4beddb7dac88f74f10544f7a083e58
100644
--- a/backends/iommufd.c
+++ b/backends/iommufd.c
@@ -72,24 +72,22 @@ static void iommufd_backend_class_init(ObjectClass *oc,
void *data)
object_class_property_add_str(oc, "fd", NULL, iommufd_backend_set_fd);
}
-int iommufd_backend_connect(IOMMUFDBackend *be, Error **errp)
+bool iommufd_backend_connect(IOMMUFDBackend *be, Error **errp)
{
- int fd, ret = 0;
+ int fd;
if (be->owned && !be->users) {
fd = qemu_open_old("/dev/iommu", O_RDWR);
if (fd < 0) {
error_setg_errno(errp, errno, "/dev/iommu opening failed");
- ret = fd;
- goto out;
+ return false;
}
be->fd = fd;
}
be->users++;
-out:
- trace_iommufd_backend_connect(be->fd, be->owned,
- be->users, ret);
- return ret;
+
+ trace_iommufd_backend_connect(be->fd, be->owned, be->users);
+ return true;
}
void iommufd_backend_disconnect(IOMMUFDBackend *be)
@@ -106,25 +104,24 @@ out:
trace_iommufd_backend_disconnect(be->fd, be->users);
}
-int iommufd_backend_alloc_ioas(IOMMUFDBackend *be, uint32_t *ioas_id,
- Error **errp)
+bool iommufd_backend_alloc_ioas(IOMMUFDBackend *be, uint32_t *ioas_id,
+ Error **errp)
{
- int ret, fd = be->fd;
+ int fd = be->fd;
struct iommu_ioas_alloc alloc_data = {
.size = sizeof(alloc_data),
.flags = 0,
};
- ret = ioctl(fd, IOMMU_IOAS_ALLOC, &alloc_data);
- if (ret) {
+ if (ioctl(fd, IOMMU_IOAS_ALLOC, &alloc_data)) {
error_setg_errno(errp, errno, "Failed to allocate ioas");
- return ret;
+ return false;
}
*ioas_id = alloc_data.out_ioas_id;
- trace_iommufd_backend_alloc_ioas(fd, *ioas_id, ret);
+ trace_iommufd_backend_alloc_ioas(fd, *ioas_id);
- return ret;
+ return true;
}
void iommufd_backend_free_id(IOMMUFDBackend *be, uint32_t id)
diff --git a/hw/vfio/iommufd.c b/hw/vfio/iommufd.c
index
6a446b16dcd64838dc9ebb6150cd5ed85ec6fef3..554f9a6292454f51015ab3a56df3fab6a482ccb7
100644
--- a/hw/vfio/iommufd.c
+++ b/hw/vfio/iommufd.c
@@ -71,7 +71,7 @@ static bool iommufd_cdev_connect_and_bind(VFIODevice
*vbasedev, Error **errp)
.flags = 0,
};
- if (iommufd_backend_connect(iommufd, errp)) {
+ if (!iommufd_backend_connect(iommufd, errp)) {
return false;
}
@@ -346,8 +346,7 @@ static bool iommufd_cdev_attach(const char *name,
VFIODevice *vbasedev,
}
/* Need to allocate a new dedicated container */
- ret = iommufd_backend_alloc_ioas(vbasedev->iommufd, &ioas_id, errp);
- if (ret < 0) {
+ if (!iommufd_backend_alloc_ioas(vbasedev->iommufd, &ioas_id, errp)) {
goto err_alloc_ioas;
}
diff --git a/backends/trace-events b/backends/trace-events
index
d45c6e31a67ed66d94787f60eb08a525cf6ff68b..211e6f374adcef25be0409ce3e42cbed6f31b744
100644
--- a/backends/trace-events
+++ b/backends/trace-events
@@ -7,11 +7,11 @@ dbus_vmstate_loading(const char *id) "id: %s"
dbus_vmstate_saving(const char *id) "id: %s"
# iommufd.c
-iommufd_backend_connect(int fd, bool owned, uint32_t users, int ret) "fd=%d
owned=%d users=%d (%d)"
+iommufd_backend_connect(int fd, bool owned, uint32_t users) "fd=%d owned=%d
users=%d"
iommufd_backend_disconnect(int fd, uint32_t users) "fd=%d users=%d"
iommu_backend_set_fd(int fd) "pre-opened /dev/iommu fd=%d"
iommufd_backend_map_dma(int iommufd, uint32_t ioas, uint64_t iova, uint64_t
size, void *vaddr, bool readonly, int ret) " iommufd=%d ioas=%d
iova=0x%"PRIx64" size=0x%"PRIx64" addr=%p readonly=%d (%d)"
iommufd_backend_unmap_dma_non_exist(int iommufd, uint32_t ioas, uint64_t iova,
uint64_t size, int ret) " Unmap nonexistent mapping: iommufd=%d ioas=%d
iova=0x%"PRIx64" size=0x%"PRIx64" (%d)"
iommufd_backend_unmap_dma(int iommufd, uint32_t ioas, uint64_t iova, uint64_t
size, int ret) " iommufd=%d ioas=%d iova=0x%"PRIx64" size=0x%"PRIx64" (%d)"
-iommufd_backend_alloc_ioas(int iommufd, uint32_t ioas, int ret) " iommufd=%d
ioas=%d (%d)"
+iommufd_backend_alloc_ioas(int iommufd, uint32_t ioas) " iommufd=%d ioas=%d"
iommufd_backend_free_id(int iommufd, uint32_t id, int ret) " iommufd=%d id=%d
(%d)"
--
2.45.1
- [PULL 20/47] vfio/pci: Use g_autofree in iommufd_cdev_get_info_iova_range(), (continued)
- [PULL 20/47] vfio/pci: Use g_autofree in iommufd_cdev_get_info_iova_range(), Cédric Le Goater, 2024/05/22
- [PULL 13/47] vfio/ccw: Make vfio_ccw_register_irq_notifier() return a bool, Cédric Le Goater, 2024/05/22
- [PULL 22/47] vfio: Make VFIOIOMMUClass::setup() return bool, Cédric Le Goater, 2024/05/22
- [PULL 24/47] vfio/container: Make vfio_connect_container() return bool, Cédric Le Goater, 2024/05/22
- [PULL 23/47] vfio: Make VFIOIOMMUClass::add_window() and its wrapper return bool, Cédric Le Goater, 2024/05/22
- [PULL 30/47] vfio/display: Fix error path in call site of ramfb_setup(), Cédric Le Goater, 2024/05/22
- [PULL 25/47] vfio/container: Make vfio_set_iommu() return bool, Cédric Le Goater, 2024/05/22
- [PULL 31/47] vfio/display: Make vfio_display_*() return bool, Cédric Le Goater, 2024/05/22
- [PULL 32/47] vfio/helpers: Use g_autofree in vfio_set_irq_signaling(), Cédric Le Goater, 2024/05/22
- [PULL 28/47] vfio/cpr: Make vfio_cpr_register_container() return bool, Cédric Le Goater, 2024/05/22
- [PULL 29/47] backends/iommufd: Make iommufd_backend_*() return bool,
Cédric Le Goater <=
- [PULL 17/47] vfio/migration: Don't emit STOP_COPY VFIO migration QAPI event twice, Cédric Le Goater, 2024/05/22
- [PULL 38/47] vfio/pci: Make vfio_pci_relocate_msix() and vfio_msix_early_setup() return a bool, Cédric Le Goater, 2024/05/22
- [PULL 34/47] vfio/helpers: Make vfio_device_get_name() return bool, Cédric Le Goater, 2024/05/22
- [PULL 36/47] vfio/ccw: Make vfio_ccw_get_region() return a bool, Cédric Le Goater, 2024/05/22
- [PULL 26/47] vfio/container: Make vfio_get_device() return bool, Cédric Le Goater, 2024/05/22
- [PULL 27/47] vfio/iommufd: Make iommufd_cdev_*() return bool, Cédric Le Goater, 2024/05/22
- [PULL 33/47] vfio/helpers: Make vfio_set_irq_signaling() return bool, Cédric Le Goater, 2024/05/22
- [PULL 37/47] vfio/pci: Make vfio_intx_enable_kvm() return a bool, Cédric Le Goater, 2024/05/22
- [PULL 39/47] vfio/pci: Make vfio_populate_device() return a bool, Cédric Le Goater, 2024/05/22
- [PULL 42/47] vfio/pci: Make capability related functions return bool, Cédric Le Goater, 2024/05/22