[PATCH 02/13] qemu: Remove unused 'qemuProcessRebootAllowed'

Peter Krempa via Devel posted 13 patches 1 month, 3 weeks ago
[PATCH 02/13] qemu: Remove unused 'qemuProcessRebootAllowed'
Posted by Peter Krempa via Devel 1 month, 3 weeks ago
From: Peter Krempa <pkrempa@redhat.com>

Signed-off-by: Peter Krempa <pkrempa@redhat.com>
---
 src/qemu/qemu_process.c | 18 ------------------
 src/qemu/qemu_process.h |  2 --
 2 files changed, 20 deletions(-)

diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 646ddda7e9..a1cdce6d17 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu/qemu_process.c
@@ -6880,24 +6880,6 @@ qemuProcessPrepareDomainHostdevs(virDomainObj *vm,
 }


-/**
- * qemuProcessRebootAllowed:
- * @def: domain definition
- *
- * This function encapsulates the logic which dictated whether '-no-reboot' was
- * used instead of '-no-shutdown' which is used  QEMU versions which don't
- * support the 'set-action' QMP command.
- */
-bool
-qemuProcessRebootAllowed(const virDomainDef *def)
-{
-    return def->onReboot != VIR_DOMAIN_LIFECYCLE_ACTION_DESTROY ||
-           def->onPoweroff != VIR_DOMAIN_LIFECYCLE_ACTION_DESTROY ||
-           (def->onCrash != VIR_DOMAIN_LIFECYCLE_ACTION_DESTROY &&
-            def->onCrash != VIR_DOMAIN_LIFECYCLE_ACTION_COREDUMP_DESTROY);
-}
-
-
 static int
 qemuProcessUpdateSEVInfo(virDomainObj *vm)
 {
diff --git a/src/qemu/qemu_process.h b/src/qemu/qemu_process.h
index 5874214596..2db5186e08 100644
--- a/src/qemu/qemu_process.h
+++ b/src/qemu/qemu_process.h
@@ -258,8 +258,6 @@ G_DEFINE_AUTOPTR_CLEANUP_FUNC(qemuProcessQMP, qemuProcessQMPFree);

 int qemuProcessQMPStart(qemuProcessQMP *proc);

-bool qemuProcessRebootAllowed(const virDomainDef *def);
-
 void qemuProcessCleanupMigrationJob(virQEMUDriver *driver,
                                     virDomainObj *vm);

-- 
2.55.0
Re: [PATCH 02/13] qemu: Remove unused 'qemuProcessRebootAllowed'
Posted by Ján Tomko via Devel 1 month, 3 weeks ago
On a Wednesday in 2026, Peter Krempa via Devel wrote:
>From: Peter Krempa <pkrempa@redhat.com>
>
>Signed-off-by: Peter Krempa <pkrempa@redhat.com>
>---
> src/qemu/qemu_process.c | 18 ------------------
> src/qemu/qemu_process.h |  2 --
> 2 files changed, 20 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano