[Qemu-devel] [PATCH v2 0/9] chardev: qio related non-default context support

Peter Xu posted 9 patches 7 years, 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20180306053320.15401-1-peterx@redhat.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test docker-quick@centos6 passed
Test ppcbe passed
Test ppcle passed
Test s390x passed
chardev/char-mux.c         |  33 ++++++++--
chardev/char-socket.c      | 153 ++++++++++++++++++++++++++++++++++-----------
chardev/char.c             |  43 +++++--------
include/chardev/char-mux.h |   2 -
include/chardev/char.h     |   2 +
include/sysemu/sysemu.h    |   2 +
stubs/machine-init-done.c  |   2 +
tests/test-char.c          |   1 -
vl.c                       |   4 +-
9 files changed, 169 insertions(+), 73 deletions(-)
[Qemu-devel] [PATCH v2 0/9] chardev: qio related non-default context support
Posted by Peter Xu 7 years, 7 months ago
Based-on: <20180305064324.9238-1-peterx@redhat.com>

This series is based on the QIO part:
  [PATCH v3 0/6] qio: general non-default GMainContext support

v2:
- fix the reported problem by patchew in patch 5
- added some r-bs from Marc-Andre

Please review, thanks.

Peter Xu (9):
  vl: export machine_init_done
  chardev: fix leak in tcp_chr_telnet_init_io()
  chardev: update net listener gcontext
  chardev: allow telnet gsource to switch gcontext
  chardev: introduce chr_machine_done hook
  chardev: use chardev's gcontext for async connect
  chardev: tcp: postpone async connection setup
  chardev: tcp: let TLS run on chardev context
  chardev: tcp: postpone TLS work until machine done

 chardev/char-mux.c         |  33 ++++++++--
 chardev/char-socket.c      | 153 ++++++++++++++++++++++++++++++++++-----------
 chardev/char.c             |  43 +++++--------
 include/chardev/char-mux.h |   2 -
 include/chardev/char.h     |   2 +
 include/sysemu/sysemu.h    |   2 +
 stubs/machine-init-done.c  |   2 +
 tests/test-char.c          |   1 -
 vl.c                       |   4 +-
 9 files changed, 169 insertions(+), 73 deletions(-)

-- 
2.14.3


Re: [Qemu-devel] [PATCH v2 0/9] chardev: qio related non-default context support
Posted by Stefan Hajnoczi 7 years, 7 months ago
On Tue, Mar 06, 2018 at 01:33:11PM +0800, Peter Xu wrote:
> Based-on: <20180305064324.9238-1-peterx@redhat.com>
> 
> This series is based on the QIO part:
>   [PATCH v3 0/6] qio: general non-default GMainContext support
> 
> v2:
> - fix the reported problem by patchew in patch 5
> - added some r-bs from Marc-Andre
> 
> Please review, thanks.
> 
> Peter Xu (9):
>   vl: export machine_init_done
>   chardev: fix leak in tcp_chr_telnet_init_io()
>   chardev: update net listener gcontext
>   chardev: allow telnet gsource to switch gcontext
>   chardev: introduce chr_machine_done hook
>   chardev: use chardev's gcontext for async connect
>   chardev: tcp: postpone async connection setup
>   chardev: tcp: let TLS run on chardev context
>   chardev: tcp: postpone TLS work until machine done
> 
>  chardev/char-mux.c         |  33 ++++++++--
>  chardev/char-socket.c      | 153 ++++++++++++++++++++++++++++++++++-----------
>  chardev/char.c             |  43 +++++--------
>  include/chardev/char-mux.h |   2 -
>  include/chardev/char.h     |   2 +
>  include/sysemu/sysemu.h    |   2 +
>  stubs/machine-init-done.c  |   2 +
>  tests/test-char.c          |   1 -
>  vl.c                       |   4 +-
>  9 files changed, 169 insertions(+), 73 deletions(-)

I'm not very familiar with chardev or qio, so I defer this to Daniel
Berrange:

Acked-by: Stefan Hajnoczi <stefanha@redhat.com>