Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/93210/?format=api
{ "id": 93210, "url": "https://patches.dpdk.org/api/patches/93210/?format=api", "web_url": "https://patches.dpdk.org/project/dpdk/patch/20210512113655.568814-1-ciara.power@intel.com/", "project": { "id": 1, "url": "https://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": "<20210512113655.568814-1-ciara.power@intel.com>", "list_archive_url": "https://inbox.dpdk.org/dev/20210512113655.568814-1-ciara.power@intel.com", "date": "2021-05-12T11:36:48", "name": "[v4,0/7] test: refactor crypto unit test framework", "commit_ref": null, "pull_url": null, "state": null, "archived": false, "hash": null, "submitter": { "id": 978, "url": "https://patches.dpdk.org/api/people/978/?format=api", "name": "Power, Ciara", "email": "ciara.power@intel.com" }, "delegate": null, "mbox": "https://patches.dpdk.org/project/dpdk/patch/20210512113655.568814-1-ciara.power@intel.com/mbox/", "series": [], "comments": "https://patches.dpdk.org/api/patches/93210/comments/", "check": "pending", "checks": "https://patches.dpdk.org/api/patches/93210/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id 31704A0C43;\n\tWed, 12 May 2021 13:37:41 +0200 (CEST)", "from [217.70.189.124] (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id E3D5041106;\n\tWed, 12 May 2021 13:37:27 +0200 (CEST)", "from mga03.intel.com (mga03.intel.com [134.134.136.65])\n by mails.dpdk.org (Postfix) with ESMTP id AB1DC4003E\n for <dev@dpdk.org>; Wed, 12 May 2021 13:37:24 +0200 (CEST)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384;\n 12 May 2021 04:37:07 -0700", "from silpixa00400355.ir.intel.com (HELO\n silpixa00400355.ger.corp.intel.com) ([10.237.223.148])\n by fmsmga005.fm.intel.com with ESMTP; 12 May 2021 04:37:03 -0700" ], "IronPort-SDR": [ "\n eM8CJ1aUGTh1/XufuZRYndxWeE6Ka8RwZr11+ME51XbFkz0StNTB/8MSU4vkMYL0Zuy4dZaPIM\n cXKgLWeRynyw==", "\n R2+QThTqMvoltFQnMl8mn1Gx1VnGjd9gxZsO5Qk6Q5+CdY00/wzRKz1I4uzX/RcBTRMBiyvM5V\n 7eKKajiM/yBw==" ], "X-IronPort-AV": [ "E=McAfee;i=\"6200,9189,9981\"; a=\"199733770\"", "E=Sophos;i=\"5.82,293,1613462400\"; d=\"scan'208\";a=\"199733770\"", "E=Sophos;i=\"5.82,293,1613462400\"; d=\"scan'208\";a=\"625296843\"" ], "X-ExtLoop1": "1", "From": "Ciara Power <ciara.power@intel.com>", "To": "dev@dpdk.org", "Cc": "declan.doherty@intel.com, gakhil@marvell.com, aconole@redhat.com,\n hemant.agrawal@nxp.com, anoobj@marvell.com, ruifeng.wang@arm.com,\n asomalap@amd.com, ajit.khaparde@broadcom.com, g.singh@nxp.com,\n roy.fan.zhang@intel.com, Ciara Power <ciara.power@intel.com>", "Date": "Wed, 12 May 2021 11:36:48 +0000", "Message-Id": "<20210512113655.568814-1-ciara.power@intel.com>", "X-Mailer": "git-send-email 2.25.1", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "Subject": "[dpdk-dev] [PATCH v4 0/7] test: refactor crypto unit test framework", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <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 <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "The current crypto unit test framework is not granular enough to\naccurately track unit test results. This is caused by one testcase\nin a suite actually running multiple testcases, but only returning\none result.\n \nThe approach taken in this patchset allows a test suite have a\nlist of sub-testsuites, and/or a list of testcases as previously used.\nThe unit test suite runner can then recursively iterate and run the\nsub-testsuites, until it reaches a suite with testcases,\nand it then runs each testcase as it had done previously.\nIn the case of a testsuite with both testcases and sub-testsuites,\nthe testcases are executed first before iterating through the\nsub-testsuites.\n \nBy allowing this further breakdown into sub-testsuites,\na refactor of the crypto unit tests solves the issue of inaccurate\nreporting, as sub-testsuites can be used in place of the testcases\nthat had multiple testcases hidden on a sub level.\nThe blockcipher tests previously had these hidden testcases,\nbut are now sub-testsuites that are dynamically created and added to a\nparent test suite, allowing for each testcase status to be reported\ndirectly to the runner.\nThe cryptodev test suite is broken down into smaller suites that are\nused as sub-testsuites, which allows for more flexibility choosing which\nsub-testsuites should run for the current device.\nThe introduction of sub-testsuites also allows for more precise\nsetup/teardown functions, that can check the capabilities required to\nrun its testcases.\n \nFor example, when running the cryptodev_aesni_mb_autotest, \nthe parent Cryptodev Test Suite is executed.\nVarious sub-testsuites are added to the parent test suite, such as\nthe static suites of testcases that were once in the cryptodev_testsuite,\nand blockcipher suites.\nThe unit test runner can then run the Cryptodev parent test suite,\nwhich in turn will run the sub-testsuites.\n\nThe user is now required to create vdevs via EAL commandline args,\nthis is no longer done within the test app for crypto autotests.\n\nDocumentation will need to be added at a later stage,\nadding to the test document that isn't yet merged. [1]\n\n[1] https://patchwork.dpdk.org/project/dpdk/patch/20210309155757.615536-1-aconole@redhat.com/\n\n\nv4:\n - Fixed some return value bugs related to patch 4.\n - Rebased onto main.\nv3:\n - Added support for a testsuite having both a list of testcases,\n and a list of sub-testsuites.\n - Replaced PMD based parent testsuites with a cryptodev testsuite\n used by all autotests, with the exception of scheduler autotest.\n - Setup functions were added for all sub-testsuites, within which\n required capability support is checked.\n - The setup functions no longer create vdevs if needed,\n this must be done by the user when running the test.\n - Patch added to standardise return values for skipped testcases.\nv2:\n - Added macro in place of testcase/testsuite loops.\n - Added more detail in the summary output.\n - Moved testcase counts to the testsuite structure.\n - Flattened testsuite structure to remove union.\n - Added patch for fix of blockcipher test return value.\n - Squashed release note into last patch.\n\nCiara Power (7):\n app/test: refactor of unit test suite runner\n test: introduce parent testsuite format\n test/crypto: refactor to use sub-testsuites\n test/crypto: replace unsupported with skipped\n test/crypto: move testsuite params to header file\n test/crypto: fix return value on test skipped\n test/crypto: dynamically build blockcipher suite\n\n app/test/test.c | 176 +-\n app/test/test.h | 23 +-\n app/test/test_cryptodev.c | 2333 ++++++++++++++----------\n app/test/test_cryptodev.h | 32 +\n app/test/test_cryptodev_asym.c | 111 +-\n app/test/test_cryptodev_blockcipher.c | 423 ++++-\n app/test/test_cryptodev_blockcipher.h | 12 +-\n app/test/test_ipsec.c | 32 +-\n doc/guides/rel_notes/release_21_05.rst | 5 +\n 9 files changed, 2004 insertions(+), 1143 deletions(-)", "diff": null, "prefixes": [ "v4", "0/7" ] }