[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-trivial] [PATCH] qemu-char: Do not disconnect whe
From: |
Michael Tokarev |
Subject: |
Re: [Qemu-devel] [Qemu-trivial] [PATCH] qemu-char: Do not disconnect when there's data for reading |
Date: |
Wed, 24 Sep 2014 11:52:37 +0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Icedove/24.7.0 |
16.09.2014 09:31, Zifei Tong wrote:
> Friendly ping.
>
> Also CC to qemu-trivial.
There was at least 2 subsequent versions of this down the line,
a v2 and a v3. (Just to show why I haven't applied it to -trivial).
Thanks,
/mjt
> On Sun, Sep 7, 2014 at 8:24 PM, Zifei Tong <address@hidden> wrote:
>> After commit 812c1057f6175ac9a9829fa2920a2b5783814193 (Handle G_IO_HUP
>> in tcp_chr_read for tcp chardev), the connection is disconnected when in
>> G_IO_HUP condition.
>>
>> However, it's possible that the channel is in G_IO_IN condition at the
>> same time, meaning there is data for reading. In that case, the
>> remaining data is not handled.
>>
>> I saw a related bug when running socat in write-only mode, with
>>
>> $ echo "quit" | socat -u - UNIX-CONNECT:qemu-monitor
>>
>> the monitor won't not run the 'quit' command.
>>
>> CC: Kirill Batuzov <address@hidden>
>> CC: Nikolay Nikolaev <address@hidden>
>> CC: Anthony Liguori <address@hidden>
>> Signed-off-by: Zifei Tong <address@hidden>
>> ---
>> qemu-char.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/qemu-char.c b/qemu-char.c
>> index 1a8d9aa..5018c3a 100644
>> --- a/qemu-char.c
>> +++ b/qemu-char.c
>> @@ -2706,7 +2706,7 @@ static gboolean tcp_chr_read(GIOChannel *chan,
>> GIOCondition cond, void *opaque)
>> uint8_t buf[READ_BUF_LEN];
>> int len, size;
>>
>> - if (cond & G_IO_HUP) {
>> + if (!(cond & G_IO_IN) && (cond & G_IO_HUP)) {
>> /* connection closed */
>> tcp_chr_disconnect(chr);
>> return TRUE;
>> --
>> 2.1.0
>>
>