Cover Comment List
List cover comments
GET /api/covers/44636/comments/?format=api
[ { "id": 85731, "web_url": "http://patches.dpdk.org/comment/85731/", "msgid": "<2eb867f9-a148-9e82-2faa-016c778b2b42@nxp.com>", "list_archive_url": "https://inbox.dpdk.org/dev/2eb867f9-a148-9e82-2faa-016c778b2b42@nxp.com", "date": "2018-09-18T14:21:08", "subject": "Re: [dpdk-dev] [PATCH 00/10] Introducing the NXP CAAM job ring\n\tdriver", "submitter": { "id": 517, "url": "http://patches.dpdk.org/api/people/517/?format=api", "name": "Akhil Goyal", "email": "akhil.goyal@nxp.com" }, "content": "Hi Gagan,\n\nOn 9/13/2018 11:38 AM, Gagandeep Singh wrote:\n> The caam_jr PMD provides poll mode crypto driver\n> support for NXP SEC 4.x+ (CAAM) hardware accelerator.\n>\n> This patch has dependancy on below patches:\n> http://patchwork.dpdk.org/patch/43986/\n> http://patchwork.dpdk.org/patch/43964/\n>\n> Hemant Agrawal (10):\n> doc: add caam jr cryptodev details\n> crypto/caam_jr: introduce basic driver\n> crypto/caam_jr: add HW config for job rings\n> crypto/caam_jr: add device configuration routines\n> crypto/caam_jr: add queue config functions\n> crypto/caam_jr: add basic session config routines\n> crypto/caam_jr: add enqueue and dequeue routines\n> crypto/caam_jr: add auth cipher and aead session support\n> crypto/caam_jr: add stats support\n> crypto/caam_jr: add security offload support\n>\n> config/common_base | 8 +\n> config/common_linuxapp | 1 +\n> config/defconfig_arm64-dpaa-linuxapp-gcc | 4 +\n> doc/guides/cryptodevs/caam_jr.rst | 159 ++\n> doc/guides/cryptodevs/index.rst | 1 +\n> drivers/crypto/Makefile | 1 +\n> drivers/crypto/caam_jr/Makefile | 46 +\n> drivers/crypto/caam_jr/caam_jr.c | 2485 +++++++++++++++++\n> drivers/crypto/caam_jr/caam_jr.h | 257 ++\n> drivers/crypto/caam_jr/caam_jr_config.h | 207 ++\n> drivers/crypto/caam_jr/caam_jr_desc.h | 289 ++\n> drivers/crypto/caam_jr/caam_jr_hw.c | 365 +++\n> drivers/crypto/caam_jr/caam_jr_hw_specific.h | 503 ++++\n> drivers/crypto/caam_jr/caam_jr_log.h | 42 +\n> drivers/crypto/caam_jr/caam_jr_pvt.h | 288 ++\n> drivers/crypto/caam_jr/caam_jr_uio.c | 491 ++++\n> drivers/crypto/caam_jr/meson.build | 14 +\n> .../caam_jr/rte_pmd_caam_jr_version.map | 4 +\n> drivers/crypto/meson.build | 2 +-\n> 19 files changed, 5166 insertions(+), 1 deletion(-)\n> create mode 100644 doc/guides/cryptodevs/caam_jr.rst\n> create mode 100644 drivers/crypto/caam_jr/Makefile\n> create mode 100644 drivers/crypto/caam_jr/caam_jr.c\n> create mode 100644 drivers/crypto/caam_jr/caam_jr.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_config.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_desc.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_hw.c\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_hw_specific.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_log.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_pvt.h\n> create mode 100644 drivers/crypto/caam_jr/caam_jr_uio.c\n> create mode 100644 drivers/crypto/caam_jr/meson.build\n> create mode 100644 drivers/crypto/caam_jr/rte_pmd_caam_jr_version.map\n>\n\nI have some generic comments on your patchset:\n1. Please add description to the patches. None of them have it.\n2. Split of the patches could be better. You can have Hardware specific \ncode in one patch and then you can use that in the next patch.\n3. SG related stuff can be a separate patch.\n4. there are checkpatch issues which can be resolved.\n5. I could see extra spaces at many places. Please check.\n6. Function definitions are not as per DPDK coding style.\n\nThanks,\nAkhil", "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 E80BD532C;\n\tTue, 18 Sep 2018 16:21:23 +0200 (CEST)", "from EUR02-AM5-obe.outbound.protection.outlook.com\n\t(mail-eopbgr00063.outbound.protection.outlook.com [40.107.0.63])\n\tby dpdk.org (Postfix) with ESMTP id 4C2954CA6\n\tfor <dev@dpdk.org>; Tue, 18 Sep 2018 16:21:22 +0200 (CEST)", "from [10.232.134.144] (14.143.30.134) by\n\tDB7PR04MB4889.eurprd04.prod.outlook.com (2603:10a6:10:18::20) with\n\tMicrosoft SMTP Server (version=TLS1_2,\n\tcipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id\n\t15.20.1143.18; Tue, 18 Sep 2018 14:21:20 +0000" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=nxp.com; s=selector1;\n\th=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck;\n\tbh=6MDOmgp56EX13TAFGFHsHqvdTR40fkegdLQak3w6nW0=;\n\tb=ioW/P71In9VSzvQC1eAGWiXSw4p8WNUfZ4Bd+35GZSvuLIM3iPKL+wSuUpbONz0rpq0uJDYZK9TY6p6OGrDczSw4xBb0OQVhH3XuD+cTvgTo1hTrY3wtBWrwQGfrKF2Dbp7fEjY+I/OWcsAU50klOfth82UI7nqzRhqHtnhCCeQ=", "Authentication-Results": "spf=none (sender IP is )\n\tsmtp.mailfrom=akhil.goyal@nxp.com; ", "To": "Gagandeep Singh <g.singh@nxp.com>, dev@dpdk.org,\n\tHemant Agrawal <hemant.agrawal@nxp.com>", "References": "<20180913060846.29930-1-g.singh@nxp.com>", "From": "Akhil Goyal <akhil.goyal@nxp.com>", "Message-ID": "<2eb867f9-a148-9e82-2faa-016c778b2b42@nxp.com>", "Date": "Tue, 18 Sep 2018 19:51:08 +0530", "User-Agent": "Mozilla/5.0 (Windows NT 10.0; WOW64; rv:52.0) Gecko/20100101\n\tThunderbird/52.9.1", "MIME-Version": "1.0", "In-Reply-To": "<20180913060846.29930-1-g.singh@nxp.com>", "Content-Type": "text/plain; charset=utf-8; format=flowed", "Content-Transfer-Encoding": "7bit", "Content-Language": "en-US", "X-Originating-IP": "[14.143.30.134]", "X-ClientProxiedBy": "BM1PR01CA0093.INDPRD01.PROD.OUTLOOK.COM\n\t(2603:1096:b00:1::33) To DB7PR04MB4889.eurprd04.prod.outlook.com\n\t(2603:10a6:10:18::20)", "X-MS-PublicTrafficType": "Email", "X-MS-Office365-Filtering-Correlation-Id": "e6bcc15b-25b3-4bca-7c79-08d61d7201ad", "X-MS-Office365-Filtering-HT": "Tenant", "X-Microsoft-Antispam": "BCL:0; PCL:0;\n\tRULEID:(7020095)(4652040)(8989299)(5600074)(711020)(4618075)(2017052603328)(7153060)(7193020);\n\tSRVR:DB7PR04MB4889; ", "X-Microsoft-Exchange-Diagnostics": [ "1; DB7PR04MB4889;\n\t3:Cu9Q0ebJxfSteZA3NZ7sB2ajctY9oLlbEF/mogNohyiXNA6LTTktIfF+X+J8FxW2N6JvJ/DTM0aelZDP/kJEKN51ATSZOORuXOsfgtku7iPtmx5y5RTRb1q4fsW7hc+RzYoQBvyiouoAp1qXvZhOpBfsgp71vWGC6k2kHYCllYUNx9Mjir6XSMsQQgyNZZjEOevkJjpkpuBCuaX6conI7WTJtui8/OwRpFYuDUPh4cMU1MMTojUT1SYC8OnBOh9d;\n\t25:bS/jKiwPMZPy5k5KvMa7nj+5/Wm+oOjztNRstYjThGL6LjIolh1N8auqzPmzsP6tvDVqZpx8520jlSBQ82HfN4KqW1hmFns3VwkG6ELF+OHJmtSP34gQO48sL0omwFQCVibIzfYXIWOLJ68iLw/FjQF233MxQunVxuU12bku/3uiL2CNb7VLMXOadUuqJIevbOYpSvCpXR7uKyThg/pbcp/qi4QG20Vzt6RfmrUhnEHTEuqn2ikbo4FhF7vjK17KoF/jtoE1MECVjzNjugaAsObl39cPHST74x5oH3lyMgSsdB1vbVw/krHhAANcpSLtLzL2e1qf3Tmixm88cSULjg==;\n\t31:CDPgGEzcfHzFsGw6JIL88SvhCUqPvTpwo9xCl8oKqna4Gf7z1WyDqWmcG+5j3M+bwuPjrEw5plJ89NNfpjb66sGUeWXWtHccF8FC7fElK1YSP8EcUQK/0qnfIdzyorSLDy2v6LsSQNsxwPS6vId+lYmzly9Ol/baNoXNmI/2/qFsVIDbBQztP3e0KyOsZH9+aChzgqI7sJt+S7ETJSakTQTQkTbuJ+O7lIbCLdXkEtc=", "1; DB7PR04MB4889;\n\t20:AnZra+7xuqgkyaBsoKP3hkhIOi7xMByrHApcdJXIxUmuQHNC+Qgyi6X8HaGpkTTm7MDGoebDF9lRqJGpRRMyjuHphQpU/zORTsRnZ0JpB8B7b2F4EQFjQbzYmQDhDx9KxGrmgw4an+eWRIU00G4sSgVN1zCm2Adqax0aVqZBuxUIQytzz2uGJkQ6utHtpvk0mvOBuuOJIRcGxPmak0o0u3xxrR1sd/Cq26LnHKcAU69GWdqM4ks2dlSyC2eX1lb/hKxcqVfObdRMXbUaR1yy2oclOlEJxbU+Y4iYMrzffMd41LAJbT3cPAV8Jh8PyjQIZmPlYXe+GAG/cfpmWXI1Qqr5zJsVeg7teSRl3xKi7/z4z0FDgBIwirCWjPo4NI2B1EsdKXVuLq5br7j9ntHiu3iXUKOjs8Z5fgRmEzvF5ILYd4/EeN/VEU0od/QbqF9+asj5HedncwuBu0AjI2NgpqFUBQ8rEr+K1x4JyhX+WlMfv7Zm2XUcdpTIl6CjTa2p;\n\t4:b2U4Hz5XtMhI/xXxzFwIFBQoMH082ZVXGhr8lGO0Woy0Jd3qeouuIHnk7Iq6jdVT4mDdX1lP/OwsC0dddXZPsCY2p8rDicV12wALBvOwKPuVmpnCyxSIFjXcVDYtY72KNNAGi7hUHrMXATUK7TorsMzMWW7P7UaRdyOdPR1BwyTRlqlmmiYUDPye0KHUp1HChQQOsqfxALL102LQ8qd3iCCrFSStRFMHmGCJeIeFxLhHmq+7WXseU2thheD9AaDtlMngzMEeXg6XU0cE7EZWs0AV2FpyP/XcFIkFxmuzPViruVjSrVUBofzGiJeMH59g", "=?utf-8?q?1=3BDB7PR04MB4889=3B23=3AUhhi?=\n\t=?utf-8?q?u5zA00tiGUjxr508lKHuz4kTmfvc9p1pHz1FcWhldSRUm3kk58NIV6sJ?=\n\t=?utf-8?q?svNB47ogMN5EDp9Vdo9QIzmpHXCJ/KCQidLXi8ZK/riE/rHGF1l5f7Aa?=\n\t=?utf-8?q?MysTaAYUCX0Inw1YWlumEpJ/mVsGwRpYXikWTPxRF7KpYYX1Yi/y49Wp?=\n\t=?utf-8?q?YGhlDhqTt3Up+HttH6YVquC7h/hCJCS9hM8Lzy1EBfBVjYseY9ronRXW?=\n\t=?utf-8?q?piO/5JnvomwCpkZI5TJeIv05E5lpdaERtEPvtkxQc+pZ2EGux0c2Jxou?=\n\t=?utf-8?q?XfZeF374zSmOxYdrfJlQyvY182xUbhLC0ulzuBgCr8TxyFt0abrZDoW+?=\n\t=?utf-8?q?Zx/aCAAQ7xCFeJMVGz8hlX+l7Pp8vMK7KLXl207SobFC8eUS9rUGsjtR?=\n\t=?utf-8?q?VCcYWDsD7OFOnkGt2dV+0acooxgw1aLmTB6p/bP85qAnMVQcC5WXV58x?=\n\t=?utf-8?q?RI74pnqw3OFhjQGCEu4tAkSR8Denffkg4Ff0+6anya1KnVgazUP3X5wh?=\n\t=?utf-8?q?2g0OXOFy/cmfz6xGhaVoE1ieep0XhxUbd+Swx02thSdpHIwVlw7Bc4EY?=\n\t=?utf-8?q?HH/T5CyOuF5r07Z1R7mMuNO1orCDbEJ4FWXm2AOwnNXemKU60+DvJ/mq?=\n\t=?utf-8?q?IxesN+hvU183QB1oCi/bDmvUmuOeCh2L1LVi8BHB+MnHtRZYJzDLwZR1?=\n\t=?utf-8?q?Fj4/W3JLbACk5bTv3bzEAaWraWQoh83mtoZ2qrgwZZzjX6wg12qR6IZb?=\n\t=?utf-8?q?mBhS2nmlhkWX73tknMVePJxUkl3ZjoeXIW1gIo8fooV7rNT5Q6QWMFQV?=\n\t=?utf-8?q?JIcdX52mI5pamftMdodXDOEKVDPz+hyBJy6U/VdQwXxcQxvs0ulQe9ho?=\n\t=?utf-8?q?TLC3+49r8myiiQB5cgHZcjROHKRf0uHbDK9nRqOew7PTZmbPnHFBAowK?=\n\t=?utf-8?q?HPIaJOzHuakzCKKBEXtvOu8dwRXdS0DSNv0JHfqP9PxcwfPPR0YmHb3l?=\n\t=?utf-8?q?dJgueKrZvVgdxhf/wzPV0IpvCthPYvdNbDmqfudwqG/igfIHu56yPG0j?=\n\t=?utf-8?q?QGVs2sy9p/FdvtyMKA+ewJbSyjYIgnfvXkK3dkajlE92P8hwVmSItgny?=\n\t=?utf-8?q?oU8hluzvjOflYZ91JkOPkjjvXfl78AZoKGKupExYfEhDMd+UB0u9uX2V?=\n\t=?utf-8?q?9xCOV1Awakug9LXKWyS+KaYN9tt7hERwG9+q/K10LOQGZCLnHP2XgJnM?=\n\t=?utf-8?q?13WIxepaBPBbsa52VzupYoMgUS2g93oae9yfZ3pNFxbxJfcEan+LHt4U?=\n\t=?utf-8?q?m8k7QFMgMeu2CaC8VCjUIsy0GaIOgl1YX8pHVMAJwAujXokQ+b7WdPlJ?=\n\t=?utf-8?q?w4gBtrVtxRDgvU/oSn8TSMLkgAbtUZ4yu/Sj4P5EVV4E5TOtLh1teiNi?=\n\t=?utf-8?q?x0aknj4H+Oj18SSyRrwuQbDZ8Ys+9878/YAuhkO36TaIK9y0vGLsMKJt?=\n\t=?utf-8?q?86Z5r6jWHTpqaJYcvdGwqR29H44QRzPZ8JQzxj92MQ1gh/rsyt5TMX91?=\n\t=?utf-8?q?jTDcgv98B7VzAsjT3qZct8feMKwa+tczl9hXoxJb2l0i2lEJUTpSzkNd?=\n\t=?utf-8?q?apSbr6jl5vseWCUXwMp+l8xkPRgfeJYWK8+MnW+mvbq1r/rCocfdbLiH?=\n\t=?utf-8?q?8VZ8/7gh5NLLzayyPuO8XZ9iMANy?=", "1; DB7PR04MB4889;\n\t6:MVajoyCeGNvusMOMg3JC+5Fk9BR8tiHVqXHZKKxzkWXtFCU91UXvCIQu5Xwte9v4oRvU+6Gj7jXJIEp0C92lXjewNzuUKhF7yHsL5zbpXoMCVFgR0mHijwFO1J/vTec9JOksqeaRrF2ZWFRcsM+QrtLNw0Y4JHqPbr/8veKFleNEpK2YAqcaafsqVItCMnS1QeR8a4yb1f1YWYQ/DMVMt/Nbx3oPrZq9eUyXNLklR8qgo4HJPsafSRCJvK2AFzdXEPC7iE7U6T4kf4nE+BlCYmo2unv/eOlxk1zKP9QpPZkv8kZlTFO5SdayEkRu8IWbmw3wVKdDtoB7L2Be7kvKFiIwUDVS4e3QkopuPb2A9zXrUrwpfCC6LioPDeUKgAWOiM7da8heanmbmDbSRmy/KfTkKC11hSreebx43w0+tLX9Z5RQGNaGONM9Rx4V8N8sYtCFNBF+JJTvj8YxJYW2cg==;\n\t5:wrB2IWGVhPryxMBHT1axGSBx73w37akRdQWdr6dkpqGaE9uu35yqhg8gPG/6au5cBy7eV6ZBRJGSLvJ1cUi46gZ+YWRHjNaS20If6g4ILS1CC3wsVCQcYdSUmu6Ns9UUA1ch+/OFtevgN3zkTMaXpkOXhNC/ESZUhWRtzIKevAs=;\n\t7:fgunjTBwjGBD124P/sWP5J5WISXWZ62CLfzRyeFt4SVi6H/5lIVAJ049+6du9RVOYMX25zAVq68VGX+anN5uu+57o3GgcFp7sPrsmF2qokPQPIqEPWb5GQrYxGqx0DjFZ7w3rUqzDgYMRLsyjIWIhIzT9W6xEBK8wMKu9V7WTh6tJsjeUJWDwpXqHHl0iYNR8cN5ktp/lmmUVC/g20QnBeXLQIFUTrh6vLP+I34oDsVpBvPEUVogWWl0QsO6A1Hc" ], "X-MS-TrafficTypeDiagnostic": "DB7PR04MB4889:", "X-Microsoft-Antispam-PRVS": "<DB7PR04MB4889BD25B23AEF36E5899D92E61D0@DB7PR04MB4889.eurprd04.prod.outlook.com>", "X-Exchange-Antispam-Report-Test": "UriScan:(192374486261705);", "X-MS-Exchange-SenderADCheck": "1", "X-Exchange-Antispam-Report-CFA-Test": "BCL:0; PCL:0;\n\tRULEID:(8211001083)(6040522)(2401047)(5005006)(8121501046)(3231355)(944501410)(52105095)(93006095)(93001095)(10201501046)(3002001)(6055026)(149027)(150027)(6041310)(20161123562045)(20161123558120)(201703131423095)(201702281528075)(20161123555045)(201703061421075)(201703061406153)(20161123564045)(20161123560045)(201708071742011)(7699050);\n\tSRVR:DB7PR04MB4889; BCL:0; PCL:0; RULEID:; SRVR:DB7PR04MB4889; ", "X-Forefront-PRVS": "0799B1B2D7", "X-Forefront-Antispam-Report": "SFV:NSPM;\n\tSFS:(10009020)(6049001)(346002)(396003)(376002)(136003)(366004)(39860400002)(189003)(199004)(966005)(6116002)(3846002)(81156014)(110136005)(81166006)(16576012)(68736007)(316002)(53936002)(6636002)(6486002)(229853002)(6306002)(5660300001)(65826007)(6666003)(25786009)(8936002)(77096007)(36756003)(26005)(47776003)(58126008)(3260700006)(5009440100003)(2906002)(64126003)(16526019)(11346002)(66066001)(53546011)(956004)(386003)(31696002)(2616005)(65806001)(305945005)(52146003)(2486003)(7736002)(476003)(105586002)(217873002)(86362001)(14444005)(76176011)(8676002)(65956001)(52116002)(486006)(67846002)(106356001)(230700001)(23676004)(478600001)(97736004)(446003)(50466002)(31686004)(44832011)(6246003)(110426005);\n\tDIR:OUT; SFP:1101; SCL:1; SRVR:DB7PR04MB4889; H:[10.232.134.144]; FPR:;\n\tSPF:None; LANG:en; PTR:InfoNoRecords; MX:1; A:1; ", "Received-SPF": "None (protection.outlook.com: nxp.com does not designate\n\tpermitted sender hosts)", "X-Microsoft-Antispam-Message-Info": "9KUIt73wEJO6/wtlCi2rnpaPJayrXRC+g8ZN23niXMSJAprnZTvU9kMJ/9u3EIG5vzI8PjbBpbU3y6zv1fMf6GylDs832630xHWxj1v69wuncHaCVrj7CdV5YBHSc5Y8tZypwcF0gYkaVbhiYgdvNsJku17YtWZYndOybIjqLcpehAk3EjOHh5RMffGuz95nqZAQmsXAatRLmzuWhaZqXNEhQ6eW29HJJh2bpKgQh5DdILRUqzk0qGm9LoIkytCsoIIvwwf6/xnjc1dROyexL4k4DHi8IooFoy1DGiJmaXStWVbP/kVLxct6RZ/HC44IeTp5X9agPCRDdop3T5AopdolJ00WEQyQxFk85zopYyY=", "SpamDiagnosticOutput": "1:99", "SpamDiagnosticMetadata": "NSPM", "X-OriginatorOrg": "nxp.com", "X-MS-Exchange-CrossTenant-OriginalArrivalTime": "18 Sep 2018 14:21:20.3102\n\t(UTC)", "X-MS-Exchange-CrossTenant-Network-Message-Id": "e6bcc15b-25b3-4bca-7c79-08d61d7201ad", "X-MS-Exchange-CrossTenant-FromEntityHeader": "Hosted", "X-MS-Exchange-CrossTenant-Id": "686ea1d3-bc2b-4c6f-a92c-d99c5c301635", "X-MS-Exchange-Transport-CrossTenantHeadersStamped": "DB7PR04MB4889", "Subject": "Re: [dpdk-dev] [PATCH 00/10] Introducing the NXP CAAM job ring\n\tdriver", "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://mails.dpdk.org/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "addressed": null } ]