From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id C3B4AA0547; Thu, 28 Oct 2021 13:03:24 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id CF86C41100; Thu, 28 Oct 2021 13:03:23 +0200 (CEST) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by mails.dpdk.org (Postfix) with ESMTP id CFF5440DF4 for ; Thu, 28 Oct 2021 13:03:20 +0200 (CEST) X-IronPort-AV: E=McAfee;i="6200,9189,10150"; a="230642476" X-IronPort-AV: E=Sophos;i="5.87,189,1631602800"; d="scan'208";a="230642476" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 28 Oct 2021 03:41:21 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.87,189,1631602800"; d="scan'208";a="447923318" Received: from orsmsx606.amr.corp.intel.com ([10.22.229.19]) by orsmga003.jf.intel.com with ESMTP; 28 Oct 2021 03:41:21 -0700 Received: from orsmsx608.amr.corp.intel.com (10.22.229.21) by ORSMSX606.amr.corp.intel.com (10.22.229.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Thu, 28 Oct 2021 03:41:21 -0700 Received: from orsmsx607.amr.corp.intel.com (10.22.229.20) by ORSMSX608.amr.corp.intel.com (10.22.229.21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12; Thu, 28 Oct 2021 03:41:20 -0700 Received: from ORSEDG602.ED.cps.intel.com (10.7.248.7) by orsmsx607.amr.corp.intel.com (10.22.229.20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2242.12 via Frontend Transport; Thu, 28 Oct 2021 03:41:20 -0700 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (104.47.56.170) by edgegateway.intel.com (134.134.137.103) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2242.12; Thu, 28 Oct 2021 03:41:20 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=dMBkv7ACqwPJsw2SEStLLnArviDoApk9ENKeMfqbKjzzOW5+x5iim/4++vKu2w1aNAB79twArD68HwTB6y2JWNfKsxRwHuzuQM3t6DyTnchN7Tu7Oy/BGmZOyAI0rvCkr2OdnC7nEInyQKvlFBpZxDnWKUWuv2ijr9Uvp5I7jaMfOwvrqVVi7Z3PGTpioILiMxZRJVFpIyhPfhxSOuHvOaC1Nz6o+9lzLJ9S+bw0p150xg8ud0CUbOvCTzQ2dK4rKzpMRPGosdYa8npMrKQ3XssyxQExWy8GTwnFmsY5l4SY8C9oLlP97aCbno6yEwrCIyPAvvDTwbWBq8ApRcVvhw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=IN6lsS2L5kQ0FRO4BF9aB81ak1BC3KTps1szUQ9z8sU=; b=CK8ogTUX+81BZPta6HVXzZduPF4DM/mPf3aRXeQM/k1cyHekQDlnBFupBXH29dLgsjiWP8WZjHL+pcr1BLMV0RliP2BKX1krbLlVZZCHzRw3K3FGpg98ghmbElXyBAC+bLSa/wqW/Bwz3RoX+DQVnEHLZBgmuPsS8DcLwb0Al7plab8gSPMAJ5iMeSRQcqBlyTugPgrLSD3KrkUeQk3EOK4MyaHDZVgcNzhDod+QBMwu0NHpR7CKSHjjzPZK5JDHh38iejDn9wWenomV3pXP8JKhHkU11IqGDozY5mg24KtHhTSauuDIfAzqkTWEV57DHfRJkRjiMH4LGV3XbPOK7Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=intel.onmicrosoft.com; s=selector2-intel-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=IN6lsS2L5kQ0FRO4BF9aB81ak1BC3KTps1szUQ9z8sU=; b=pAqsr3EDNG1NUK630QH68lsluo8MdXjOjDesZxLNvC0lxNiK9EwOD8lGNZpnXTYAkW/cwyF7Nwg2xO+qr0ENziLwXiTIcKQuEweDwmnSzmfUGO2iNc0P9Rwi/WvW+nX3lAv0Gyv6cXKSCfl/SW7LwoG7OJPX9Xf63NUNHgz7bks= Received: from BYAPR11MB3399.namprd11.prod.outlook.com (2603:10b6:a03:79::15) by BY5PR11MB3910.namprd11.prod.outlook.com (2603:10b6:a03:185::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4649.13; Thu, 28 Oct 2021 10:41:19 +0000 Received: from BYAPR11MB3399.namprd11.prod.outlook.com ([fe80::2d17:a4c8:8edc:713e]) by BYAPR11MB3399.namprd11.prod.outlook.com ([fe80::2d17:a4c8:8edc:713e%4]) with mapi id 15.20.4628.020; Thu, 28 Oct 2021 10:41:19 +0000 From: "Troy, Rebecca" To: "Coyle, David" , "dev@dpdk.org" CC: "Zhang, Roy Fan" , "Power, Ciara" , "Doherty, Declan" , "Akhil Goyal" Thread-Topic: [PATCH] test/crypto: refactor docsis to show hidden cases Thread-Index: AQHXxzMGUl7M/dyzRkqBxSWc4jC2xKvoQGCAgAAAwCA= Date: Thu, 28 Oct 2021 10:41:19 +0000 Message-ID: References: <20211022105302.3652-1-rebecca.troy@intel.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-product: dlpe-windows dlp-version: 11.6.200.16 dlp-reaction: no-action authentication-results: intel.com; dkim=none (message not signed) header.d=none;intel.com; dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 6eeb2210-114d-4664-d837-08d999ff7a24 x-ms-traffictypediagnostic: BY5PR11MB3910: x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:10000; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: DMDHnIX5Fegcvn3QrVyc0u48QAQznUF+yBzWVPDJTOlti1yokj6rgdvmPb4dGBeBBC3Np9TcjRGqQ+xaAuFUj1CMRsBi31HwUDOTwpVI3L9BDo5O3wJFpDlI1UMIWirOT/0CLkG7SeKBk2Ce5022LvTSkDjqS4EuQhzVtojMftPlykEXZdBZC2GtWzr2vHoGKau6u+vj699InVwDZO90xNOC0zkijkQrk2HYGSJH9GMShgjNFKmYE4GwwAVChlMqLOv9LoeG1Jrt6bGdVYBbhDjdejWdlmp21Hisp5YkC7Q+E5js6wsH/otQWgaZiAwoNUJswGQMmDXrULWGmyaXunKva/yASk/SWdav1GSNg+ZJzcETXLIjHuG19H2IErJQvdZ0ITV/j78UuABJhy5l/YxetS60roIiIWt/eq/+7YACHcRUWFzZxozbLgKB7xlKwy8pExTwWsko4GYUhUDOq2fJD65314m4q1LW7ygf9fYJMStHomqO8vDwfxuzpRaS3dNx8UxXIEUJNXBZZ4lKep6cB34hp86pEI4Tp49wO56cokV3kPcBmYhAnNFg0d5pmdUnZBP6iHVSVHjfGtlSmczul2Q94VvAxcRICbf5Yk654WyjcvT66p6klnv4fGNt2ty37u8r2Hb23dmecaKX2sPj3i7TyzO5sqKBS9+/YTdRNnKJ4YgE7XJpFnkV9/WPWDMWAjqycOG7JUuElA7VQw== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:BYAPR11MB3399.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(366004)(52536014)(54906003)(82960400001)(110136005)(8676002)(122000001)(38100700002)(8936002)(4326008)(5660300002)(76116006)(6506007)(86362001)(53546011)(66556008)(26005)(66446008)(38070700005)(64756008)(66476007)(316002)(7696005)(55016002)(186003)(83380400001)(71200400001)(508600001)(2906002)(33656002)(9686003)(66946007); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?AS16wRJ4nuRfiMpwrHFdXTkqIdqUXTMkC75PwLUy12L21tTxvQAkvIzldWFL?= =?us-ascii?Q?tGr7dLRvOXMd4u0EBco2EWAPGs2LzTfeQ8TSVV50WkGr0Vh1/hAblgw4UjN+?= =?us-ascii?Q?wHPmWprPGsIbGgt8oooDYxNe6OP1QdVKsM7BHdajHbfnHTXHeXYNLV/bomoA?= =?us-ascii?Q?WYCKiaqR7KVn5PYksdZ7UgXOK1LoyfJaksonmg+wn/aRs9L/RAUA5qFez9Qu?= =?us-ascii?Q?qZFev4Ej2KJ0uXuKlpu6MI9ioqmXykg+ENP/NiC2Y+tvSqLR6LMBaReygI9b?= =?us-ascii?Q?ohO7qPRH9QZ1RpdENfkXrv4/dLc2jOHv2ExybSHwa35nXDIFddwXjcOFB2lh?= =?us-ascii?Q?YCHBP1EMofm9NSXivUw5DG04DE3b2E9XRI45lXKJPj3dh6qMpF6e9QYjyMuZ?= =?us-ascii?Q?I70R5XEULAGbmx1E609VBY5jgLca59Wwmg54kbKQQPte7GNRSV9k/2pBijZp?= =?us-ascii?Q?QTF5GmkBwuzkZHNeR0UXCUY2vJrh7EmB5vImIreD2jYYzWimfz27iAqS6Rur?= =?us-ascii?Q?RiVg0sSyBLuyJG3O44ndDvIgnWLQN32w8IxGockuRmKmBXvzueNhtXAar7LG?= =?us-ascii?Q?UlTgsLqntL/32IKURorL7qTiNoavjivmuhvNCQTgC4dv3eGLRJ0g7B15XA/t?= =?us-ascii?Q?DdDz2XQoyR/JjzVHhojHDjCFAR3j0sQGz/Br6yHnb6tlf9/BXcUMFxtaXucU?= =?us-ascii?Q?1zABVEkTStkGGQY5BHC25TbNePQEQyk/2ERQBO9L4ldySrjRRvNwiAIRUgRk?= =?us-ascii?Q?hhjpMFLhzDFdjmYpE3uHJwOG/cRbdumzB9ZAihQ/S6FmxVoWuvxjKlhNx1SQ?= =?us-ascii?Q?paYT3SyeSs2SWdyqH8a6a0FLe0tI/k4lkC/4r+QUqPwXBL4g7od1Ns8dJc8t?= =?us-ascii?Q?OmuCQkQEWbaTCG5ZXmewPXaH68aEM+xlGK3toWnJ0AtoFe6NlhKFpQeL6GIz?= =?us-ascii?Q?lJbk1MdVBk+WflicBzVjLtKC6jrCnRChSIzoUu8kNrNM5GeGMBUvdfPrITrt?= =?us-ascii?Q?4Zo05gFKQmwjUn7VEjJwQndh7Nr1ciqKR9GrGiLB040jPi8oaMtJ9l8UB16Q?= =?us-ascii?Q?+6YznY1VsDh+Zdc6cDz1bSzi2Rij3n7dsQW5iHvxbgs1wKm5BVAYenoSRmNV?= =?us-ascii?Q?o94chfaDXl2A76gAY3oPqLM4GsE8Ei+M+QD1m7SewCHG1rGDAHafYi7uDSmV?= =?us-ascii?Q?r6sIv0X2mIn0HXi0xJqszEr5XVPzgsBrCPHqjnejJJ7fOuRIvA1Ke4tFBB7y?= =?us-ascii?Q?p1CAbF+FBP1NTH1klkBdJCZ8VGo8e4C/PkHqB3+dAH7hU1bFmTXdckdcK/Yi?= =?us-ascii?Q?WKdzG3i03+Pt2oBparwJBy8NPiooZUZjfutej3Pvf8sFrqW5RO13kw9Ja0IA?= =?us-ascii?Q?YeU6Vh2/a4e9E6XCKlK6K3UvmhmTlfqKN9rDIvK6EhswbncKxg5s3w9pH7/0?= =?us-ascii?Q?e/8WUusx4mFRqB7gQF2XncGVyY5J1WaEWTMl549JjNq0gfqbozrfbKXZ9Rli?= =?us-ascii?Q?2GeVZDKlpon0K3xQsf+psF7mf5WvRmrHBuBpThHjMIl7qW1wBtRLjF7KdkWn?= =?us-ascii?Q?bKTHFDlD0a0hfRROJYhQkx9/ApHA3f6GJTN4uj/NZM8jr/jhQDWEsWgTcfEM?= =?us-ascii?Q?G2rV6Xk5wIp8nZ9N0GsQzgs=3D?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: BYAPR11MB3399.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 6eeb2210-114d-4664-d837-08d999ff7a24 X-MS-Exchange-CrossTenant-originalarrivaltime: 28 Oct 2021 10:41:19.2912 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: xGxrUFDRuYyMuyoDX/k2QowNIzk5P11SA/r+eVEII8ZQDfTvm6x/VgGH+IhZDAvw7tXL4KcUmMaAwuk5dk3QrA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY5PR11MB3910 X-OriginatorOrg: intel.com Subject: Re: [dpdk-dev] [PATCH] test/crypto: refactor docsis to show hidden cases X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" > -----Original Message----- > From: Coyle, David > Sent: Thursday 28 October 2021 11:34 > To: Troy, Rebecca ; dev@dpdk.org > Cc: Power, Ciara ; Zhang, Roy Fan > ; Akhil Goyal ; Doherty, > Declan > Subject: RE: [PATCH] test/crypto: refactor docsis to show hidden cases >=20 > Hi Rebecca, see below >=20 >=20 > > -----Original Message----- > > From: Troy, Rebecca > > > > In the current implementation, the docsis test cases are running and > > being reported as one test, despite the fact that multiple test cases > > are hidden inside i.e. "test_DOCSIS_PROTO_all" runs > > 52 test cases. Each docsis test case should be reported individually in= stead. >=20 > [DC] Should make "docsis" all uppercase in the commit message - "DOCSIS" >=20 > > > > This commit achieves this by removing the use of the > > test_DOCSIS_PROTO_all function and statically listing the test cases > > to run when building the test suite, which are then reported to the > > user by description. > > > > Signed-off-by: Rebecca Troy > > --- > > app/test/test_cryptodev.c | 265 +++++++----------- > > ...t_cryptodev_security_docsis_test_vectors.h | 159 +++++++++-- > > 2 files changed, 241 insertions(+), 183 deletions(-) > > > > >=20 > >=20 > > + > > static struct unit_test_suite docsis_proto_testsuite =3D { > > .suite_name =3D "Docsis Proto Unit Test Suite", >=20 > [DC] Outside the specific changes of this patch, but "Docsis" should be a= ll > uppercase in the suite name too - "DOCSIS"... could take this opportunity= to > fix up this minor one >=20 > > .setup =3D docsis_proto_testsuite_setup, > > .unit_test_cases =3D { > > - TEST_CASE_ST(ut_setup_security, ut_teardown, > > - test_DOCSIS_PROTO_all), > > + /* Uplink */ > > + ADD_UPLINK_TESTCASE(docsis_test_case_1) > > + ADD_UPLINK_TESTCASE(docsis_test_case_2) > > + ADD_UPLINK_TESTCASE(docsis_test_case_3) > > + ADD_UPLINK_TESTCASE(docsis_test_case_4) >=20 > >=20 > > > > -struct docsis_test_data docsis_test_case_1 =3D { > > +const struct docsis_test_data docsis_test_case_1 =3D { > > + .test_descr_uplink =3D {"AES-DOCSIS-BPI-128 and CRC Verify (24-byte " > > + "frame, Small offset and runt block encryption)"}, > > + .test_descr_downlink =3D {"CRC Generate and AES-DOCSIS-BPI-128 > > (24-byte " > > + "frame, Small offset and runt block encryption)"}, >=20 > [DC] This one is my fault when I supplied the descriptions, so apologies = about > this, but all the uplink descriptions should say "decryption" instead of > "encryption" >=20 > Also I think all the descriptions should say "Uplink" or "Downlink" at th= e > start. > This can be inferred from the order of AES-DOCSIS-BPI and CRC in the > description, but when I ran the test cases, I still had to think are thes= e Uplink > or Downlink tests. > It would be much clearer if it's stated explicitly >=20 > Regards, > David Great, thanks for the feedback David! Will look at implementing these chang= es in a v2. Thanks, Rebecca.