[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 4/9] chardev: allow telnet gsource to switch
From: |
Daniel P . Berrangé |
Subject: |
Re: [Qemu-devel] [PATCH v2 4/9] chardev: allow telnet gsource to switch gcontext |
Date: |
Wed, 7 Mar 2018 12:28:26 +0000 |
User-agent: |
Mutt/1.9.2 (2017-12-15) |
On Tue, Mar 06, 2018 at 01:33:15PM +0800, Peter Xu wrote:
> It was originally created by qio_channel_add_watch() so it's always
> assigning the task to main context. Now we use the new API called
> qio_channel_add_watch_source() so that we get the GSource handle rather
> than the tag ID.
>
> Meanwhile, caching the gsource and TCPChardevTelnetInit (which holds the
> handshake data) in SocketChardev.telnet_source so that we can also do
> dynamic context switch when update read handlers.
>
> Signed-off-by: Peter Xu <address@hidden>
> ---
> chardev/char-socket.c | 67
> +++++++++++++++++++++++++++++++++++++++------------
> 1 file changed, 51 insertions(+), 16 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 v2 0/9] chardev: qio related non-default context support, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 2/9] chardev: fix leak in tcp_chr_telnet_init_io(), Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 1/9] vl: export machine_init_done, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 3/9] chardev: update net listener gcontext, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 4/9] chardev: allow telnet gsource to switch gcontext, Peter Xu, 2018/03/06
- Re: [Qemu-devel] [PATCH v2 4/9] chardev: allow telnet gsource to switch gcontext,
Daniel P . Berrangé <=
- [Qemu-devel] [PATCH v2 5/9] chardev: introduce chr_machine_done hook, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 6/9] chardev: use chardev's gcontext for async connect, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 7/9] chardev: tcp: postpone async connection setup, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 8/9] chardev: tcp: let TLS run on chardev context, Peter Xu, 2018/03/06
- [Qemu-devel] [PATCH v2 9/9] chardev: tcp: postpone TLS work until machine done, Peter Xu, 2018/03/06