diff options
author | Cédric Le Goater <clg@redhat.com> | 2024-02-26 17:31:22 -0300 |
---|---|---|
committer | Peter Xu <peterx@redhat.com> | 2024-02-28 11:31:28 +0800 |
commit | 9425ef3f990a42b98329d5059362f40714e70442 (patch) | |
tree | 78c4fc7d79eb07e5b2e436677c89d051cfce8407 /scripts/qapi/parser.py | |
parent | 22b04245f0d5237b0e4f7b10fa05577eff6522ea (diff) | |
download | qemu-9425ef3f990a42b98329d5059362f40714e70442.zip qemu-9425ef3f990a42b98329d5059362f40714e70442.tar.gz qemu-9425ef3f990a42b98329d5059362f40714e70442.tar.bz2 |
migration: Use migrate_has_error() in close_return_path_on_source()
close_return_path_on_source() retrieves the migration error from the
the QEMUFile '->to_dst_file' to know if a shutdown is required. This
shutdown is required to exit the return-path thread.
Avoid relying on '->to_dst_file' and use migrate_has_error() instead.
(using to_dst_file is a heuristic to infer whether
rp_state.from_dst_file might be stuck on a recvmsg(). Using a generic
method for detecting errors is more reliable. We also want to reduce
dependency on QEMUFile::last_error)
Suggested-by: Peter Xu <peterx@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
[added some words about the motivation for this patch]
Signed-off-by: Fabiano Rosas <farosas@suse.de>
Link: https://lore.kernel.org/r/20240226203122.22894-3-farosas@suse.de
Signed-off-by: Peter Xu <peterx@redhat.com>
Diffstat (limited to 'scripts/qapi/parser.py')
0 files changed, 0 insertions, 0 deletions