diff options
author | 2023-02-10 03:36:28 -0300 | |
---|---|---|
committer | 2023-02-13 03:44:44 +0100 | |
commit | e5bac1f525472d6042a4cdba31dda5825cde0086 (patch) | |
tree | d120569aceec723b6b3117e8ffd9508bfdf6e4a2 /scripts/coverage/compare_gcov_json.py | |
parent | migration: Postpone postcopy preempt channel to be after main (diff) | |
download | qemu-e5bac1f525472d6042a4cdba31dda5825cde0086.tar.xz qemu-e5bac1f525472d6042a4cdba31dda5825cde0086.zip |
migration/multifd: Change multifd_load_cleanup() signature and usage
Since it's introduction in commit f986c3d256 ("migration: Create multifd
migration threads"), multifd_load_cleanup() never returned any value
different than 0, neither set up any error on errp.
Even though, on process_incoming_migration_bh() an if clause uses it's
return value to decide on setting autostart = false, which will never
happen.
In order to simplify the codebase, change multifd_load_cleanup() signature
to 'void multifd_load_cleanup(void)', and for every usage remove error
handling or decision made based on return value != 0.
Fixes: b5eea99ec2 ("migration: Add yank feature")
Reported-by: Li Xiaohui <xiaohli@redhat.com>
Signed-off-by: Leonardo Bras <leobras@redhat.com>
Reviewed-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Peter Xu <peterx@redhat.com>
Signed-off-by: Juan Quintela <quintela@redhat.com>
Diffstat (limited to 'scripts/coverage/compare_gcov_json.py')
0 files changed, 0 insertions, 0 deletions