Re: [Qemu-devel] [PATCH v2 0/3] add exit-script option to qemu

[Dominik Csapak]

On 10/8/18 5:28 AM, Jason Wang wrote:
> On 2018年10月04日 19:43, Dominik Csapak wrote:
> > this patch series aims to execute a script when qemu exits
> > so that one can do cleanups when using --daemonize without
> > having to use the qmp monitor
>
> Hi:
>
> Can you give a example of why it must be done through this way? It looks 
> to me that we can do this by monitor the pid and behave accordingly 
> through bash.
>
> Thanks

Hi,

as i said in a reply on my original patch, the reason was twofold

first, i thought having an additional process just for monitoring
the exit of qemu with qmp/pidfile was too much for some users, as 
opposed to a simple script that executes then

ans second (more important), i wanted to know the reason qemu
exited (e.g. user initiated shutdown/reset/etc.),
especially when '-no-reboot' is set

please ignore this series (maybe the refactoring of the launch script
is interesting, i don't know), i will send a patch to be able
to distinguish the shutdown/reset with no-reboot in a seperate
patch for qmp

> > changes since v1:
> >
> > * refactored as qemu_launch_script, only for non-windows platforms
> > * updated net/tap.c to use qemu_launch_script instead of launch_script
> > * fixed a small error in the option description
> >
> > Dominik Csapak (3):
> >    osdep: add qemu_launch_script for executing scripts
> >    tap: use qemu_launch_script instead of launch_script
> >    vl.c: call optional script when exiting
> >
> >   include/qemu/osdep.h | 12 +++++++++++
> >   net/tap.c            | 56 
> > ++++++++++------------------------------------------
> >   qemu-options.hx      | 20 +++++++++++++++++++
> >   util/oslib-posix.c   | 34 +++++++++++++++++++++++++++++++
> >   util/oslib-win32.c   |  8 ++++++++
> >   vl.c                 | 29 +++++++++++++++++++++++++++
> >   6 files changed, 113 insertions(+), 46 deletions(-)