On Thu 07 Feb 2019 03:26:49 PM CET, Daniel P. Berrangé wrote:
>> @@ -457,11 +469,12 @@ static void tcp_chr_disconnect(Chardev *chr)
>> }
>> update_disconnected_filename(s);
>> if (emit_close) {
>> - qemu_chr_be_event(chr, CHR_EVENT_CLOSED);
>> + qemu_idle_add(tcp_chr_be_event_closed, chr, chr->gcontext);
>
> I'm a little concerned that this change might trigger some unexpected
> interactions with the qemu_chr_wait_connected() function. I added some
> more testing around that code, so if it passes the new tests I have in:
>
> https://lists.gnu.org/archive/html/qemu-devel/2019-01/msg05962.html
Forget my previous e-mail, there was a mistake on my side. The test does
pass after rebasing my series on top of that one.
> and also works with the vhostuser reconnect device, then we are
> probably ok.
How do I test this?
Berto
