[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/5] vhost-user: Shutdown vhost-user connecti
From: |
Tetsuya Mukawa |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/5] vhost-user: Shutdown vhost-user connection when wrong messages are passed |
Date: |
Tue, 23 Jun 2015 17:33:57 +0900 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
On 2015/06/22 16:54, Michael S. Tsirkin wrote:
> On Mon, Jun 22, 2015 at 12:50:46PM +0900, Tetsuya Mukawa wrote:
>> When wrong vhost-user message are passed, the connection should be shutdown.
>>
>> Signed-off-by: Tetsuya Mukawa <address@hidden>
> This silently changes the protocol semantics: previously
> unknown messages were ignored. We can't do this.
> See email titled "vhost-user: protocol extensions" which relies on this
> to detect protocol version.
>
Hi Michael,
Thanks for commenting.
Instead of disconnecting the backend, I will add warning message and
leave link status down.
Regards,
Tetsuya
- [Qemu-devel] [PATCH v2 0/5] Add feature to start QEMU without vhost-user backend, Tetsuya Mukawa, 2015/06/21
- [Qemu-devel] [PATCH v2 1/5] vhost-user: Add ability to know vhost-user backend disconnection, Tetsuya Mukawa, 2015/06/21
- [Qemu-devel] [PATCH v2 3/5] vhost-user: Shutdown vhost-user connection when wrong messages are passed, Tetsuya Mukawa, 2015/06/21
- [Qemu-devel] [PATCH v2 2/5] qemu-char: Add qemu_chr_disconnect to close a fd accepted by listen fd, Tetsuya Mukawa, 2015/06/21
- [Qemu-devel] [PATCH v2 4/5] vhost-user: Enable 'nowait' and 'reconnect' option, Tetsuya Mukawa, 2015/06/21
- [Qemu-devel] [PATCH v2 5/5] vhost-user: Add new option to specify vhost-user backend supports, Tetsuya Mukawa, 2015/06/21
- Re: [Qemu-devel] [PATCH v2 0/5] Add feature to start QEMU without vhost-user backend, Michael S. Tsirkin, 2015/06/22