[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator
From: |
Marc-André Lureau |
Subject: |
Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator |
Date: |
Tue, 26 Sep 2017 15:55:05 +0200 |
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
- [Qemu-devel] [PATCH v8 5/8] tmp backend: Add new api to read backend TpmInfo, (continued)
- [Qemu-devel] [PATCH v7 6/8] tpm-backend: Move realloc_buffer() implementation to tpm-tis model, Amarnath Valluri, 2017/09/22
- [Qemu-devel] [PATCH v7 7/8] tpm-passthrough: move reusable code to utils, Amarnath Valluri, 2017/09/22
- [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Amarnath Valluri, 2017/09/22
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Stefan Berger, 2017/09/22
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Marc-André Lureau, 2017/09/24
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Stefan Berger, 2017/09/24
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Valluri, Amarnath, 2017/09/26
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Marc-André Lureau, 2017/09/26
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Valluri, Amarnath, 2017/09/26
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator,
Marc-André Lureau <=
- Re: [Qemu-devel] [PATCH v7 8/8] tpm: Added support for TPM emulator, Eric Blake, 2017/09/26
- [Qemu-devel] [PATCH v8 8/8] tpm: Added support for TPM emulator, Amarnath Valluri, 2017/09/27
- Re: [Qemu-devel] [PATCH v8 8/8] tpm: Added support for TPM emulator, Marc-André Lureau, 2017/09/27