[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse
From: |
Peter Maydell |
Subject: |
Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind |
Date: |
Mon, 30 Jun 2014 13:33:27 +0100 |
On 30 June 2014 13:29, Paolo Bonzini <address@hidden> wrote:
> Il 30/06/2014 12:57, Gerd Hoffmann ha scritto:
>> On Mo, 2014-06-30 at 11:33 +0100, Peter Maydell wrote:
>>> In that case is qemu_chr_open_socket_fd() incorrect
>>> in marking the socket as nonblocking in the
>>> is_listen && is_waitconnect case?
>
>
> It's unnecessary. tcp_chr_accept calls tcp_chr_add_client, which takes care
> of that. But it doesn't hurt either.
I think the tcp_chr_accept->tcp_chr_add_client->set_nonblock
is marking the new fd returned from accept() as nonblocking.
The call in qemu_chr_open_socket_fd() is marking the listening
fd as nonblocking. So those are different things...
thanks
-- PMM
- [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Peter Maydell, 2014/06/20
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Markus Armbruster, 2014/06/23
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Peter Maydell, 2014/06/23
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Gerd Hoffmann, 2014/06/30
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Peter Maydell, 2014/06/30
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Gerd Hoffmann, 2014/06/30
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Peter Maydell, 2014/06/30
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Paolo Bonzini, 2014/06/30
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind,
Peter Maydell <=
- Re: [Qemu-devel] [PATCH] qemu-char: Convert socket char backend to parse/kind, Paolo Bonzini, 2014/06/30