Re: [Qemu-devel] [PATCH 3/5] Add getfd and closefd monitor commands

[Jamie Lokier]

Avi Kivity wrote:
> >That misses the point though.  We process one command at a time in the 
> >monitor so we only need to buffer one fd at a time.  When we start to 
> >process multiple commands at once in the monitor, we'll do so with 
> >multiple monitor states and we'll want to have one fd per monitor state.
> 
> Again I'm thinking of the client.  If two client threads issue commands 
> in parallel there'd be >1 fds on the wire.  But qemu can consume them 
> sequentially so I agree queue may hold just a single fd.

No, even though QEMU processes commands sequentially.

If the client sends

    "command_1_with_fd" + FD1
    "command_2_with_fd" + FD2

and then waits for the results of commands 1 and 2,
QEMU calls recvmsg() and will read

    "command_1_with_fd\r\ncommand_2_with_fd\r\n" + FD1 + FD2

unless QEMU is consuming only one byte at a time from the socket, and
dispatching commands as soon as it sees the line terminator.

If QEMU reads more than one byte at a time, and the client sends more
than one command without waiting for the previous one's result, QEMU
needs to buffer more FDs otherwise some will get lost.

-- Jamie