[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 1/9] yank: Unregister function when using TLS migration
From: |
Leonardo Brás |
Subject: |
Re: [PULL 1/9] yank: Unregister function when using TLS migration |
Date: |
Tue, 15 Jun 2021 18:17:26 -0300 |
User-agent: |
Evolution 3.40.2 |
On Mon, 2021-06-14 at 07:14 +0200, Philippe Mathieu-Daudé wrote:
> Hi Leonardo,
>
> On 6/9/21 4:45 PM, Dr. David Alan Gilbert (git) wrote:
> > From: Leonardo Bras <leobras.c@gmail.com>
> >
> > After yank feature was introduced in migration, whenever migration
> > is started using TLS, the following error happens in both source
> > and
> > destination hosts:
> >
> > (qemu) qemu-kvm: ../util/yank.c:107: yank_unregister_instance:
> > Assertion `QLIST_EMPTY(&entry->yankfns)' failed.
> >
> > This happens because of a missing yank_unregister_function() when
> > using
> > qio-channel-tls.
> >
> > Fix this by also allowing TYPE_QIO_CHANNEL_TLS object type to
> > perform
> > yank_unregister_function() in channel_close() and
> > multifd_load_cleanup().
> >
> > Also, inside migration_channel_connect() and
> > migration_channel_process_incoming() move yank_register_function()
> > so
> > it only runs once on a TLS migration.
> >
> > Fixes: b5eea99ec2f ("migration: Add yank feature", 2021-01-13)
> > Buglink: https://bugzilla.redhat.com/show_bug.cgi?id=1964326
> > Signed-off-by: Leonardo Bras <leobras.c@gmail.com>
> > Reviewed-by: Lukas Straub <lukasstraub2@web.de>
> > Reviewed-by: Peter Xu <peterx@redhat.com>
> >
> > --
>
> Please use the '---' separator, otherwise tools don't strip
> out your comments and they end in the repository, adding
> confusion.
>
Sure, sorry about that.
> > Changes since v2:
> > - Dropped all references to ioc->master
> > - yank_register_function() and yank_unregister_function() now only
> > run
> > once in a TLS migration.
> >
> > Changes since v1:
> > - Cast p->c to QIOChannelTLS into multifd_load_cleanup()
> > Message-Id: <20210601054030.1153249-1-leobras.c@gmail.com>
> >
> > Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
> > ---
> > migration/channel.c | 26 ++++++++++++++------------
> > migration/multifd.c | 3 ++-
> > migration/qemu-file-channel.c | 4 +++-
> > 3 files changed, 19 insertions(+), 14 deletions(-)
- [PULL 0/9] migration queue, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 1/9] yank: Unregister function when using TLS migration, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 2/9] migration/rdma: Fix cm event use after free, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 3/9] channel-socket: Only set CLOEXEC if we have space for fds, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 4/9] io/net-listener: Call the notifier during finalize, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 6/9] migration/socket: Close the listener at the end, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 7/9] sockets: Support multipath TCP, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 5/9] migration: Add cleanup hook for inwards migration, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 9/9] hmp: Add "calc_dirty_rate" and "info dirty_rate" cmds, Dr. David Alan Gilbert (git), 2021/06/09
- [PULL 8/9] migration/dirtyrate: make sample page count configurable, Dr. David Alan Gilbert (git), 2021/06/09
- Re: [PULL 0/9] migration queue, Peter Maydell, 2021/06/09
- Re: [PULL 0/9] migration queue, no-reply, 2021/06/14