[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH RFC v7 5/9] migration: fix the multifd code when
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH RFC v7 5/9] migration: fix the multifd code when sending less channels |
Date: |
Fri, 2 Nov 2018 10:37:38 +0800 |
User-agent: |
Mutt/1.10.1 (2018-07-13) |
On Thu, Nov 01, 2018 at 06:17:11PM +0800, Fei Li wrote:
> Set the migration state to "failed" instead of "setup" when failing
> to send packet via some channel.
Could you please provide more information in the commit message?
E.g., what will happen if without this patch? Will it crash the
source or stall the source migration or others? Otherwise it's a bit
hard for me to understand what's this patch for.
Normally I would prefer to not touch global states in feature specific
code path, but I'd like to know the problem more first...
Thanks,
>
> Cc: Peter Xu <address@hidden>
> Signed-off-by: Fei Li <address@hidden>
> ---
> migration/ram.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/migration/ram.c b/migration/ram.c
> index 4db3b3e8f4..c84d164fc8 100644
> --- a/migration/ram.c
> +++ b/migration/ram.c
> @@ -1072,6 +1072,7 @@ out:
> static void multifd_new_send_channel_async(QIOTask *task, gpointer opaque)
> {
> MultiFDSendParams *p = opaque;
> + MigrationState *s = migrate_get_current();
> QIOChannel *sioc = QIO_CHANNEL(qio_task_get_source(task));
> Error *local_err = NULL;
>
> @@ -1083,6 +1084,7 @@ static void multifd_new_send_channel_async(QIOTask
> *task, gpointer opaque)
> if (multifd_save_cleanup(&local_err) != 0) {
> migrate_set_error(migrate_get_current(), local_err);
> }
> + migrate_set_state(&s->state, s->state, MIGRATION_STATUS_FAILED);
> } else {
> p->c = QIO_CHANNEL(sioc);
> qio_channel_set_delay(p->c, false);
> --
> 2.13.7
>
Regards,
--
Peter Xu
- [Qemu-devel] [PATCH RFC v7 3/9] qemu_thread_join: fix segmentation fault, (continued)
- [Qemu-devel] [PATCH RFC v7 3/9] qemu_thread_join: fix segmentation fault, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 2/9] qemu_init_vcpu: add a new Error parameter to propagate, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 1/9] Fix segmentation fault when qemu_signal_init fails, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 4/9] migration: fix some segmentation faults when using multifd, Fei Li, 2018/11/01
- [Qemu-devel] [PATCH RFC v7 5/9] migration: fix the multifd code when sending less channels, Fei Li, 2018/11/01
[Qemu-devel] [PATCH RFC v7 7/9] migration: remove unused &local_err parameter in migrate_set_error, Fei Li, 2018/11/01
[Qemu-devel] [PATCH RFC v7 6/9] migration: fix the multifd code when receiving less channels, Fei Li, 2018/11/01