[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session ha
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler |
Date: |
Tue, 28 Nov 2017 12:19:35 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 |
On 28/11/2017 12:06, Gonglei (Arei) wrote:
>>> You mean we can share control virtqueue to DPDK as well? Like data queues?
>> I don't know :) but why not?
>>
> Current there are two main reasons for this design:
>
> 1) we should use another cpu to polling the control virtqueue, which is
> expensive.
IIRC DPDK also supports interrupt mode, doesn't it? Is it possible to
do interrupt mode for some virtqueues and poll mode for others?
> 2) we should copy the logic of parsing control message to DPDK, which break
> current layered architecture .
But isn't it already a layering violation that you're adding *some*
control messages to the vhost-user protocol? I am not sure why only
these two are necessary.
Paolo
> I'm not sure if there are any other hidden issues for future scalability,
> such as
> using Qemu to manage some control messages, avoiding D-Dos attack etc.
>
> Thanks,
> -Gonglei
- [Qemu-devel] [PATCH 0/4] cryptodev: add vhost support, Gonglei, 2017/11/28
- [Qemu-devel] [PATCH 4/4] cryptodev-vhost-user: set the key length, Gonglei, 2017/11/28
- [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Gonglei, 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Paolo Bonzini, 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Gonglei (Arei), 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Paolo Bonzini, 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Gonglei (Arei), 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Gonglei (Arei), 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Michael S. Tsirkin, 2017/11/28
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Paolo Bonzini, 2017/11/29
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Tan, Jianfeng, 2017/11/29
- Re: [Qemu-devel] [PATCH 3/4] cryptodev-vhost-user: add crypto session handler, Michael S. Tsirkin, 2017/11/28
[Qemu-devel] [PATCH 2/4] cryptodev: add vhost support, Gonglei, 2017/11/28
[Qemu-devel] [PATCH 1/4] cryptodev: add vhost-user as a new cryptodev backend, Gonglei, 2017/11/28
Re: [Qemu-devel] [PATCH 0/4] cryptodev: add vhost support, no-reply, 2017/11/28