On Fri, Sep 16, 2022 at 10:20:14 +0800, dinglimin wrote: Please don't write all of your patch's description into the summary line and also make sure to follow the established formatting we have for almost all patches. Also describe when you ran into the problem if possible. > Signed-off-by: dinglimin <dinglimin@xxxxxxxxxxxxxxxxxxxx> > --- > src/qemu/qemu_process.c | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c > index cbfdd3bda5..e025e10873 100644 > --- a/src/qemu/qemu_process.c > +++ b/src/qemu/qemu_process.c > @@ -8006,9 +8006,11 @@ qemuProcessKill(virDomainObj *vm, unsigned int flags) > } > > if (flags & VIR_QEMU_PROCESS_KILL_NOWAIT) { > - virProcessKill(vm->pid, > + if (virProcessKill(vm->pid, > (flags & VIR_QEMU_PROCESS_KILL_FORCE) ? > - SIGKILL : SIGTERM); > + SIGKILL : SIGTERM) != 0) { > + VIR_DEBUG("Failed to terminate qemu process"); > + } Please make sure to follow the formatting of existing code. You've added indentation but didn't fix the rest of the function parameters and the VIR_DEBUG statement is also totally mis-indented. > return 0; > } > > -- > 2.30.0.windows.2 > > >