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

Jay Zhou posted 4 patches 7 years, 7 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/cover.1519712347.git.arei.gonglei@huawei.com
Test checkpatch passed
Test docker-build@min-glib passed
Test docker-mingw@fedora passed
Test docker-quick@centos6 passed
Test ppcbe passed
Test ppcle passed
Test s390x passed
backends/Makefile.objs                |   6 +
backends/cryptodev-builtin.c          |   1 +
backends/cryptodev-vhost-user.c       | 377 ++++++++++++++++++++++++++++++++++
backends/cryptodev-vhost.c            | 347 +++++++++++++++++++++++++++++++
configure                             |  15 ++
docs/interop/vhost-user.txt           |  26 +++
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 +
15 files changed, 1191 insertions(+)
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
[Qemu-devel] [PATCH v8 0/4] cryptodev: add vhost support
Posted by Jay Zhou 7 years, 7 months ago
From: Gonglei <arei.gonglei@huawei.com>

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 v2 0/7] crypto: add virtio poll mode driver
http://dpdk.org/ml/archives/dev/2018-February/091410.html

v8:
  - make cryptodev-vhost.c complied linux independently
v7:
  - make virtio crypto enabled on non-Linux
  - fix format-string issues
  - fix error reported by clang
  - fix a typo when setting length of cipher key
  - rebased on the master
v6:
  - Fix compile error about backends/cryptodev-vhost-user.o and rebase on
    the master
v5:
  - squash [PATCH v4 5/5] into previous patches [Michael]
v4:
  - "[PATCH v4 5/5] cryptodev-vhost-user: depend on CONFIG_VHOST_CRYPTO
    and CONFIG_VHOST_USER" newly added to fix compilation dependency [Michael]
v3:
  - New added vhost user messages should be sent only when feature
    has been successfully negotiated [Michael]
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       | 377 ++++++++++++++++++++++++++++++++++
 backends/cryptodev-vhost.c            | 347 +++++++++++++++++++++++++++++++
 configure                             |  15 ++
 docs/interop/vhost-user.txt           |  26 +++
 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 +
 15 files changed, 1191 insertions(+)
 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



Re: [Qemu-devel] [PATCH v8 0/4] cryptodev: add vhost support
Posted by Zhoujian (jay) 7 years, 7 months ago
The Intel guy Fan Zhang(CCed) reported a config issue when enabling and testing
vhost_crypto, so I resend this version, thanks!

Regards,
Jay

> -----Original Message-----
> From: Zhoujian (jay)
> Sent: Tuesday, February 27, 2018 2:33 PM
> To: qemu-devel@nongnu.org
> Cc: mst@redhat.com; pbonzini@redhat.com; Huangweidong (C)
> <weidong.huang@huawei.com>; stefanha@redhat.com; Zhoujian (jay)
> <jianjay.zhou@huawei.com>; pasic@linux.vnet.ibm.com; longpeng
> <longpeng2@huawei.com>; xin.zeng@intel.com; roy.fan.zhang@intel.com; Gonglei
> (Arei) <arei.gonglei@huawei.com>; wangxin (U) <wangxinxin.wang@huawei.com>
> Subject: [PATCH v8 0/4] cryptodev: add vhost support
> 
> From: Gonglei <arei.gonglei@huawei.com>
> 
> 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 v2 0/7] crypto: add virtio poll mode driver
> http://dpdk.org/ml/archives/dev/2018-February/091410.html
> 
> v8:
>   - make cryptodev-vhost.c complied linux independently
> v7:
>   - make virtio crypto enabled on non-Linux
>   - fix format-string issues
>   - fix error reported by clang
>   - fix a typo when setting length of cipher key
>   - rebased on the master
> v6:
>   - Fix compile error about backends/cryptodev-vhost-user.o and rebase on
>     the master
> v5:
>   - squash [PATCH v4 5/5] into previous patches [Michael]
> v4:
>   - "[PATCH v4 5/5] cryptodev-vhost-user: depend on CONFIG_VHOST_CRYPTO
>     and CONFIG_VHOST_USER" newly added to fix compilation dependency [Michael]
> v3:
>   - New added vhost user messages should be sent only when feature
>     has been successfully negotiated [Michael]
> 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       | 377
> ++++++++++++++++++++++++++++++++++
>  backends/cryptodev-vhost.c            | 347 +++++++++++++++++++++++++++++++
>  configure                             |  15 ++
>  docs/interop/vhost-user.txt           |  26 +++
>  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 +
>  15 files changed, 1191 insertions(+)
>  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
>