[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 3/4] cryptodev-vhost-user: add crypto session
From: |
Zhang, Roy Fan |
Subject: |
Re: [Qemu-devel] [PATCH v3 3/4] cryptodev-vhost-user: add crypto session handler |
Date: |
Wed, 21 Mar 2018 03:40:15 +0000 |
Hi Jay,
Excellent work! The patch, although need some minor rework, has improved the
performance.
Some comments:
1. you need to set up capabilities for virtio_crypto PMD. As in Qemu vhost
crypto proxy backend only AESCBC and SHA1 are supported (in
cryptodev_vhost_user_init() definition), I believe in this version these two
algorithms shall be enough. Actually for the same reason I suggest you to
remove all AES_CTR test cases in the virtio_crypto PMD functional test, as they
will fail when vhost_user crypto backend is used.
You may use driver/crypto/qat/qat_crypto_capabilities.h as example. The const
capabilities array shall be returned to the application when
virtio_crypto_dev_info_get() is called.
2. there is a bug in virtio_crypto_queue_setup(), you declared " uint32_t i,
j;" while i may be used uninitialized later.
Regards,
Fan
> -----Original Message-----
> From: Jay Zhou [mailto:address@hidden
> Sent: Tuesday, January 16, 2018 2:07 PM
> To: address@hidden
> Cc: address@hidden; address@hidden; address@hidden;
> address@hidden; address@hidden;
> address@hidden; address@hidden; Zeng, Xin
> <address@hidden>; Zhang, Roy Fan <address@hidden>;
> address@hidden
> Subject: [PATCH v3 3/4] cryptodev-vhost-user: add crypto session handler
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [Qemu-devel] [PATCH v3 3/4] cryptodev-vhost-user: add crypto session handler,
Zhang, Roy Fan <=