[PATCH] qemuProcessSetupRawIO: Refactor return value and remove useless #ifdef

Peter Krempa posted 1 patch 2 weeks, 2 days ago
src/qemu/qemu_process.c | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)
[PATCH] qemuProcessSetupRawIO: Refactor return value and remove useless #ifdef
Posted by Peter Krempa 2 weeks, 2 days ago
The function can return directly rather than setting 'ret' as there's no
cleanup.

It also doesn't make sense to conditionally compile out the 'break'
statement when checking whether a disk has rawio enabled if
'CAP_SYS_RAWIO' is _not_ defined as the function will still behave the
same.

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/qemu/qemu_process.c | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 242c93284e..2e4ee9e305 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -5183,7 +5183,6 @@ qemuProcessSetupRawIO(virDomainObj *vm,
 {
     bool rawio = false;
     size_t i;
-    int ret = -1;

     /* in case a certain disk is desirous of CAP_SYS_RAWIO, add this */
     for (i = 0; i < vm->def->ndisks; i++) {
@@ -5191,9 +5190,7 @@ qemuProcessSetupRawIO(virDomainObj *vm,

         if (disk->rawio == VIR_TRISTATE_BOOL_YES) {
             rawio = true;
-#ifndef CAP_SYS_RAWIO
             break;
-#endif
         }
     }

@@ -5213,18 +5210,16 @@ qemuProcessSetupRawIO(virDomainObj *vm,
         }
     }

-    ret = 0;
-
     if (rawio) {
 #ifdef CAP_SYS_RAWIO
         virCommandAllowCap(cmd, CAP_SYS_RAWIO);
 #else
         virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
                        _("Raw I/O is not supported on this platform"));
-        ret = -1;
+        return -1;
 #endif
     }
-    return ret;
+    return 0;
 }


-- 
2.46.0
Re: [PATCH] qemuProcessSetupRawIO: Refactor return value and remove useless #ifdef
Posted by Jiri Denemark 2 weeks, 2 days ago
On Tue, Sep 03, 2024 at 10:50:07 +0200, Peter Krempa wrote:
> The function can return directly rather than setting 'ret' as there's no
> cleanup.
> 
> It also doesn't make sense to conditionally compile out the 'break'
> statement when checking whether a disk has rawio enabled if
> 'CAP_SYS_RAWIO' is _not_ defined as the function will still behave the
> same.
> 
> Signed-off-by: Peter Krempa <pkrempa@redhat.com>
> ---
>  src/qemu/qemu_process.c | 9 ++-------
>  1 file changed, 2 insertions(+), 7 deletions(-)

Reviewed-by: Jiri Denemark <jdenemar@redhat.com>