Re: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-09 Thread Juan Quintela
Vladimir Sementsov-Ogievskiy wrote: > COLO is not listed as running state in migrate_is_running(), so, it's > theoretically possible to disable colo capability in COLO state and the > unexpected error in migration_iteration_finish() is reachable. > > Let's disallow that in qmp_migrate_set_capabili

Re: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-09 Thread Juan Quintela
t; Sent: Saturday, April 29, 2023 3:49 AM >> >> To: qemu-devel@nongnu.org >> >> Cc: lukasstra...@web.de; quint...@redhat.com; Zhang, Chen >> >> ; vsement...@yandex-team.ru; Peter Xu >> >> ; Leonardo Bras >> >> Subject: [PATCH v4 09/10] mi

RE: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-04 Thread Zhang, Chen
> >> Cc: lukasstra...@web.de; quint...@redhat.com; Zhang, Chen > >> ; vsement...@yandex-team.ru; Peter Xu > >> ; Leonardo Bras > >> Subject: [PATCH v4 09/10] migration: disallow change capabilities in > >> COLO state > >> > >> COLO is no

Re: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-04 Thread Vladimir Sementsov-Ogievskiy
: [PATCH v4 09/10] migration: disallow change capabilities in COLO state COLO is not listed as running state in migrate_is_running(), so, it's theoretically possible to disable colo capability in COLO state and the unexpected error in migration_iteration_finish() is reachable. Let's disall

RE: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-04 Thread Zhang, Chen
ATCH v4 09/10] migration: disallow change capabilities in COLO > state > > COLO is not listed as running state in migrate_is_running(), so, it's > theoretically possible to disable colo capability in COLO state and the > unexpected error in migration_iteration_finish

Re: [PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-05-02 Thread Peter Xu
On Fri, Apr 28, 2023 at 10:49:27PM +0300, Vladimir Sementsov-Ogievskiy wrote: > COLO is not listed as running state in migrate_is_running(), so, it's > theoretically possible to disable colo capability in COLO state and the > unexpected error in migration_iteration_finish() is reachable. > > Let's

[PATCH v4 09/10] migration: disallow change capabilities in COLO state

2023-04-28 Thread Vladimir Sementsov-Ogievskiy
COLO is not listed as running state in migrate_is_running(), so, it's theoretically possible to disable colo capability in COLO state and the unexpected error in migration_iteration_finish() is reachable. Let's disallow that in qmp_migrate_set_capabilities. Than the error becomes absolutely unreac