Cover Detail
Show a cover letter.
GET /api/covers/41033/?format=api
{ "id": 41033, "url": "http://patches.dpdk.org/api/covers/41033/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/cover/1528892062-4997-1-git-send-email-tomaszx.jozwiak@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": "<1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com", "date": "2018-06-13T12:13:44", "name": "[v3,00/38] crypto/qat: refactor to support multiple services", "submitter": { "id": 949, "url": "http://patches.dpdk.org/api/people/949/?format=api", "name": "Tomasz Jozwiak", "email": "tomaszx.jozwiak@intel.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/cover/1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com/mbox/", "series": [ { "id": 111, "url": "http://patches.dpdk.org/api/series/111/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=111", "date": "2018-06-13T12:13:44", "name": "crypto/qat: refactor to support multiple services", "version": 3, "mbox": "http://patches.dpdk.org/series/111/mbox/" } ], "comments": "http://patches.dpdk.org/api/covers/41033/comments/", "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 4934C1D9E8;\n\tWed, 13 Jun 2018 14:14:28 +0200 (CEST)", "from mga01.intel.com (mga01.intel.com [192.55.52.88])\n\tby dpdk.org (Postfix) with ESMTP id EDB231D5F4\n\tfor <dev@dpdk.org>; Wed, 13 Jun 2018 14:14:26 +0200 (CEST)", "from fmsmga001.fm.intel.com ([10.253.24.23])\n\tby fmsmga101.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t13 Jun 2018 05:14:25 -0700", "from tjozwiax-mobl.ger.corp.intel.com (HELO localhost.localdomain)\n\t([10.103.104.42])\n\tby fmsmga001.fm.intel.com with ESMTP; 13 Jun 2018 05:14:23 -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,218,1526367600\"; d=\"scan'208\";a=\"63727630\"", "From": "Tomasz Jozwiak <tomaszx.jozwiak@intel.com>", "To": "fiona.trahe@intel.com,\n\ttomaszx.jozwiak@intel.com,\n\tdev@dpdk.org", "Date": "Wed, 13 Jun 2018 14:13:44 +0200", "Message-Id": "<1528892062-4997-1-git-send-email-tomaszx.jozwiak@intel.com>", "X-Mailer": "git-send-email 2.7.4", "In-Reply-To": "<1523040732-3290-1-git-send-email-fiona.trahe@intel.com>", "References": "<1523040732-3290-1-git-send-email-fiona.trahe@intel.com>", "Subject": "[dpdk-dev] [PATCH v3 00/38] crypto/qat: refactor to support\n\tmultiple services", "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": "From: Pablo de Lara <pablo.de.lara.guarch@intel.com>\n\nThe QAT crypto PMD was peppered with references to rte_cryptodev artefacts.\nAlso the pci device it presented to the API layer was entirely owned by the one\ncryptodev device instance created by the probe.\nThis patchset refactors the PMD so one pci device can present out multiple different\ndevice instances to implement different APIs, specifically cryptodev and compressdev instances.\nThe code is refactored into\n - common code which handles the pci device discovery, configuration,\n queue-pair setup, stats gathering and data-path enqueue and dequeue.\n - service-specific code for symmetric crypto, which is a thin layer\n implementing the cryptodev API and using the common code.\n - place-holder files for service-specific code to support asymmetric\n crypto and compression services in future.\nThe code is all still in the crypto/qat directory as it is functionally unchanged,\nstill only supporting a symmetric crypto service. The code will move to drivers/common,\n/crypto and /compress in a future patch when this is needed for a compression PMD.\n\nChanges for v3:\n - fixed some commit messages\n - fixed compilation issue in one of the patches\n - split two commits into multiple commits\n\nChanges for v2:\n - fixed some bugs compiled in when debug enabled\n - added patch to remove MAX_NB_SESSIONS from config file\n - removed SPDX license patch as already applied\n - documented device name formats\n\nFiona Trahe (36):\n crypto/qat: add qat common header\n crypto/qat: add qat device files\n crypto/qat: add symmetric session file\n crypto/qat: change filename crypto to sym\n crypto/qat: rename fns for consistency\n crypto/qat: renamed sym-specific structs\n crypto/qat: make enqueue function generic\n crypto/qat: make dequeue function generic\n crypto/qat: move generic qp fn to qp file\n crypto/qat: separate sym-specific from generic qp setup\n crypto/qat: move sym-specific qp code to sym file\n crypto/qat: remove dependencies on cryptodev from common\n crypto/qat: move defines from sym to qp header file\n crypto/qat: create structures to support various generations\n crypto/qat: rename sgl related objects\n crypto/qat: move sgl related element to appropriate files\n crypto/qat: add QAT PCI device struct\n crypto/qat: use generic driver name for PCI registration\n crypto/qat: move to using new device structure\n crypto/qat: use common stats structures\n crypto/qat: rename functions which depend on cryptodev\n crypto/qat: move code into appropriate files\n crypto/qat: add lock around csr access and change logic\n crypto/qat: remove incorrect usage of bundle number\n crypto/qat: rename variables\n crypto/qat: modify debug message\n crypto/qat: free cookie pool on queue creation error\n crypto/qat: remove unused macro\n crypto/qat: move macro to common file\n crypto/qat: register appropriately named device\n crypto/qat: add max PCI devices to config file\n crypto/qat: optimize adf modulo function\n crypto/qat: remove unused arguments\n crypto/qat: make response process function inline\n crypto/qat: check for service type\n doc/qat: specify QAT driver and device name formats\n\nTomasz Jozwiak (2):\n crypto/qat: remove unused includes\n crypto/qat: remove configurable max number of sessions\n\n config/common_base | 6 +-\n config/rte_config.h | 7 +-\n doc/guides/cryptodevs/qat.rst | 12 +\n drivers/crypto/qat/Makefile | 10 +-\n drivers/crypto/qat/meson.build | 9 +-\n .../qat/qat_adf/adf_transport_access_macros.h | 9 +-\n drivers/crypto/qat/qat_adf/qat_algs.h | 126 --\n drivers/crypto/qat/qat_asym_pmd.c | 17 +\n drivers/crypto/qat/qat_asym_pmd.h | 15 +\n drivers/crypto/qat/qat_common.c | 107 ++\n drivers/crypto/qat/qat_common.h | 76 +\n drivers/crypto/qat/qat_comp_pmd.c | 18 +\n drivers/crypto/qat/qat_comp_pmd.h | 29 +\n drivers/crypto/qat/qat_crypto.c | 1696 -----------------\n drivers/crypto/qat/qat_crypto.h | 150 --\n drivers/crypto/qat/qat_device.c | 242 +++\n drivers/crypto/qat/qat_device.h | 78 +\n drivers/crypto/qat/qat_qp.c | 463 +++--\n drivers/crypto/qat/qat_qp.h | 106 ++\n drivers/crypto/qat/qat_sym.c | 568 ++++++\n drivers/crypto/qat/qat_sym.h | 154 ++\n ..._capabilities.h => qat_sym_capabilities.h} | 6 +-\n drivers/crypto/qat/qat_sym_pmd.c | 335 ++++\n drivers/crypto/qat/qat_sym_pmd.h | 40 +\n ...at_algs_build_desc.c => qat_sym_session.c} | 786 +++++++-\n drivers/crypto/qat/qat_sym_session.h | 143 ++\n drivers/crypto/qat/rte_qat_cryptodev.c | 180 --\n 27 files changed, 3042 insertions(+), 2346 deletions(-)\n delete mode 100644 drivers/crypto/qat/qat_adf/qat_algs.h\n create mode 100644 drivers/crypto/qat/qat_asym_pmd.c\n create mode 100644 drivers/crypto/qat/qat_asym_pmd.h\n create mode 100644 drivers/crypto/qat/qat_common.c\n create mode 100644 drivers/crypto/qat/qat_common.h\n create mode 100644 drivers/crypto/qat/qat_comp_pmd.c\n create mode 100644 drivers/crypto/qat/qat_comp_pmd.h\n delete mode 100644 drivers/crypto/qat/qat_crypto.c\n delete mode 100644 drivers/crypto/qat/qat_crypto.h\n create mode 100644 drivers/crypto/qat/qat_device.c\n create mode 100644 drivers/crypto/qat/qat_device.h\n create mode 100644 drivers/crypto/qat/qat_qp.h\n create mode 100644 drivers/crypto/qat/qat_sym.c\n create mode 100644 drivers/crypto/qat/qat_sym.h\n rename drivers/crypto/qat/{qat_crypto_capabilities.h => qat_sym_capabilities.h} (99%)\n create mode 100644 drivers/crypto/qat/qat_sym_pmd.c\n create mode 100644 drivers/crypto/qat/qat_sym_pmd.h\n rename drivers/crypto/qat/{qat_adf/qat_algs_build_desc.c => qat_sym_session.c} (59%)\n create mode 100644 drivers/crypto/qat/qat_sym_session.h\n delete mode 100644 drivers/crypto/qat/rte_qat_cryptodev.c" }