Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/41118/?format=api
{ "id": 41118, "url": "http://patches.dpdk.org/api/patches/41118/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20180614110257.10967-1-roy.fan.zhang@intel.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20180614110257.10967-1-roy.fan.zhang@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20180614110257.10967-1-roy.fan.zhang@intel.com", "date": "2018-06-14T11:02:57", "name": "crypto/virtio: fix iv physical address", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "91a05032d17822e3054c225068bf58b53016716c", "submitter": { "id": 304, "url": "http://patches.dpdk.org/api/people/304/?format=api", "name": "Fan Zhang", "email": "roy.fan.zhang@intel.com" }, "delegate": { "id": 22, "url": "http://patches.dpdk.org/api/users/22/?format=api", "username": "pdelarag", "first_name": "Pablo", "last_name": "de Lara Guarch", "email": "pablo.de.lara.guarch@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20180614110257.10967-1-roy.fan.zhang@intel.com/mbox/", "series": [ { "id": 124, "url": "http://patches.dpdk.org/api/series/124/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=124", "date": "2018-06-14T11:02:57", "name": "crypto/virtio: fix iv physical address", "version": 1, "mbox": "http://patches.dpdk.org/series/124/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/41118/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/41118/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [127.0.0.1])\n\tby dpdk.org (Postfix) with ESMTP id 6C9751E34F;\n\tThu, 14 Jun 2018 13:12:21 +0200 (CEST)", "from mga05.intel.com (mga05.intel.com [192.55.52.43])\n\tby dpdk.org (Postfix) with ESMTP id 12F5D1E345;\n\tThu, 14 Jun 2018 13:12:19 +0200 (CEST)", "from orsmga003.jf.intel.com ([10.7.209.27])\n\tby fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t14 Jun 2018 04:12:18 -0700", "from silpixa00398673.ir.intel.com (HELO\n\tsilpixa00398673.ger.corp.intel.com) ([10.237.223.54])\n\tby orsmga003.jf.intel.com with ESMTP; 14 Jun 2018 04:12:16 -0700" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.51,222,1526367600\"; d=\"scan'208\";a=\"59181549\"", "From": "Fan Zhang <roy.fan.zhang@intel.com>", "To": "dev@dpdk.org", "Cc": "pablo.de.lara.guarch@intel.com, jianjay.zhou@huawei.com, stable@dpdk.org", "Date": "Thu, 14 Jun 2018 12:02:57 +0100", "Message-Id": "<20180614110257.10967-1-roy.fan.zhang@intel.com>", "X-Mailer": "git-send-email 2.13.6", "Subject": "[dpdk-dev] [PATCH] crypto/virtio: fix iv physical address", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://dpdk.org/ml/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "The physical address of IV required by Virtio was computed using\ncrypto operations' physical address plus the offset. However not\nall crypto ops will have physical address field initialized and\ncompute it runtimely is costly. This patch fixes this problem by\nadding iv field in virtio_crypto_op_cookie and does a memcpy of\niv instead.\n\nFixes: 82adb12a1fce (\"crypto/virtio: support burst enqueue/dequeue\")\nCc: stable@dpdk.org\n\nSigned-off-by: Fan Zhang <roy.fan.zhang@intel.com>\n---\n drivers/crypto/virtio/virtio_cryptodev.c | 6 ++++++\n drivers/crypto/virtio/virtio_cryptodev.h | 3 +++\n drivers/crypto/virtio/virtio_rxtx.c | 8 +++++++-\n 3 files changed, 16 insertions(+), 1 deletion(-)", "diff": "diff --git a/drivers/crypto/virtio/virtio_cryptodev.c b/drivers/crypto/virtio/virtio_cryptodev.c\nindex df88953f6..6ffa7619c 100644\n--- a/drivers/crypto/virtio/virtio_cryptodev.c\n+++ b/drivers/crypto/virtio/virtio_cryptodev.c\n@@ -1223,6 +1223,12 @@ virtio_crypto_sym_pad_op_ctrl_req(\n \t/* Get cipher xform from crypto xform chain */\n \tcipher_xform = virtio_crypto_get_cipher_xform(xform);\n \tif (cipher_xform) {\n+\t\tif (cipher_xform->iv.length > VIRTIO_CRYPTO_MAX_IV_SIZE) {\n+\t\t\tVIRTIO_CRYPTO_SESSION_LOG_ERR(\n+\t\t\t\t\"cipher IV cannot longer than %u\",\n+\t\t\t\tVIRTIO_CRYPTO_MAX_IV_SIZE);\n+\t\t\treturn -1;\n+\t\t}\n \t\tif (is_chainned)\n \t\t\tret = virtio_crypto_sym_pad_cipher_param(\n \t\t\t\t&ctrl->u.sym_create_session.u.chain.para\ndiff --git a/drivers/crypto/virtio/virtio_cryptodev.h b/drivers/crypto/virtio/virtio_cryptodev.h\nindex e402c0309..676e008d9 100644\n--- a/drivers/crypto/virtio/virtio_cryptodev.h\n+++ b/drivers/crypto/virtio/virtio_cryptodev.h\n@@ -16,6 +16,8 @@\n \n #define NUM_ENTRY_VIRTIO_CRYPTO_OP 7\n \n+#define VIRTIO_CRYPTO_MAX_IV_SIZE 32\n+\n extern uint8_t cryptodev_virtio_driver_id;\n \n enum virtio_crypto_cmd_id {\n@@ -29,6 +31,7 @@ struct virtio_crypto_op_cookie {\n \tstruct virtio_crypto_op_data_req data_req;\n \tstruct virtio_crypto_inhdr inhdr;\n \tstruct vring_desc desc[NUM_ENTRY_VIRTIO_CRYPTO_OP];\n+\tuint8_t iv[VIRTIO_CRYPTO_MAX_IV_SIZE];\n };\n \n /*\ndiff --git a/drivers/crypto/virtio/virtio_rxtx.c b/drivers/crypto/virtio/virtio_rxtx.c\nindex 450392843..1b16221d0 100644\n--- a/drivers/crypto/virtio/virtio_rxtx.c\n+++ b/drivers/crypto/virtio/virtio_rxtx.c\n@@ -203,6 +203,8 @@ virtqueue_crypto_sym_enqueue_xmit(\n \tuint16_t req_data_len = sizeof(struct virtio_crypto_op_data_req);\n \tuint32_t indirect_vring_addr_offset = req_data_len +\n \t\tsizeof(struct virtio_crypto_inhdr);\n+\tuint32_t indirect_iv_addr_offset = indirect_vring_addr_offset +\n+\t\t\tsizeof(struct vring_desc) * NUM_ENTRY_VIRTIO_CRYPTO_OP;\n \tstruct rte_crypto_sym_op *sym_op = cop->sym;\n \tstruct virtio_crypto_session *session =\n \t\t(struct virtio_crypto_session *)get_session_private_data(\n@@ -259,7 +261,11 @@ virtqueue_crypto_sym_enqueue_xmit(\n \n \t/* indirect vring: iv of cipher */\n \tif (session->iv.length) {\n-\t\tdesc[idx].addr = cop->phys_addr + session->iv.offset;\n+\t\trte_memcpy(crypto_op_cookie->iv, rte_crypto_op_ctod_offset(cop,\n+\t\t\t\tuint8_t *, session->iv.offset),\n+\t\t\t\tsession->iv.length);\n+\t\tdesc[idx].addr = indirect_op_data_req_phys_addr +\n+\t\t\t\tindirect_iv_addr_offset;\n \t\tdesc[idx].len = session->iv.length;\n \t\tdesc[idx++].flags = VRING_DESC_F_NEXT;\n \t}\n", "prefixes": [] }