Re: [Qemu-devel] [PATCH v6 0/4] cryptodev: add vhost support

[Jay Zhou]

On 2018/2/2 1:06, Michael S. Tsirkin wrote:
> Yes, I plan to merge it in the next pull.
> Pls don't assume anything until it's merged upstream though,
> some issues surface late.

Okay, I see. Thanks for reviewing!

Regards,
Jay

> On Thu, Feb 01, 2018 at 11:29:15AM +0000, Zhoujian (jay) wrote:
> > Hi Michael,
> > I am wondering whether this version is OK for you?
> > Any comment will be appreciated, thanks.
> >
> > Regards,
> > Jay
> >
> > > -----Original Message-----
> > > From: Zhoujian (jay)
> > > Sent: Sunday, January 21, 2018 8:55 PM
> > > To: address@hidden
> > > Cc: address@hidden; address@hidden; Huangweidong (C)
> > > <address@hidden>; address@hidden; Zhoujian (jay)
> > > <address@hidden>; address@hidden; longpeng
> > > <address@hidden>; address@hidden; address@hidden; Gonglei
> > > (Arei) <address@hidden>; wangxin (U) <address@hidden>
> > > Subject: [PATCH v6 0/4] cryptodev: add vhost support
> > >
> > > From: Gonglei <address@hidden>
> > >
> > > I posted the RFC verion a few months ago for DPDK vhost-crypto implmention,
> > > and now it's time to send the formal version. Because we need an user space
> > > scheme for better performance.
> > >
> > > The vhost user crypto server side patches had been sent to DPDK community,
> > > pls see
> > >
> > > [RFC PATCH 0/6] lib/librte_vhost: introduce new vhost_user crypto backend
> > > support http://dpdk.org/ml/archives/dev/2017-November/081048.html
> > >
> > > You also can get virtio-crypto polling mode driver from:
> > >
> > > [PATCH] virtio: add new driver for crypto devices
> > > http://dpdk.org/ml/archives/dev/2017-November/081985.html
> > >
> > > v5 -> v6:
> > >    Fix compile error about backends/cryptodev-vhost-user.o and rebase on
> > >    the master
> > > v4 -> v5:
> > >    squash [PATCH v4 5/5] into previous patches [Michael]
> > > v3 -> v4:
> > >    "[PATCH v4 5/5] cryptodev-vhost-user: depend on CONFIG_VHOST_CRYPTO
> > >    and CONFIG_VHOST_USER" newly added to fix compilation dependency [Michael]
> > > v2 -> v3:
> > >    New added vhost user messages should be sent only when feature
> > >    has been successfully negotiated [Michael]
> > > v1 -> v2:
> > >    Fix compile error on mingw32
> > >
> > > Gonglei (4):
> > >    cryptodev: add vhost-user as a new cryptodev backend
> > >    cryptodev: add vhost support
> > >    cryptodev-vhost-user: add crypto session handler
> > >    cryptodev-vhost-user: set the key length
> > >
> > >   backends/Makefile.objs                |   6 +
> > >   backends/cryptodev-builtin.c          |   1 +
> > >   backends/cryptodev-vhost-user.c       | 379
> > > ++++++++++++++++++++++++++++++++++
> > >   backends/cryptodev-vhost.c            | 347 +++++++++++++++++++++++++++++++
> > >   configure                             |  15 ++
> > >   docs/interop/vhost-user.txt           |  26 +++
> > >   hw/virtio/Makefile.objs               |   2 +-
> > >   hw/virtio/vhost-user.c                | 104 ++++++++++
> > >   hw/virtio/virtio-crypto.c             |  70 +++++++
> > >   include/hw/virtio/vhost-backend.h     |   8 +
> > >   include/hw/virtio/virtio-crypto.h     |   1 +
> > >   include/sysemu/cryptodev-vhost-user.h |  47 +++++
> > >   include/sysemu/cryptodev-vhost.h      | 154 ++++++++++++++
> > >   include/sysemu/cryptodev.h            |   8 +
> > >   qemu-options.hx                       |  21 ++
> > >   vl.c                                  |   6 +
> > >   16 files changed, 1194 insertions(+), 1 deletion(-)  create mode 100644
> > > backends/cryptodev-vhost-user.c  create mode 100644 backends/cryptodev-
> > > vhost.c  create mode 100644 include/sysemu/cryptodev-vhost-user.h
> > >   create mode 100644 include/sysemu/cryptodev-vhost.h
> > >
> > > --
> > > 1.8.3.1
>
> .