[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v5 03/10] migration: avoid concurrent invoke cha
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH v5 03/10] migration: avoid concurrent invoke channel_close by different threads |
Date: |
Wed, 13 Jun 2018 18:02:59 +0100 |
User-agent: |
Mutt/1.9.5 (2018-04-13) |
On Tue, Jun 05, 2018 at 11:28:02PM +0800, Lidong Chen wrote:
> The channel_close maybe invoked by different threads. For example, source
> qemu invokes qemu_fclose in main thread, migration thread and return path
> thread. Destination qemu invokes qemu_fclose in main thread, listen thread
> and COLO incoming thread.
The only alternative approach would be to try to get all close calls
happening in the same thread, by having the other threads just set
some flag the primary thread watches. This likely quite alot of
work to achieve though, so I agree just doing mutex locking is a
prudent solution.
>
> Signed-off-by: Lidong Chen <address@hidden>
> ---
> migration/migration.c | 2 ++
> migration/migration.h | 7 +++++++
> migration/qemu-file.c | 6 ++++--
> 3 files changed, 13 insertions(+), 2 deletions(-)
Reviewed-by: Daniel P. Berrangé <address@hidden>
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
- [Qemu-devel] [PATCH v5 00/10] Enable postcopy RDMA live migration, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 01/10] migration: disable RDMA WRITE after postcopy started, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 02/10] migration: create a dedicated connection for rdma return path, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 03/10] migration: avoid concurrent invoke channel_close by different threads, Lidong Chen, 2018/06/05
- Re: [Qemu-devel] [PATCH v5 03/10] migration: avoid concurrent invoke channel_close by different threads,
Daniel P . Berrangé <=
- [Qemu-devel] [PATCH v5 05/10] migration: Stop rdma yielding during incoming postcopy, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 04/10] migration: implement bi-directional RDMA QIOChannel, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 07/10] migration: invoke qio_channel_yield only when qemu_in_coroutine(), Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 06/10] migration: implement io_set_aio_fd_handler function for RDMA QIOChannel, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 08/10] migration: create a dedicated thread to release rdma resource, Lidong Chen, 2018/06/05
- [Qemu-devel] [PATCH v5 09/10] migration: poll the cm event while wait RDMA work request completion, Lidong Chen, 2018/06/05