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:[email protected]] > Sent: Tuesday, January 16, 2018 2:07 PM > To: [email protected] > Cc: [email protected]; [email protected]; [email protected]; > [email protected]; [email protected]; > [email protected]; [email protected]; Zeng, Xin > <[email protected]>; Zhang, Roy Fan <[email protected]>; > [email protected] > Subject: [PATCH v3 3/4] cryptodev-vhost-user: add crypto session handler >
