[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 00/16] migration: Switchover phase refactoring
From: |
Jiri Denemark |
Subject: |
Re: [PATCH 00/16] migration: Switchover phase refactoring |
Date: |
Wed, 15 Jan 2025 10:12:45 +0100 |
User-agent: |
Mutt/2.2.13 (2024-03-09) |
On Tue, Jan 14, 2025 at 18:07:30 -0500, Peter Xu wrote:
> CI: https://gitlab.com/peterx/qemu/-/pipelines/1625266692
> (note: warning is present on rust stuff, but shouldn't be relevant)
>
> This series refactors the migration switchover path quite a bit. I started
> this work initially to measure the JSON writer overhead, but then I decided
> to cleanup the switchover path in general when I am at it altogether, as I
> wanted to do this for a long time.
>
> A few major things I tried to do:
>
...
> - DEVICE migration state
>
> QEMU has a very special DEVICE migration state, that only happens with
> precopy, and only when pause-before-switchover capability is enabled.
> Due to that specialty we can't merge precopy and postcopy code on
> switchover starts, because the state machine will be different.
>
> However after I checked the history and also with libvirt developers,
> this seems unnecessary. So I had one patch making DEVICE state to be
> the "switchover" phase for precopy/postcopy unconditionally. That will
> make the state machine much easier for both modes, meanwhile nothing is
> expected to break with it (but please still shoot if anyone knows /
> suspect something will, or could, break..).
No problem from libvirt side...
Tested-by: Jiri Denemark <jdenemar@redhat.com>
- [PATCH 08/16] migration: Adjust locking in migration_maybe_pause(), (continued)
- [PATCH 08/16] migration: Adjust locking in migration_maybe_pause(), Peter Xu, 2025/01/14
- [PATCH 07/16] migration: Adjust postcopy bandwidth during switchover, Peter Xu, 2025/01/14
- [PATCH 10/16] migration: Take BQL slightly longer in postcopy_start(), Peter Xu, 2025/01/14
- [PATCH 12/16] migration: Unwrap qemu_savevm_state_complete_precopy() in postcopy, Peter Xu, 2025/01/14
- [PATCH 13/16] migration: Cleanup qemu_savevm_state_complete_precopy(), Peter Xu, 2025/01/14
- [PATCH 11/16] migration: Notify COMPLETE once for postcopy, Peter Xu, 2025/01/14
- [PATCH 14/16] migration: Always set DEVICE state, Peter Xu, 2025/01/14
- [PATCH 15/16] migration: Merge precopy/postcopy on switchover start, Peter Xu, 2025/01/14
- [PATCH 16/16] migration: Trivial cleanup on JSON writer of vmstate_save(), Peter Xu, 2025/01/14
- [PATCH 09/16] migration: Drop cached migration state in migration_maybe_pause(), Peter Xu, 2025/01/14
- Re: [PATCH 00/16] migration: Switchover phase refactoring,
Jiri Denemark <=
- Re: [PATCH 00/16] migration: Switchover phase refactoring, Juraj Marcin, 2025/01/15
- Re: [PATCH 00/16] migration: Switchover phase refactoring, Fabiano Rosas, 2025/01/15