Re: [Qemu-devel] [PATCH v3 05/22] libqtest: Add a layer of abstraciton to send/recv

2019-09-19 Thread Oleinik, Alexander
On Thu, 2019-09-19 at 13:18 +0200, Paolo Bonzini wrote: > I think you can pass "s" to the tx handler as well, and remove the > send_opaque and recv_opaque fields? Qtest also uses this function to communicate over qmp (different fd). I can probably make the tx handler a wrapper which accepts "s",

Re: [Qemu-devel] [PATCH v3 05/22] libqtest: Add a layer of abstraciton to send/recv

2019-09-19 Thread Paolo Bonzini
On 19/09/19 01:19, Oleinik, Alexander wrote: > +static GString *qtest_client_inproc_recv_line(void *); This is not defined as of this patch, please move it to patch 14. > > +qtest_client_set_rx_handler(s, qtest_client_socket_recv_line, s); > +qtest_client_set_tx_handler(s,

Re: [Qemu-devel] [PATCH v3 05/22] libqtest: Add a layer of abstraciton to send/recv

2019-09-19 Thread Stefan Hajnoczi
On Wed, Sep 18, 2019 at 11:19:32PM +, Oleinik, Alexander wrote: > This makes it simple to swap the transport functions for qtest commands > to and from the qtest client. For example, now it is possible to > directly pass qtest commands to a server handler that exists within the > same process,