Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator

[Marc-André Lureau]

Hi

On Tue, Sep 26, 2017 at 3:28 PM, Valluri, Amarnath
<address@hidden> wrote:
> On Tue, 2017-09-26 at 14:24 +0200, Marc-André Lureau wrote:
>> > > close fds[1] ?
>> > I guess we are not supposed to close the other end of the
>> > socketpair/pipe, as it is not forked process.
>> You will close this end, not the other end.
> fds[1] is handed over to other end, fds[0] is used by qemu, so we
> cannot close unless error case i guess.

You should be able to close fds[1] in qemu, and keep the communication
working between the two processes. If you don't close, it will leak a
fd in qemu.

-- 
Marc-André Lureau