diff options
| author | Zhenzhong Duan <zhenzhong.duan@intel.com> | 2024-05-07 14:42:44 +0800 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2024-05-16 16:59:20 +0200 |
| commit | b77548355ab5d8c8377ba8f981c7c597507de37a (patch) | |
| tree | e70d40f836675b943ee765c32bf6b7f41b89b029 /hw/vfio/pci.c | |
| parent | f3758413b77d396dd4db68643f7068fb49ec77d9 (diff) | |
| download | focaccia-qemu-b77548355ab5d8c8377ba8f981c7c597507de37a.tar.gz focaccia-qemu-b77548355ab5d8c8377ba8f981c7c597507de37a.zip | |
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>
Diffstat (limited to 'hw/vfio/pci.c')
| -rw-r--r-- | hw/vfio/pci.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c index 84f7bff664..c1adef5cf8 100644 --- a/hw/vfio/pci.c +++ b/hw/vfio/pci.c @@ -3027,9 +3027,8 @@ static void vfio_realize(PCIDevice *pdev, Error **errp) name = g_strdup(vbasedev->name); } - ret = vfio_attach_device(name, vbasedev, - pci_device_iommu_address_space(pdev), errp); - if (ret) { + if (!vfio_attach_device(name, vbasedev, + pci_device_iommu_address_space(pdev), errp)) { goto error; } |