[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_pos
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy' |
Date: |
Wed, 25 Sep 2019 11:37:21 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
* Dr. David Alan Gilbert (git) (address@hidden) wrote:
> From: "Dr. David Alan Gilbert" <address@hidden>
>
> Various parts of the migration code do different things when they're
> in postcopy mode; prior to this patch this has been 'postcopy-active'.
> This patch extends 'in_postcopy' to include 'postcopy-paused' and
> 'postcopy-recover'.
>
> In particular, when you set the max-postcopy-bandwidth parameter, this
> only affects the current migration fd if we're 'in_postcopy';
> this leads to a race in the postcopy recovery test where it increases
> the speed from 4k/sec to unlimited, but that increase can get ignored
> if the change is made between the point at which the reconnection
> happens and it transitions back to active.
>
> Signed-off-by: Dr. David Alan Gilbert <address@hidden>
Queued
> ---
> migration/migration.c | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/migration/migration.c b/migration/migration.c
> index 01863a95f5..5f7e4d15e9 100644
> --- a/migration/migration.c
> +++ b/migration/migration.c
> @@ -1659,7 +1659,14 @@ bool migration_in_postcopy(void)
> {
> MigrationState *s = migrate_get_current();
>
> - return (s->state == MIGRATION_STATUS_POSTCOPY_ACTIVE);
> + switch (s->state) {
> + case MIGRATION_STATUS_POSTCOPY_ACTIVE:
> + case MIGRATION_STATUS_POSTCOPY_PAUSED:
> + case MIGRATION_STATUS_POSTCOPY_RECOVER:
> + return true;
> + default:
> + return false;
> + }
> }
>
> bool migration_in_postcopy_after_devices(MigrationState *s)
> --
> 2.21.0
>
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Dr. David Alan Gilbert (git), 2019/09/23
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Alex Bennée, 2019/09/23
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Peter Xu, 2019/09/23
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Juan Quintela, 2019/09/24
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Alex Bennée, 2019/09/24
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy', Markus Armbruster, 2019/09/25
- Re: [PATCH] migration/postcopy: Recognise the recovery states as 'in_postcopy',
Dr. David Alan Gilbert <=