[PATCH v2 0/4] Add a new backend for cryptodev

Lei He posted 4 patches 3 years, 4 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20221008085030.70212-1-helei.sig11@bytedance.com
Maintainers: "Gonglei (Arei)" <arei.gonglei@huawei.com>, "Daniel P. Berrangé" <berrange@redhat.com>, "Michael S. Tsirkin" <mst@redhat.com>, Eric Blake <eblake@redhat.com>, Markus Armbruster <armbru@redhat.com>, Paolo Bonzini <pbonzini@redhat.com>, Eduardo Habkost <eduardo@habkost.net>
There is a newer version of this series
backends/cryptodev-builtin.c    |  69 +++--
backends/cryptodev-lkcf.c       | 645 ++++++++++++++++++++++++++++++++++++++++
backends/cryptodev-vhost-user.c |  51 +++-
backends/cryptodev.c            |  44 +--
backends/meson.build            |   3 +
crypto/akcipher.c               |  18 ++
crypto/der.c                    | 307 +++++++++++++++++--
crypto/der.h                    | 211 ++++++++++++-
crypto/rsakey.c                 |  42 +++
crypto/rsakey.h                 |  11 +-
hw/virtio/virtio-crypto.c       | 324 +++++++++++---------
include/crypto/akcipher.h       |  21 ++
include/sysemu/cryptodev.h      |  61 ++--
qapi/qom.json                   |   2 +
tests/unit/test-crypto-der.c    | 126 ++++++--
15 files changed, 1675 insertions(+), 260 deletions(-)
create mode 100644 backends/cryptodev-lkcf.c
[PATCH v2 0/4] Add a new backend for cryptodev
Posted by Lei He 3 years, 4 months ago
v1 --> v2:
- Fix compile errors when neither 'nettle' nor 'gcrypt' are enabled.
- Trivial changes to error codes when neither 'nettle' nor 'gcrypt' are
enabled.

This patch adds a new backend called LKCF to cryptodev, LKCF stands
for Linux Kernel Cryptography Framework. If a cryptographic
accelerator that supports LKCF is installed on the the host (you can
see which algorithms are supported in host's LKCF by executing
'cat /proc/crypto'), then RSA operations can be offloaded.
More background info can refer to: https://lwn.net/Articles/895399/,
'keyctl[5]' in the picture.

This patch:
1. Modified some interfaces of cryptodev and cryptodev-backend to
support asynchronous requests.
2. Extended the DER encoder in crypto, so that we can export the
RSA private key into PKCS#8 format and upload it to host kernel.
3. Added a new backend for cryptodev.

I tested the backend with a QAT card, the qps of RSA-2048-decryption
is about 25k/s, and the main-loop becomes the bottleneck. The qps
using OpenSSL directly is about 6k/s (with 6 vCPUs). We will support 
IO-thread for cryptodev in another series later.


Lei He (4):
  virtio-crypto: Support asynchronous mode
  crypto: Support DER encodings
  crypto: Support export akcipher to pkcs8
  cryptodev: Add a lkcf-backend for cryptodev

 backends/cryptodev-builtin.c    |  69 +++--
 backends/cryptodev-lkcf.c       | 645 ++++++++++++++++++++++++++++++++++++++++
 backends/cryptodev-vhost-user.c |  51 +++-
 backends/cryptodev.c            |  44 +--
 backends/meson.build            |   3 +
 crypto/akcipher.c               |  18 ++
 crypto/der.c                    | 307 +++++++++++++++++--
 crypto/der.h                    | 211 ++++++++++++-
 crypto/rsakey.c                 |  42 +++
 crypto/rsakey.h                 |  11 +-
 hw/virtio/virtio-crypto.c       | 324 +++++++++++---------
 include/crypto/akcipher.h       |  21 ++
 include/sysemu/cryptodev.h      |  61 ++--
 qapi/qom.json                   |   2 +
 tests/unit/test-crypto-der.c    | 126 ++++++--
 15 files changed, 1675 insertions(+), 260 deletions(-)
 create mode 100644 backends/cryptodev-lkcf.c

-- 
2.11.0
Re: [PATCH v2 0/4] Add a new backend for cryptodev
Posted by Lei He 3 years, 3 months ago
On 2022/10/8 16:50, Lei He wrote:
> v1 --> v2:
> - Fix compile errors when neither 'nettle' nor 'gcrypt' are enabled.
> - Trivial changes to error codes when neither 'nettle' nor 'gcrypt' are
> enabled.
> 

Hi, lei:
   Daniel has reviewed the crypto part of this patch(thanks a lot for 
Daniel), can you help to have a glance at rest codes? Thanks.

Best regards,
Lei He
--
helei.sig11@bytedance.com