On Tue, Dec 19, 2017 at 04:45:31PM +0800, Peter Xu wrote:
> In commit 6bbb6c0644 ("chardev: use per-dev context for
> io_add_watch_poll", 2017-09-22) all the chardev watches are converted to
> use per-chardev gcontext to support chardev to be run outside default
> main thread. However that's still missing one call from the frontend
> code. Touch that up.
>
> Signed-off-by: Peter Xu <pet...@redhat.com>
> ---
> chardev/char-fe.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/chardev/char-fe.c b/chardev/char-fe.c
> index ee6d596100..462c529f19 100644
> --- a/chardev/char-fe.c
> +++ b/chardev/char-fe.c
> @@ -356,7 +356,7 @@ guint qemu_chr_fe_add_watch(CharBackend *be, GIOCondition
> cond,
> }
>
> g_source_set_callback(src, (GSourceFunc)func, user_data, NULL);
> - tag = g_source_attach(src, NULL);
> + tag = g_source_attach(src, be->chr->gcontext);
If you respin, it's a little nicer to use the s local variable instead:
tag = g_source_attach(src, s->gcontext);
signature.asc
Description: PGP signature
