diff options
| author | Eric Blake <eblake@redhat.com> | 2023-05-02 15:52:12 -0500 |
|---|---|---|
| committer | Kevin Wolf <kwolf@redhat.com> | 2023-05-10 14:16:53 +0200 |
| commit | 6dab4c93ecfae48e2e67b984d1032c1e988d3005 (patch) | |
| tree | 1da12c5c2b737e2a722adbd597659ca3e035c24c /scripts/qapi/parser.py | |
| parent | cf6052f111f8d6a2ef083f3e2848bc6f8180bb96 (diff) | |
| download | focaccia-qemu-6dab4c93ecfae48e2e67b984d1032c1e988d3005.tar.gz focaccia-qemu-6dab4c93ecfae48e2e67b984d1032c1e988d3005.zip | |
migration: Attempt disk reactivation in more failure scenarios
Commit fe904ea824 added a fail_inactivate label, which tries to reactivate disks on the source after a failure while s->state == MIGRATION_STATUS_ACTIVE, but didn't actually use the label if qemu_savevm_state_complete_precopy() failed. This failure to reactivate is also present in commit 6039dd5b1c (also covering the new s->state == MIGRATION_STATUS_DEVICE state) and 403d18ae (ensuring s->block_inactive is set more reliably). Consolidate the two labels back into one - no matter HOW migration is failed, if there is any chance we can reach vm_start() after having attempted inactivation, it is essential that we have tried to restart disks before then. This also makes the cleanup more like migrate_fd_cancel(). Suggested-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Eric Blake <eblake@redhat.com> Message-Id: <20230502205212.134680-1-eblake@redhat.com> Acked-by: Peter Xu <peterx@redhat.com> Reviewed-by: Juan Quintela <quintela@redhat.com> Reviewed-by: Kevin Wolf <kwolf@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'scripts/qapi/parser.py')
0 files changed, 0 insertions, 0 deletions