[libvirt] [PATCH v5 17/36] qemu_process: Cleanup qemuProcessQmpStop function
Jiri Denemark
jdenemar at redhat.com
Thu Jan 3 14:50:10 UTC 2019
On Sun, Dec 02, 2018 at 23:10:11 -0600, Chris Venteicher wrote:
> qemuProcessQmpStop is one of the 4 public functions used to create and
> manage a Qemu process for QMP command exchanges.
>
> Add comment header and debug message.
>
> Other minor code formatting cleanup.
This formatting "cleanup" is actually wrong.
> No change in functionality is intended.
And this statement can be dropped from the commit message too.
>
> Signed-off-by: Chris Venteicher <cventeic at redhat.com>
> ---
> src/qemu/qemu_process.c | 16 +++++++++++++---
> 1 file changed, 13 insertions(+), 3 deletions(-)
>
> diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
> index faf86dac5d..e9b50745d3 100644
> --- a/src/qemu/qemu_process.c
> +++ b/src/qemu/qemu_process.c
> @@ -8390,10 +8390,20 @@ qemuProcessQmpStart(qemuProcessQmpPtr proc)
> goto cleanup;
> }
>
> -
> -void
> -qemuProcessQmpStop(qemuProcessQmpPtr proc)
> +/**
> + * qemuProcessStop:
> + * @proc: Stores Process and Connection State
> + *
> + * Stop Monitor Connection and QEMU Process
> + */
OK, But I Don't Understand Why All Words Need To Be Capitalized :-)
> +void qemuProcessQmpStop(qemuProcessQmpPtr proc)
This is against our coding style.
> {
> + if (!proc)
> + return;
This change belongs to "qemu_capabilities: Stop QEMU process before
freeing".
> +
> + VIR_DEBUG("Shutting down proc=%p emulator=%s mon=%p pid=%lld",
> + proc, proc->binary, proc->mon, (long long)proc->pid);
> +
> if (proc->mon) {
> virObjectUnlock(proc->mon);
> qemuMonitorClose(proc->mon);
Jirka
More information about the libvir-list
mailing list