[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 5/7] python/machine: handle "fast" QEMU terminations
From: |
Willian Rampazzo |
Subject: |
Re: [PATCH 5/7] python/machine: handle "fast" QEMU terminations |
Date: |
Fri, 19 Nov 2021 13:09:30 -0300 |
On Thu, Nov 18, 2021 at 5:49 PM John Snow <jsnow@redhat.com> wrote:
>
> In the case that the QEMU process actually launches -- but then dies so
> quickly that we can't establish a QMP connection to it -- QEMUMachine
> currently calls _post_shutdown() assuming that it never launched the VM
> process.
>
> This isn't true, though: it "merely" may have failed to establish a QMP
> connection and the process is in the middle of its own exit path.
>
> If we don't wait for the subprocess, the caller may get a bogus `None`
> return for .exitcode(). This behavior was observed from
> device-crash-test; after the switch to Async QMP, the timings were
> changed such that it was now seemingly possible to witness the failure
> of "vm.launch()" *prior* to the exitcode becoming available.
>
> The semantic of the `_launched` property is changed in this
> patch. Instead of representing the condition "launch() executed
> successfully", it will now represent "has forked a child process
> successfully". This way, wait() when called in the exit path won't
> become a no-op.
> Signed-off-by: John Snow <jsnow@redhat.com>
Nitpick: double signed-off-by
>
> Signed-off-by: John Snow <jsnow@redhat.com>
> ---
> python/qemu/machine/machine.py | 19 ++++++++++++-------
> 1 file changed, 12 insertions(+), 7 deletions(-)
>
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
- [PATCH 0/7] python: More fixes for 6.2, John Snow, 2021/11/18
- [PATCH 1/7] python/machine: add @sock_dir property, John Snow, 2021/11/18
- [PATCH 5/7] python/machine: handle "fast" QEMU terminations, John Snow, 2021/11/18
- Re: [PATCH 5/7] python/machine: handle "fast" QEMU terminations,
Willian Rampazzo <=
- [PATCH 2/7] python/machine: remove _remove_monitor_sockfile property, John Snow, 2021/11/18
- [PATCH 4/7] python/machine: move more variable initializations to _pre_launch, John Snow, 2021/11/18
- [PATCH 6/7] scripts/device-crash-test: Use a QMP timeout, John Snow, 2021/11/18
- [PATCH 3/7] python/machine: add instance disambiguator to default nickname, John Snow, 2021/11/18
- [PATCH 7/7] python/aqmp: fix send_fd_scm for python 3.6.x, John Snow, 2021/11/18