[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 04/14] migration: let incoming side use thread c
From: |
Peter Xu |
Subject: |
Re: [Qemu-devel] [PATCH 04/14] migration: let incoming side use thread context |
Date: |
Thu, 1 Mar 2018 12:33:22 +0800 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Wed, Feb 28, 2018 at 09:10:58AM +0000, Daniel P. Berrangé wrote:
> On Wed, Feb 28, 2018 at 01:06:23PM +0800, Peter Xu wrote:
> > The old incoming migration is running in main thread and default
> > gcontext. With the new qio_channel_add_watch_full() we can now let it
> > run in the thread's own gcontext (if there is one).
> >
> > Currently this patch does nothing alone. But when any of the incoming
> > migration is run in another iothread (e.g., the upcoming migrate-recover
> > command), this patch will bind the incoming logic to the iothread
> > instead of the main thread (which may already get page faulted and
> > hanged).
> >
> > RDMA is not considered for now since it's not even using the QIO APIs at
> > all.
>
> Errm, yes, it is.
>
> struct QIOChannelRDMA {
> QIOChannel parent;
> RDMAContext *rdma;
> QEMUFile *file;
> size_t len;
> bool blocking; /* XXX we don't actually honour this yet */
> };
> ....
Ah, you are right. :)
I should say that "it's not using QIO watch framework" since it's
using qemu_set_fd_handler() so it's always on main thread.
Thanks,
--
Peter Xu
- Re: [Qemu-devel] [PATCH 01/14] chardev: fix leak in tcp_chr_telnet_init_io(), (continued)
[Qemu-devel] [PATCH 04/14] migration: let incoming side use thread context, Peter Xu, 2018/02/28
[Qemu-devel] [PATCH 05/14] qio: refactor net listener source operations, Peter Xu, 2018/02/28
[Qemu-devel] [PATCH 06/14] qio: store gsources for net listeners, Peter Xu, 2018/02/28
[Qemu-devel] [PATCH 07/14] qio/chardev: update net listener gcontext, Peter Xu, 2018/02/28
[Qemu-devel] [PATCH 08/14] chardev: allow telnet gsource to switch gcontext, Peter Xu, 2018/02/28
[Qemu-devel] [PATCH 09/14] qio: basic non-default context support for thread, Peter Xu, 2018/02/28