[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH v3 00/16] chardev: refactoring & many bugfixes relat
From: |
Daniel P . Berrangé |
Subject: |
[Qemu-devel] [PATCH v3 00/16] chardev: refactoring & many bugfixes related tcp_chr_wait_connected |
Date: |
Mon, 11 Feb 2019 18:24:26 +0000 |
This is a followup to
v1: https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg03344.html
v2: http://lists.nongnu.org/archive/html/qemu-devel/2019-01/msg05947.html
This series comes out of a discussion between myself & Yongji Xie in:
https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg01881.html
I eventually understood that the problem faced was that
tcp_chr_wait_connected was racing with the background connection attempt
previously started, causing two connections to be established. This
broke because some vhost user servers only allow a single connection.
After messing around with the code alot the final solution was in fact
very easy. We simply have to delay the first background connection
attempt until the main loop is running. It will then automatically
turn into a no-op if tcp_chr_wait_connected has been run. This is
dealt with in the last patch in this series
I believe this should solve the problem Yongji Xie faced, and thus not
require us to add support for "nowait" option with client sockets at
all. The reconnect=1 option effectively already implements nowait
semantics, and now plays nicely with tcp_chr_wait_connected.
In investigating this I found various other bugs that needed fixing and
identified some useful refactoring to simplify / clarify the code, hence
this very long series.
This series passes make check-unit & check-qtest (for x86_64 target).
I did a test with tests/vhost-user-bridge too, which was in fact already
broken for the same reason Yongji Xie found - it only accepts a single
connection. This series fixes use of reconnect=1 with vhost-user-bridge.
Changed in v3:
- Fix thread between qio_task_thread_worker unlocking mutex
and qio_task_thread_result free'ing mutex (Peter)
- Ensure tcp_chr_wait_connected destroys any pending
reconnect_timer GSource
- Use g_setenv/g_unsetenv instead of setenv/unsetenv for
platform portability
Changed in v2:
- Rewrite the way we synchronize in tcp_chr_wait_connected again
to cope with chardev+device hotplug scenario
- Add extensive unit test coverage
- Resolve conflicts with git master
Daniel P. Berrangé (16):
io: store reference to thread information in the QIOTask struct
io: add qio_task_wait_thread to join with a background thread
chardev: fix validation of options for QMP created chardevs
chardev: forbid 'reconnect' option with server sockets
chardev: forbid 'wait' option with client sockets
chardev: remove many local variables in qemu_chr_parse_socket
chardev: ensure qemu_chr_parse_compat reports missing driver error
chardev: remove unused 'sioc' variable & cleanup paths
chardev: split tcp_chr_wait_connected into two methods
chardev: split up qmp_chardev_open_socket connection code
chardev: use a state machine for socket connection state
chardev: honour the reconnect setting in tcp_chr_wait_connected
chardev: disallow TLS/telnet/websocket with tcp_chr_wait_connected
chardev: fix race with client connections in tcp_chr_wait_connected
tests: expand coverage of socket chardev test
chardev: ensure termios is fully initialized
chardev/char-serial.c | 2 +-
chardev/char-socket.c | 490 ++++++++++++++++++-------
chardev/char.c | 2 +
include/io/task.h | 29 +-
io/task.c | 101 ++++--
io/trace-events | 2 +
tests/ivshmem-test.c | 2 +-
tests/libqtest.c | 4 +-
tests/test-char.c | 643 ++++++++++++++++++++++++---------
tests/test-filter-redirector.c | 4 +-
10 files changed, 939 insertions(+), 340 deletions(-)
--
2.20.1
- [Qemu-devel] [PATCH v3 00/16] chardev: refactoring & many bugfixes related tcp_chr_wait_connected,
Daniel P . Berrangé <=
- [Qemu-devel] [PATCH v3 01/16] io: store reference to thread information in the QIOTask struct, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 02/16] io: add qio_task_wait_thread to join with a background thread, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 03/16] chardev: fix validation of options for QMP created chardevs, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 04/16] chardev: forbid 'reconnect' option with server sockets, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 05/16] chardev: forbid 'wait' option with client sockets, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 06/16] chardev: remove many local variables in qemu_chr_parse_socket, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 07/16] chardev: ensure qemu_chr_parse_compat reports missing driver error, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 08/16] chardev: remove unused 'sioc' variable & cleanup paths, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 09/16] chardev: split tcp_chr_wait_connected into two methods, Daniel P . Berrangé, 2019/02/11
- [Qemu-devel] [PATCH v3 10/16] chardev: split up qmp_chardev_open_socket connection code, Daniel P . Berrangé, 2019/02/11