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 7468CA00C2; Tue, 27 Sep 2022 11:38:32 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 151E541133; Tue, 27 Sep 2022 11:38:32 +0200 (CEST) Received: from mga03.intel.com (mga03.intel.com [134.134.136.65]) by mails.dpdk.org (Postfix) with ESMTP id 47531410D0 for ; Tue, 27 Sep 2022 11:38:30 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1664271510; x=1695807510; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=Q5u4rOvMsHPB3C8Rsx9P0oy0zyQfJ7BGn5W3MmvZ2eM=; b=cMUq5eBumDi8zPKVZUsmSmOERiGJvaqRPmcFwRAMsIKHV1xu0+PYcQ4w wTdARjx1Wbib4YkbKUT4rtUYZeKpsJHqN/8y1CYJuezjYVXr1IkBnYhd7 thh1Tewy2q700Tt50yNwuXjLMddxQq6povsPeRwg9H7xBi8AxpE7P8pGC 8kKvEywafwXX5BoQb8G2rzBUeKkuoLofbukJe7YE2+Fy1ktcC06Y1QseI SXE+sMNL7nVpjUyetMjY3JtFQmSi49W70EnY/Hcm5evErePFz4xvczkfC MpLcP7x3FA3NAUYiwSwL0vyr6UPQUxJ28SMobNw9It51y/X0i3bdOrtHV w==; X-IronPort-AV: E=McAfee;i="6500,9779,10482"; a="302751536" X-IronPort-AV: E=Sophos;i="5.93,349,1654585200"; d="scan'208";a="302751536" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by orsmga103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 27 Sep 2022 02:38:29 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10482"; a="796708422" X-IronPort-AV: E=Sophos;i="5.93,349,1654585200"; d="scan'208";a="796708422" Received: from fmsmsx601.amr.corp.intel.com ([10.18.126.81]) by orsmga005.jf.intel.com with ESMTP; 27 Sep 2022 02:38:28 -0700 Received: from fmsmsx611.amr.corp.intel.com (10.18.126.91) by fmsmsx601.amr.corp.intel.com (10.18.126.81) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31; Tue, 27 Sep 2022 02:38:28 -0700 Received: from fmsedg601.ED.cps.intel.com (10.1.192.135) by fmsmsx611.amr.corp.intel.com (10.18.126.91) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2375.31 via Frontend Transport; Tue, 27 Sep 2022 02:38:28 -0700 Received: from NAM11-DM6-obe.outbound.protection.outlook.com (104.47.57.168) by edgegateway.intel.com (192.55.55.70) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2375.31; Tue, 27 Sep 2022 02:38:27 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=BnqIy738EGCpChTg8JTUu2EkOK2fHCa4X4UJQnsD/blgv3qyJGJ0/d814BTYyXa45JRJ5j+x5Wup+i1hAA4x+m6DxyWcqS88rx8vxXpON7Zac0GPCbIqWbsuavZNBoNWMtJ2gKOW3wxupVVhghcXJH2oqtOilqOzmT+aqCRx/ZCXvJC9oy2u948i5BPVZZRD8Z2kDOpxUFGETdyr8E078X84J8P2OudS1djFDU658fJ9Lvwb2O23pE1n0Dp2DoAz3a8lSsRRxPABSHlwe+sUuVqZ1IYxdGFyEoje9fncatDt29UUZ/AjQCejxV/p6vOfTURejh1bMvq5c85/Ic5w5A== 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=eMFKGvqBzsOZowfuGT/XI03eFRcvfkcltgl83LUWPBY=; b=HE2hPkPv+ad93wlmjhOgrxHG7eLYcBpDYSPDj+PkuV1zTDho99WU6hOtnZ3QOaAyqSHLB5US3/AUUnHgi6LdbH3dbMx6lRpiu9G4GoubhJ5Feq+Batx94smmCf6nojrPUg4gN+AGGXLfsDzpPbSB6ghDhhHABjg7FQN4176RU/VeUjkaKKz4oIGU7z1kqW56fnc4iNqlVzbkfBOk92TDUT9Vbif7XYNhBvteBaMvshLr5hqxeCtCJRuM8X81eIIluCiIcmZQJdTN4WJji5VavMdLfn5pQkyn9KOhS2bus1BAPAYYb5mdbo9IqyJfcOXd4t87RdF5XCMVht6LMQ0kow== 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 Received: from PH0PR11MB5832.namprd11.prod.outlook.com (2603:10b6:510:141::7) by IA1PR11MB6267.namprd11.prod.outlook.com (2603:10b6:208:3e5::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5654.25; Tue, 27 Sep 2022 09:38:20 +0000 Received: from PH0PR11MB5832.namprd11.prod.outlook.com ([fe80::e9d1:3c1e:d9ed:57b2]) by PH0PR11MB5832.namprd11.prod.outlook.com ([fe80::e9d1:3c1e:d9ed:57b2%6]) with mapi id 15.20.5654.024; Tue, 27 Sep 2022 09:38:20 +0000 From: "Gujjar, Abhinandan S" To: Volodymyr Fialko , "dev@dpdk.org" , Jerin Jacob Kollanukkaran , Pavan Nikhilesh Bhagavatula , Shijith Thotton , "Hemant Agrawal" , Sachin Saxena , "Jayatheerthan, Jay" CC: Akhil Goyal , Anoob Joseph Subject: RE: [PATCH 1/3] eventdev: introduce event cryptodev vector type Thread-Topic: [PATCH 1/3] eventdev: introduce event cryptodev vector type Thread-Index: AQHYp+j7eGHDtoEzhEu4kLvCSEfnEK3ufXbQgANhlgCAAXEGYIAABhKAgAAAn8A= Date: Tue, 27 Sep 2022 09:38:20 +0000 Message-ID: References: <20220622013839.405771-1-vfialko@marvell.com> <20220804095907.97895-1-vfialko@marvell.com> <20220804095907.97895-2-vfialko@marvell.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-reaction: no-action dlp-version: 11.6.500.17 authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PH0PR11MB5832:EE_|IA1PR11MB6267:EE_ x-ms-office365-filtering-correlation-id: a376e76e-7d27-477f-79c7-08daa06c03c5 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: 2mg+iooA14aQhkN+gjIknHsJyU4Uz3HwCQecHvaOlTQ7zfrjH6tMNy0CzXtG+I64oNK2X0H7jcQ0T74/wsMyvfZePFTUDnAXTdFjDR0TjA+kH2a+xslqOBDuRowmbwBr0aY00hzLLe7RnVCHBLDXx0SJMwdmM9NsmtymoAly01GctSWTsaQXfZgwXYHThDKGtpi/fMAaN7u+9e89qjnnkBzL46cb3IQ+ouShd1nvPRT6/E64492zqbzto+NzqYrEV4xvJZcC2LjIen4OJmZvpSmf0bCvYRT+n3rR4nU2VbI7LNxvLeCaIx6N4WlkJPIN2HEOnUmXUSAfa4XjQgGD8jHcSbOq5Cojv6rzOIPFvCuzWfsvU5b3fOmJq1TI9UR7K64Gn/v/3+O+Gz75Ey1PK9ZE8y+yIv34nMI6SDURUHntlpVbdnQlCvqBYxW96sywWA5HzSAp3YmCEKBkBFeqnaI9bOE1nmA9RlB0Gt9BCkqSXrt+Za8ivglC/A6icmD1HdgZyOEiugd/0iZ9daTD7erY2QYbzO5wv5mYb60JWsDWXADjol39FXT8um+4EXD4Jn50hXKLbxzDNGvsAWH+vW4tS1fXJQIMGVbi7eqS/K+8pHmiBLcgaUKVngT2BdpUZQFMquZC0MW+pJ49nB7dQzm3kLTGggeAb43UB3wznWEgJFbuuFNJpfncuob5X9j9nzInpNGCDmD8XHkLRN61hfwM0/J6S46fwqBV41nMY6sk1YCXU0Z3FjaOnL/ptTeuIwRMFrcz6Js/Io5SRwH1VQ== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR11MB5832.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230022)(136003)(39860400002)(376002)(366004)(346002)(396003)(451199015)(71200400001)(478600001)(7696005)(53546011)(41300700001)(6506007)(83380400001)(9686003)(186003)(2906002)(5660300002)(26005)(55016003)(30864003)(64756008)(54906003)(6636002)(110136005)(66476007)(66556008)(66946007)(76116006)(52536014)(8676002)(8936002)(66446008)(316002)(4326008)(82960400001)(86362001)(38100700002)(38070700005)(122000001)(33656002)(559001)(579004); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?a0J7FsPnfkjAoLkjXD+P+WsjgsAzInvelGAi8y3R/bQuo/Yk72z0N2mHESFF?= =?us-ascii?Q?UDCMjtd7b4H3vdIj/VmsoitZwEA9wS8nzqFITAbaJUVBNxj2x+RZPckacSuH?= =?us-ascii?Q?C30MXF6ICJrBLohU8rRgAbNsIYfkkSU+Vf2ueEGooR532F5ua0QaaTX0Xd8s?= =?us-ascii?Q?u8xJq/+Aq6f7oa3KoPu7jgB3ojPlMGG2JV9NpAS5wJlt0dHs0vOIt2KYCsWi?= =?us-ascii?Q?Os+kV2sERNIk4Y2BSgvkpyvxCGBPtclMoI2B22GWsaBsJky7pcuao34lSLPl?= =?us-ascii?Q?sqgsMrzSDH7zvbwMe64Ajih7P61z9xB2MDLiVMhwlD24kWF4dMQK/bSzHq0c?= =?us-ascii?Q?nhfOgKLuQpA4gYlN/f+HzhbzRiReSqc2BADkwYDuaODubtmdSkBbTbUJHxTm?= =?us-ascii?Q?ZvH1nZzaK2HrjiVitZw35nFqJTD0kB2tiK9ocoUezkdfKYQ/pXFN1WTo/rru?= =?us-ascii?Q?UVIOHP38iPkJNu2TRLdvVe3MXsEoIWclZTgPcD2/NdxnCThqNg2JERaHvOc8?= =?us-ascii?Q?kPhWafOIjMMNP7xRsL2qG7Axd040z5wU1XONMXqoIpYPQI8nKUHDZMdDPbZU?= =?us-ascii?Q?++dJIHxdqMYFGxtuzKBCmKPxlbGmE0MlxYw5pomZP3O+BZqQDGLYVUFZhMVb?= =?us-ascii?Q?DtzcjyuVrKFte/qlpZIKOgFcLDi/ruQBWYL7oVdNX8ysHCxuSbSeLcx8xTyH?= =?us-ascii?Q?r+vu8IbtUXlWyXlP7v4vAJIIYpNGp0tXgFVERxCcV92ZKXHBJel01ZsgniKi?= =?us-ascii?Q?Io/soEEEcqPvQEfCuihswhuJ8PxsYIxbz8GI+PbhC3oVxPrGFqIph3g42Xl+?= =?us-ascii?Q?5wmz4f/rdssOMq5Zs8DrxggrcNz25lQsFRzikr/TNUG0Aba8cAsoIRMce/OJ?= =?us-ascii?Q?x2mYdC+kw1p0PDOhC6M7VW1npLydj0mGvCrH2XJXN/PGREQkp66pi+g9dmp+?= =?us-ascii?Q?oVOgfpLCZlf748aR8xKzVykw3ZdDKHp1GUNm8ImRSV0SPi+TzKnfrbzfUm1o?= =?us-ascii?Q?3Mn3RdtKKi+Tu9VhZXgcklMFM7snYodBZ8m/hxD08Hhpol17PIebIq0DWXLC?= =?us-ascii?Q?W4hTOmQ76BW1N2hrx7ScsUGIkpjM/n1gvPJwRyNzD+2hozb2tvpKGCogHZqC?= =?us-ascii?Q?K47aTHcLm4gK+hIYLJWwLX2JK//MobZbBuLsqsw/6JfQQeEZfjMcSzWIZygH?= =?us-ascii?Q?nKw9CQ0w5GXAlMAoefqazYaIiNzn4OTRfyoLkeCS5/9e4ihxAMxv5rJnIvDB?= =?us-ascii?Q?zc4M+iSJxPaEZmbiUijv99RCWREnsLYECxgm5zab7sIF98/S/Bqzu/GzEaIr?= =?us-ascii?Q?jqegXLg3Nkr5CbMuRi8ncsJupoMsbQfNBIORWaLAzg53xb7RWd2lHnbBPvce?= =?us-ascii?Q?9IAMrB+lAlnG47YdH+F9B9YZiOxmSWdTUzcvpCfDCm6ARywZd2iYdBjUst/g?= =?us-ascii?Q?oDOpu85jghmFi/q7vJWgDpHEtyrYXIDKyNgekVEcLMsunM1UKLDfFJAe7A72?= =?us-ascii?Q?zXWjpyfjslpEqV0EXDbD3U0yRPmu/9sjpEPVfIIr29TntvWNiVYuImaLhsNn?= =?us-ascii?Q?JtzhkDH34oqZMpTbTX8Qli1if3LzusywxUpXId96gNDi3dADoHGoXUC3YloP?= =?us-ascii?Q?Jg=3D=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: PH0PR11MB5832.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: a376e76e-7d27-477f-79c7-08daa06c03c5 X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Sep 2022 09:38:20.6063 (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: P09izzrX22zcYNeTCFFQAgZ3cGJiOlQmnDqaWzsRyaBuhM5qcR0Cg4muaDO+ohiNme+rmyO+fyVVsA93L+EkpNJpzHkThy5cGeaNkBtFaVE= X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB6267 X-OriginatorOrg: intel.com 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 > -----Original Message----- > From: Volodymyr Fialko > Sent: Tuesday, September 27, 2022 2:55 PM > To: Gujjar, Abhinandan S ; dev@dpdk.org; Jer= in > Jacob Kollanukkaran ; Pavan Nikhilesh Bhagavatula > ; Shijith Thotton ; > Hemant Agrawal ; Sachin Saxena > ; Jayatheerthan, Jay > > Cc: Akhil Goyal ; Anoob Joseph > Subject: RE: [PATCH 1/3] eventdev: introduce event cryptodev vector type >=20 >=20 >=20 > > -----Original Message----- > > From: Gujjar, Abhinandan S > > Sent: Tuesday, September 27, 2022 11:05 AM > > To: Volodymyr Fialko ; dev@dpdk.org; Jerin Jacob > > Kollanukkaran ; Pavan Nikhilesh Bhagavatula > > ; Shijith Thotton ; > > Hemant Agrawal ; Sachin Saxena > > ; Jayatheerthan, Jay > > > > Cc: Akhil Goyal ; Anoob Joseph > > > > Subject: [EXT] RE: [PATCH 1/3] eventdev: introduce event cryptodev > > vector type > > > > External Email > > > > ---------------------------------------------------------------------- > > > > > > > -----Original Message----- > > > From: Volodymyr Fialko > > > Sent: Monday, September 26, 2022 4:32 PM > > > To: Gujjar, Abhinandan S ; > > > dev@dpdk.org; Jerin Jacob Kollanukkaran ; Pavan > > > Nikhilesh Bhagavatula ; Shijith Thotton > > > ; Hemant Agrawal ; > > > Sachin Saxena ; Jayatheerthan, Jay > > > > > > Cc: Akhil Goyal ; Anoob Joseph > > > > > > Subject: RE: [PATCH 1/3] eventdev: introduce event cryptodev vector > > > type > > > > > > > > > > > > > -----Original Message----- > > > > From: Gujjar, Abhinandan S > > > > Sent: Saturday, September 24, 2022 10:44 AM > > > > To: Volodymyr Fialko ; dev@dpdk.org; Jerin > > > > Jacob Kollanukkaran ; Pavan Nikhilesh > > > > Bhagavatula ; Shijith Thotton > > > > ; Hemant Agrawal ; > > > > Sachin Saxena ; Jayatheerthan, Jay > > > > > > > > Cc: Akhil Goyal ; Anoob Joseph > > > > > > > > Subject: [EXT] RE: [PATCH 1/3] eventdev: introduce event cryptodev > > > > vector type > > > > > > > > External Email > > > > > > > > ------------------------------------------------------------------ > > > > -- > > > > -- > > > > > > > > > > > > > -----Original Message----- > > > > > From: Volodymyr Fialko > > > > > Sent: Thursday, August 4, 2022 3:29 PM > > > > > To: dev@dpdk.org; Jerin Jacob ; Gujjar, > > > > > Abhinandan S ; Pavan Nikhilesh > > > > > ; Shijith Thotton > > > > > ; Hemant Agrawal > ; > > > > > Sachin Saxena ; Jayatheerthan, Jay > > > > > > > > > > Cc: gakhil@marvell.com; anoobj@marvell.com; Volodymyr Fialko > > > > > > > > > > Subject: [PATCH 1/3] eventdev: introduce event cryptodev vector > > > > > type > > > > > > > > > > Introduce ability to aggregate crypto operations processed by > > > > > event crypto adapter into single event containing > > > > > rte_event_vector whose event type is > RTE_EVENT_TYPE_CRYPTODEV_VECTOR. > > > > > > > > > > Application should set RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR > in > > > > > rte_event_crypto_adapter_queue_conf::flag and provide vector > > > > > configuration with respect of > > > > > rte_event_crypto_adapter_vector_limits, > > > > > which could be obtained by calling > > > > > rte_event_crypto_adapter_vector_limits_get, to enable vectorizati= on. > > > > > > > > > > The event crypto adapter would be responsible for vectorizing > > > > > the crypto operations based on provided response information in > > > > > rte_event_crypto_metadata::response_info. > > > > > > > > > > Updated drivers and tests accordingly to new API. > > > > > > > > > > Signed-off-by: Volodymyr Fialko > > > > > --- > > > > > app/test-eventdev/test_perf_common.c | 10 +- > > > > > app/test/test_event_crypto_adapter.c | 12 ++- > > > > > .../prog_guide/event_crypto_adapter.rst | 23 +++- > > > > > drivers/event/cnxk/cn10k_eventdev.c | 4 +- > > > > > drivers/event/cnxk/cn9k_eventdev.c | 5 +- > > > > > drivers/event/dpaa/dpaa_eventdev.c | 9 +- > > > > > drivers/event/dpaa2/dpaa2_eventdev.c | 9 +- > > > > > drivers/event/octeontx/ssovf_evdev.c | 4 +- > > > > > lib/eventdev/eventdev_pmd.h | 35 +++++- > > > > > lib/eventdev/eventdev_trace.h | 6 +- > > > > > lib/eventdev/rte_event_crypto_adapter.c | 90 ++++++++++++= ++-- > > > > > lib/eventdev/rte_event_crypto_adapter.h | 101 > +++++++++++++++++- > > > > > lib/eventdev/rte_event_eth_rx_adapter.h | 3 +- > > > > > lib/eventdev/rte_eventdev.h | 8 ++ > > > > > 14 files changed, 276 insertions(+), 43 deletions(-) > > > > > > > > > > > > > I don't see dataplane implementation of vectorization in the crypto > adapter! > > > > Is it missed out in the patch? > > > > comments inline. > > > > > > > Right now we are targeting crypto_cn10k PMD and ipsec-secgw event > > > mode to support vectorization. > > Is there a way to test this? When can be dataplane changes expected? > > > If the spec looks okay, support in s/w crypto adapter and other h/w PMDs = can > be added by respective maintainers. Currently, we are adding library chan= ge, > support for one PMD and an application to test the feature. Feature is ex= posed > with capability flag to not break existing functionality. Got it. How do I test this feature without data plane changes? This design is right now tested for cn10k, I am not sure this works for sw = adapter. If there is a way to test this, please let me know. I need to have perf data with and without vectorization support to approve. >=20 > > > > > > > > diff --git a/app/test-eventdev/test_perf_common.c b/app/test- > > > > > eventdev/test_perf_common.c index 81420be73a..c770bc93f6 100644 > > > > > --- a/app/test-eventdev/test_perf_common.c > > > > > +++ b/app/test-eventdev/test_perf_common.c > > > > > @@ -837,14 +837,14 @@ perf_event_crypto_adapter_setup(struct > > > > > test_perf *t, struct prod_data *p) > > > > > } > > > > > > > > > > if (cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) { > > > > > - struct rte_event response_info; > > > > > + struct rte_event_crypto_adapter_queue_conf conf; > > > > > > > > > > - response_info.event =3D 0; > > > > > - response_info.sched_type =3D RTE_SCHED_TYPE_ATOMIC; > > > > > - response_info.queue_id =3D p->queue_id; > > > > > + memset(&conf, 0, sizeof(conf)); > > > > > + conf.ev.sched_type =3D RTE_SCHED_TYPE_ATOMIC; > > > > > + conf.ev.queue_id =3D p->queue_id; > > > > > ret =3D rte_event_crypto_adapter_queue_pair_add( > > > > > TEST_PERF_CA_ID, p->ca.cdev_id, p->ca.cdev_qp_id, > > > > > - &response_info); > > > > > + &conf); > > > > > } else { > > > > > ret =3D rte_event_crypto_adapter_queue_pair_add( > > > > > TEST_PERF_CA_ID, p->ca.cdev_id, p->ca.cdev_qp_id, > > > NULL); diff > > > > > --git a/app/test/test_event_crypto_adapter.c > > > > > b/app/test/test_event_crypto_adapter.c > > > > > index 2ecc7e2cea..bb617c1042 100644 > > > > > --- a/app/test/test_event_crypto_adapter.c > > > > > +++ b/app/test/test_event_crypto_adapter.c > > > > > @@ -1175,6 +1175,10 @@ test_crypto_adapter_create(void) static > > > > > int > > > > > test_crypto_adapter_qp_add_del(void) > > > > > { > > > > > + struct rte_event_crypto_adapter_queue_conf queue_conf =3D { > > > > > + .ev =3D response_info, > > > > > + }; > > > > > + > > > > > uint32_t cap; > > > > > int ret; > > > > > > > > > > @@ -1183,7 +1187,7 @@ test_crypto_adapter_qp_add_del(void) > > > > > > > > > > if (cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) { > > > > > ret =3D > > > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID, > > > > > - TEST_CDEV_ID, TEST_CDEV_QP_ID, > > > > > &response_info); > > > > > + TEST_CDEV_ID, TEST_CDEV_QP_ID, > > > > > &queue_conf); > > > > > } else > > > > > ret =3D > > > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID, > > > > > TEST_CDEV_ID, TEST_CDEV_QP_ID, > > > NULL); @@ -1206,6 +1210,10 @@ > > > > > configure_event_crypto_adapter(enum > > > > > rte_event_crypto_adapter_mode mode) > > > > > .new_event_threshold =3D 1200, > > > > > }; > > > > > > > > > > + struct rte_event_crypto_adapter_queue_conf queue_conf =3D { > > > > > + .ev =3D response_info, > > > > > + }; > > > > > + > > > > > uint32_t cap; > > > > > int ret; > > > > > > > > > > @@ -1238,7 +1246,7 @@ configure_event_crypto_adapter(enum > > > > > rte_event_crypto_adapter_mode mode) > > > > > > > > > > if (cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) { > > > > > ret =3D > > > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID, > > > > > - TEST_CDEV_ID, TEST_CDEV_QP_ID, > > > > > &response_info); > > > > > + TEST_CDEV_ID, TEST_CDEV_QP_ID, > > > > > &queue_conf); > > > > > } else > > > > > ret =3D > > > > > rte_event_crypto_adapter_queue_pair_add(TEST_ADAPTER_ID, > > > > > TEST_CDEV_ID, TEST_CDEV_QP_ID, NULL); diff > > > --git > > > > > a/doc/guides/prog_guide/event_crypto_adapter.rst > > > > > b/doc/guides/prog_guide/event_crypto_adapter.rst > > > > > index 4fb5c688e0..554df7e358 100644 > > > > > --- a/doc/guides/prog_guide/event_crypto_adapter.rst > > > > > +++ b/doc/guides/prog_guide/event_crypto_adapter.rst > > > > > @@ -201,10 +201,10 @@ capability, event information must be > > > > > passed to the add API. > > > > > > > > > > ret =3D rte_event_crypto_adapter_caps_get(id, evdev, &ca= p); > > > > > if (cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) { > > > > > - struct rte_event event; > > > > > + struct rte_event_crypto_adapter_queue_conf > > > > > + conf; > > > > > > > > > > - // Fill in event information & pass it to add AP= I > > > > > - rte_event_crypto_adapter_queue_pair_add(id, cdev= _id, > qp_id, > > > > > &event); > > > > > + // Fill in conf.event information & pass it to a= dd API > > > > > + rte_event_crypto_adapter_queue_pair_add(id, > > > > > + cdev_id, qp_id, &conf); > > > > > } else > > > > > rte_event_crypto_adapter_queue_pair_add(id, > > > > > cdev_id, qp_id, NULL); > > > > > > > > > > @@ -291,6 +291,23 @@ the ``rte_crypto_op``. > > > > > rte_memcpy(op + len, &m_data, sizeof(m_data)); > > > > > } > > > > > > > > > > +Enable event vectorization > > > > > +~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > > + > > > > > +The event crypto adapter can aggregate outcoming crypto > > > > > +operations based on provided response information of > > > > > +``rte_event_crypto_metadata::response_info`` > > > > > +and generate a ``rte_event`` containing ``rte_event_vector`` > > > > > +whose event type is ``RTE_EVENT_TYPE_CRYPTODEV_VECTOR``. > > > > > +To enable vectorization application should set > > > > > +RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR in > > > > > +``rte_event_crypto_adapter_queue_conf::flag`` and provide > > > > > +vector configuration(size, mempool, etc.) with respect of > > > > > +``rte_event_crypto_adapter_vector_limits``, which could be > > > > > +obtained by calling ``rte_event_crypto_adapter_vector_limits_get= ()``. > > > > > + > > > > > +The RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR capability > > > indicates > > > > > +whether PMD supports this feature. > > > > > + > > > > > Start the adapter instance > > > > > ~~~~~~~~~~~~~~~~~~~~~~~~~~ > > > > > > > > > > diff --git a/drivers/event/cnxk/cn10k_eventdev.c > > > > > b/drivers/event/cnxk/cn10k_eventdev.c > > > > > index 5a0cab40a9..e74ec57382 100644 > > > > > --- a/drivers/event/cnxk/cn10k_eventdev.c > > > > > +++ b/drivers/event/cnxk/cn10k_eventdev.c > > > > > @@ -889,11 +889,11 @@ static int > > > > > cn10k_crypto_adapter_qp_add(const struct rte_eventdev *event_dev, > > > > > const struct rte_cryptodev *cdev, > > > > > int32_t queue_pair_id, > > > > > - const struct rte_event *event) > > > > > + const struct > rte_event_crypto_adapter_queue_conf > > > > > *conf) > > > > > { > > > > > struct cnxk_sso_evdev *dev =3D cnxk_sso_pmd_priv(event_dev); > > > > > > > > > > - RTE_SET_USED(event); > > > > > + RTE_SET_USED(conf); > > > > > > > > > > CNXK_VALID_DEV_OR_ERR_RET(event_dev->dev, "event_cn10k"); > > > > > CNXK_VALID_DEV_OR_ERR_RET(cdev->device, "crypto_cn10k"); diff > > > > > -- git a/drivers/event/cnxk/cn9k_eventdev.c > > > > > b/drivers/event/cnxk/cn9k_eventdev.c > > > > > index 2e27030049..45ed547cb0 100644 > > > > > --- a/drivers/event/cnxk/cn9k_eventdev.c > > > > > +++ b/drivers/event/cnxk/cn9k_eventdev.c > > > > > @@ -1120,11 +1120,12 @@ cn9k_crypto_adapter_caps_get(const > > > > > struct rte_eventdev *event_dev, static int > > > > > cn9k_crypto_adapter_qp_add(const struct rte_eventdev *event_dev, > > > > > const struct rte_cryptodev *cdev, > > > > > - int32_t queue_pair_id, const struct rte_event *event) > > > > > + int32_t queue_pair_id, > > > > > + const struct > rte_event_crypto_adapter_queue_conf > > > > > *conf) > > > > > { > > > > > struct cnxk_sso_evdev *dev =3D cnxk_sso_pmd_priv(event_dev); > > > > > > > > > > - RTE_SET_USED(event); > > > > > + RTE_SET_USED(conf); > > > > > > > > > > CNXK_VALID_DEV_OR_ERR_RET(event_dev->dev, "event_cn9k"); > > > > > CNXK_VALID_DEV_OR_ERR_RET(cdev->device, "crypto_cn9k"); diff > > > > > --git a/drivers/event/dpaa/dpaa_eventdev.c > > > > > b/drivers/event/dpaa/dpaa_eventdev.c > > > > > index ff6cc0be18..2b9ecd9fbf 100644 > > > > > --- a/drivers/event/dpaa/dpaa_eventdev.c > > > > > +++ b/drivers/event/dpaa/dpaa_eventdev.c > > > > > @@ -26,6 +26,7 @@ > > > > > #include > > > > > #include > > > > > #include > > > > > +#include > > > > > #include #include > > > > > #include @@ > > > > > -775,10 > > > > > +776,10 @@ static int dpaa_eventdev_crypto_queue_add(const > > > > > +struct > > > > > rte_eventdev *dev, > > > > > const struct rte_cryptodev *cryptodev, > > > > > int32_t rx_queue_id, > > > > > - const struct rte_event *ev) > > > > > + const struct rte_event_crypto_adapter_queue_conf > *conf) > > > > > { > > > > > struct dpaa_eventdev *priv =3D dev->data->dev_private; > > > > > - uint8_t ev_qid =3D ev->queue_id; > > > > > + uint8_t ev_qid =3D conf->ev.queue_id; > > > > > u16 ch_id =3D priv->evq_info[ev_qid].ch_id; > > > > > int ret; > > > > > > > > > > @@ -786,10 +787,10 @@ dpaa_eventdev_crypto_queue_add(const > > > > > struct rte_eventdev *dev, > > > > > > > > > > if (rx_queue_id =3D=3D -1) > > > > > return dpaa_eventdev_crypto_queue_add_all(dev, > > > > > - cryptodev, ev); > > > > > + cryptodev, &conf->ev); > > > > > > > > > > ret =3D dpaa_sec_eventq_attach(cryptodev, rx_queue_id, > > > > > - ch_id, ev); > > > > > + ch_id, &conf->ev); > > > > > if (ret) { > > > > > DPAA_EVENTDEV_ERR( > > > > > "dpaa_sec_eventq_attach failed: ret: %d\n", ret); diff - > > > -git > > > > > a/drivers/event/dpaa2/dpaa2_eventdev.c > > > > > b/drivers/event/dpaa2/dpaa2_eventdev.c > > > > > index ffc7b8b073..0137736794 100644 > > > > > --- a/drivers/event/dpaa2/dpaa2_eventdev.c > > > > > +++ b/drivers/event/dpaa2/dpaa2_eventdev.c > > > > > @@ -26,6 +26,7 @@ > > > > > #include > > > > > #include > > > > > #include > > > > > +#include > > > > > #include #include > > > > > > > > > > > > > > > @@ -865,10 +866,10 @@ static int > > > > > dpaa2_eventdev_crypto_queue_add(const struct rte_eventdev *dev, > > > > > const struct rte_cryptodev *cryptodev, > > > > > int32_t rx_queue_id, > > > > > - const struct rte_event *ev) > > > > > + const struct rte_event_crypto_adapter_queue_conf > *conf) > > > > > { > > > > > struct dpaa2_eventdev *priv =3D dev->data->dev_private; > > > > > - uint8_t ev_qid =3D ev->queue_id; > > > > > + uint8_t ev_qid =3D conf->ev.queue_id; > > > > > struct dpaa2_dpcon_dev *dpcon =3D priv->evq_info[ev_qid].dpcon; > > > > > int ret; > > > > > > > > > > @@ -876,10 +877,10 @@ dpaa2_eventdev_crypto_queue_add(const > > > > > struct rte_eventdev *dev, > > > > > > > > > > if (rx_queue_id =3D=3D -1) > > > > > return dpaa2_eventdev_crypto_queue_add_all(dev, > > > > > - cryptodev, ev); > > > > > + cryptodev, &conf->ev); > > > > > > > > > > ret =3D dpaa2_sec_eventq_attach(cryptodev, rx_queue_id, > > > > > - dpcon, ev); > > > > > + dpcon, &conf->ev); > > > > > if (ret) { > > > > > DPAA2_EVENTDEV_ERR( > > > > > "dpaa2_sec_eventq_attach failed: ret: %d\n", ret); diff > > > --git > > > > > a/drivers/event/octeontx/ssovf_evdev.c > > > > > b/drivers/event/octeontx/ssovf_evdev.c > > > > > index 9e14e35d10..17acd8ef64 100644 > > > > > --- a/drivers/event/octeontx/ssovf_evdev.c > > > > > +++ b/drivers/event/octeontx/ssovf_evdev.c > > > > > @@ -745,12 +745,12 @@ static int > > > > > ssovf_crypto_adapter_qp_add(const struct rte_eventdev *dev, > > > > > const struct rte_cryptodev *cdev, > > > > > int32_t queue_pair_id, > > > > > - const struct rte_event *event) > > > > > + const struct > rte_event_crypto_adapter_queue_conf > > > > > *conf) > > > > > { > > > > > struct cpt_instance *qp; > > > > > uint8_t qp_id; > > > > > > > > > > - RTE_SET_USED(event); > > > > > + RTE_SET_USED(conf); > > > > > > > > > > if (queue_pair_id =3D=3D -1) { > > > > > for (qp_id =3D 0; qp_id < cdev->data->nb_queue_pairs; qp_id++) > > > > > { diff --git a/lib/eventdev/eventdev_pmd.h > > > > > b/lib/eventdev/eventdev_pmd.h index 69402668d8..bcfc9cbcb2 > > > > > 100644 > > > > > --- a/lib/eventdev/eventdev_pmd.h > > > > > +++ b/lib/eventdev/eventdev_pmd.h > > > > > @@ -907,6 +907,7 @@ rte_event_pmd_selftest_seqn(struct rte_mbuf > > > > > *mbuf) } > > > > > > > > > > struct rte_cryptodev; > > > > > +struct rte_event_crypto_adapter_queue_conf; > > > > > > > > > > /** > > > > > * This API may change without prior notice @@ -961,11 +962,11 > > > > > @@ typedef int (*eventdev_crypto_adapter_caps_get_t) > > > > > * - <0: Error code returned by the driver function. > > > > > * > > > > > */ > > > > > -typedef int (*eventdev_crypto_adapter_queue_pair_add_t) > > > > > - (const struct rte_eventdev *dev, > > > > > - const struct rte_cryptodev *cdev, > > > > > - int32_t queue_pair_id, > > > > > - const struct rte_event *event); > > > > > +typedef int (*eventdev_crypto_adapter_queue_pair_add_t)( > > > > > + const struct rte_eventdev *dev, > > > > > + const struct rte_cryptodev *cdev, > > > > > + int32_t queue_pair_id, > > > > > + const struct rte_event_crypto_adapter_queue_conf > > > > > *queue_conf); > > > > > > > > > > > > > > > /** > > > > > @@ -1074,6 +1075,27 @@ typedef int > > > > > (*eventdev_crypto_adapter_stats_reset) > > > > > (const struct rte_eventdev *dev, > > > > > const struct rte_cryptodev *cdev); > > > > > > > > > > +struct rte_event_crypto_adapter_vector_limits; > > > > > +/** > > > > > + * Get event vector limits for a given event, crypto device pair= . > > > > > + * > > > > > + * @param dev > > > > > + * Event device pointer > > > > > + * > > > > > + * @param cdev > > > > > + * Crypto device pointer > > > > > + * > > > > > + * @param[out] limits > > > > > + * Pointer to the limits structure to be filled. > > > > > + * > > > > > + * @return > > > > > + * - 0: Success. > > > > > + * - <0: Error code returned by the driver function. > > > > > + */ > > > > > +typedef int (*eventdev_crypto_adapter_vector_limits_get_t)( > > > > > + const struct rte_eventdev *dev, const struct rte_cryptodev > *cdev, > > > > > + struct rte_event_crypto_adapter_vector_limits *limits); > > > > > + > > > > > /** > > > > > * Retrieve the event device's eth Tx adapter capabilities. > > > > > * > > > > > @@ -1339,6 +1361,9 @@ struct eventdev_ops { > > > > > /**< Get crypto stats */ > > > > > eventdev_crypto_adapter_stats_reset crypto_adapter_stats_reset; > > > > > /**< Reset crypto stats */ > > > > > + eventdev_crypto_adapter_vector_limits_get_t > > > > > + crypto_adapter_vector_limits_get; > > > > > + /**< Get event vector limits for the crypto adapter */ > > > > > > > > > > eventdev_eth_rx_adapter_q_stats_get > > > > > eth_rx_adapter_queue_stats_get; > > > > > /**< Get ethernet Rx queue stats */ diff --git > > > > > a/lib/eventdev/eventdev_trace.h b/lib/eventdev/eventdev_trace.h > > > > > index 5ec43d80ee..d48cd58850 100644 > > > > > --- a/lib/eventdev/eventdev_trace.h > > > > > +++ b/lib/eventdev/eventdev_trace.h > > > > > @@ -18,6 +18,7 @@ extern "C" { > > > > > #include > > > > > > > > > > #include "rte_eventdev.h" > > > > > +#include "rte_event_crypto_adapter.h" > > > > > #include "rte_event_eth_rx_adapter.h" > > > > > #include "rte_event_timer_adapter.h" > > > > > > > > > > @@ -271,11 +272,12 @@ RTE_TRACE_POINT( RTE_TRACE_POINT( > > > > > rte_eventdev_trace_crypto_adapter_queue_pair_add, > > > > > RTE_TRACE_POINT_ARGS(uint8_t adptr_id, uint8_t cdev_id, > > > > > - const void *event, int32_t queue_pair_id), > > > > > + int32_t queue_pair_id, > > > > > + const struct rte_event_crypto_adapter_queue_conf > *conf), > > > > > rte_trace_point_emit_u8(adptr_id); > > > > > rte_trace_point_emit_u8(cdev_id); > > > > > rte_trace_point_emit_i32(queue_pair_id); > > > > > - rte_trace_point_emit_ptr(event); > > > > > + rte_trace_point_emit_ptr(conf); > > > > > ) > > > > > > > > > > RTE_TRACE_POINT( > > > > > diff --git a/lib/eventdev/rte_event_crypto_adapter.c > > > > > b/lib/eventdev/rte_event_crypto_adapter.c > > > > > index 7c695176f4..73a4f231e2 100644 > > > > > --- a/lib/eventdev/rte_event_crypto_adapter.c > > > > > +++ b/lib/eventdev/rte_event_crypto_adapter.c > > > > > @@ -921,11 +921,12 @@ int > > > > > rte_event_crypto_adapter_queue_pair_add(uint8_t id, > > > > > uint8_t cdev_id, > > > > > int32_t queue_pair_id, > > > > > - const struct rte_event *event) > > > > > + const struct > rte_event_crypto_adapter_queue_conf > > > > > *conf) > > > > > { > > > > > + struct rte_event_crypto_adapter_vector_limits limits; > > > > > struct event_crypto_adapter *adapter; > > > > > - struct rte_eventdev *dev; > > > > > struct crypto_device_info *dev_info; > > > > > + struct rte_eventdev *dev; > > > > > uint32_t cap; > > > > > int ret; > > > > > > > > > > @@ -951,11 +952,47 @@ > > > > > rte_event_crypto_adapter_queue_pair_add(uint8_t > > > > > id, > > > > > } > > > > > > > > > > if ((cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND) > && > > > > > - (event =3D=3D NULL)) { > > > > > + (conf =3D=3D NULL)) { > > > > > RTE_EDEV_LOG_ERR("Conf value can not be NULL for > > > dev_id=3D%u", > > > > > cdev_id); > > > > > return -EINVAL; > > > > > } > > > > Newline? > > > > > > > Ack > > > > > > > > + if ((conf !=3D NULL) && > > > > Checking conf twice? > > > Will rewrite as if conf =3D=3D NULL/else, to avoid double checking. > > > > > > > > + (conf->flags & > RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR)) { > > > > Else condition if the flags is not set? > > > There's no additional handing for case when flag is no set. > > > > > > > > + if ((cap & > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR) =3D=3D 0) { > > > > > + RTE_EDEV_LOG_ERR("Event vectorization is > not > > > > > supported," > > > > > + "dev %" PRIu8 " cdev %" > PRIu8, id, > > > > > + cdev_id); > > > > > + return -ENOTSUP; > > > > > + } > > > > > + > > > > > + ret =3D rte_event_crypto_adapter_vector_limits_get( > > > > > + adapter->eventdev_id, cdev_id, &limits); > > > > > + if (ret < 0) { > > > > > + RTE_EDEV_LOG_ERR("Failed to get event > device vector > > > > > " > > > > > + "limits, dev %" PRIu8 " cdev > %" PRIu8, > > > > > + id, cdev_id); > > > > > + return -EINVAL; > > > > > + } > > > > New line? Please check other cases. > > > > > > > Ack > > > > > > > > + if (conf->vector_sz < limits.min_sz || > > > > > + conf->vector_sz > limits.max_sz || > > > > > + conf->vector_timeout_ns < limits.min_timeout_ns || > > > > > + conf->vector_timeout_ns > limits.max_timeout_ns > || > > > > > + conf->vector_mp =3D=3D NULL) { > > > > > + RTE_EDEV_LOG_ERR("Invalid event vector > > > > > configuration," > > > > > + " dev %" PRIu8 " cdev %" > PRIu8, > > > > > + id, cdev_id); > > > > > + return -EINVAL; > > > > > + } > > > > > + if (conf->vector_mp->elt_size < > > > > > + (sizeof(struct rte_event_vector) + > > > > > + (sizeof(uintptr_t) * conf->vector_sz))) { > > > > > + RTE_EDEV_LOG_ERR("Invalid event vector > > > > > configuration," > > > > > + " dev %" PRIu8 " cdev %" > PRIu8, > > > > > + id, cdev_id); > > > > > + return -EINVAL; > > > > > + } > > > > > + } > > > > > > > > > > dev_info =3D &adapter->cdevs[cdev_id]; > > > > > > > > > > @@ -990,7 +1027,7 @@ > > > rte_event_crypto_adapter_queue_pair_add(uint8_t > > > > > id, > > > > > ret =3D (*dev->dev_ops->crypto_adapter_queue_pair_add)(dev, > > > > > dev_info->dev, > > > > > queue_pair_id, > > > > > - event); > > > > > + conf); > > > > > if (ret) > > > > > return ret; > > > > > > > > > > @@ -1030,8 +1067,8 @@ > > > > > rte_event_crypto_adapter_queue_pair_add(uint8_t > > > > > id, > > > > > rte_service_component_runstate_set(adapter->service_id, 1); > > > > > } > > > > > > > > > > - rte_eventdev_trace_crypto_adapter_queue_pair_add(id, cdev_id, > > > > > event, > > > > > - queue_pair_id); > > > > > + rte_eventdev_trace_crypto_adapter_queue_pair_add(id, > cdev_id, > > > > > + queue_pair_id, conf); > > > > > return 0; > > > > > } > > > > > > > > > > @@ -1290,3 +1327,44 @@ > > > > > rte_event_crypto_adapter_event_port_get(uint8_t > > > > > id, uint8_t *event_port_id) > > > > > > > > > > return 0; > > > > > } > > > > > + > > > > > +int > > > > > +rte_event_crypto_adapter_vector_limits_get( > > > > > + uint8_t dev_id, uint16_t cdev_id, > > > > > + struct rte_event_crypto_adapter_vector_limits *limits) { > > > > > + struct rte_cryptodev *cdev; > > > > > + struct rte_eventdev *dev; > > > > > + uint32_t cap; > > > > > + int ret; > > > > > + > > > > > + RTE_EVENTDEV_VALID_DEVID_OR_ERR_RET(dev_id, -EINVAL); > > > > > + > > > > > + if (!rte_cryptodev_is_valid_dev(cdev_id)) { > > > > > + RTE_EDEV_LOG_ERR("Invalid dev_id=3D%" PRIu8, > cdev_id); > > > > > + return -EINVAL; > > > > > + } > > > > > + > > > > > + if (limits =3D=3D NULL) > > > > > + return -EINVAL; > > > > Add appropriate error message like above? > > > Ack, will add. > > > > > > > > + > > > > > + dev =3D &rte_eventdevs[dev_id]; > > > > > + cdev =3D rte_cryptodev_pmd_get_dev(cdev_id); > > > > > + > > > > > + ret =3D rte_event_crypto_adapter_caps_get(dev_id, cdev_id, > &cap); > > > > > + if (ret) { > > > > > + RTE_EDEV_LOG_ERR("Failed to get adapter caps edev > %" PRIu8 > > > > > + "cdev %" PRIu16, dev_id, cdev_id); > > > > > + return ret; > > > > > + } > > > > > + > > > > > + if (!(cap & > RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR)) > > > > > + return -ENOTSUP; > > > > Same here. > > > Ack, will add. > > > > > > > > + > > > > > + RTE_FUNC_PTR_OR_ERR_RET( > > > > > + *dev->dev_ops->crypto_adapter_vector_limits_get, > > > > > + -ENOTSUP); > > > > > + > > > > > + return dev->dev_ops->crypto_adapter_vector_limits_get( > > > > > + dev, cdev, limits); > > > > > +} > > > > > diff --git a/lib/eventdev/rte_event_crypto_adapter.h > > > > > b/lib/eventdev/rte_event_crypto_adapter.h > > > > > index d90a19e72c..7dd6171b9b 100644 > > > > > --- a/lib/eventdev/rte_event_crypto_adapter.h > > > > > +++ b/lib/eventdev/rte_event_crypto_adapter.h > > > > > @@ -253,6 +253,78 @@ struct rte_event_crypto_adapter_conf { > > > > > */ > > > > > }; > > > > > > > > > > +#define RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR 0x1 > > > > > +/**< This flag indicates that crypto operations processed on > > > > > +the crypto > > > > > + * adapter need to be vectorized > > > > > + * @see rte_event_crypto_adapter_queue_conf::flags > > > > > + */ > > > > > + > > > > > +/** > > > > > + * Adapter queue configuration structure */ struct > > > > > +rte_event_crypto_adapter_queue_conf { > > > > > + uint32_t flags; > > > > > + /**< Flags for handling crypto operations > > > > > + * @see RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR > > > > > + */ > > > > > + struct rte_event ev; > > > > > + /**< If HW supports cryptodev queue pair to event queue > binding, > > > > > + * application is expected to fill in event information. > > > > > + * @see > > > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND > > > > > + */ > > > > > + uint16_t vector_sz; > > > > > + /**< Indicates the maximum number for crypto operations to > > > > > +combine > > > > > and > > > > > + * form a vector. > > > > > + * @see rte_event_crypto_adapter_vector_limits::min_sz > > > > > + * @see rte_event_crypto_adapter_vector_limits::max_sz > > > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR > flag is > > > > > set in > > > > > + * @see > rte_event_crypto_adapter_queue_conf::rx_queue_flags > > > > rx_queue_flags?? > > > Typo, should be conf::flags. > > > > > > > > + */ > > > > > + uint64_t vector_timeout_ns; > > > > > + /**< > > > > > + * Indicates the maximum number of nanoseconds to wait for > > > > > aggregating > > > > > + * crypto operations. Should be within vectorization limits of > the > > > > > + * adapter > > > > > + * @see > rte_event_crypto_adapter_vector_limits::min_timeout_ns > > > > > + * @see > rte_event_crypto_adapter_vector_limits::max_timeout_ns > > > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR > flag is > > > > > set in > > > > > + * @see rte_event_crypto_adapter_queue_conf::flags > > > > > + */ > > > > > + struct rte_mempool *vector_mp; > > > > > + /**< Indicates the mempool that should be used for allocating > > > > > + * rte_event_vector container. > > > > > + * Should be created by using `rte_event_vector_pool_create`. > > > > > + * Valid when RTE_EVENT_CRYPTO_ADAPTER_EVENT_VECTOR > flag is > > > > > set in > > > > > + * @see rte_event_crypto_adapter_queue_conf::flags. > > > > > + */ > > > > > +}; > > > > > + > > > > > +/** > > > > > + * A structure used to retrieve event crypto adapter vector limi= ts. > > > > > + */ > > > > > +struct rte_event_crypto_adapter_vector_limits { > > > > > + uint16_t min_sz; > > > > > + /**< Minimum vector limit configurable. > > > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz > > > > > + */ > > > > > + uint16_t max_sz; > > > > > + /**< Maximum vector limit configurable. > > > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz > > > > > + */ > > > > > + uint8_t log2_sz; > > > > > + /**< True if the size configured should be in log2. > > > > > + * @see rte_event_crypto_adapter_queue_conf::vector_sz > > > > > + */ > > > > > + uint64_t min_timeout_ns; > > > > > + /**< Minimum vector timeout configurable. > > > > > + * @see > rte_event_crypto_adapter_queue_conf::vector_timeout_ns > > > > > + */ > > > > > + uint64_t max_timeout_ns; > > > > > + /**< Maximum vector timeout configurable. > > > > > + * @see > rte_event_crypto_adapter_queue_conf::vector_timeout_ns > > > > > + */ > > > > > +}; > > > > > + > > > > > /** > > > > > * Function type used for adapter configuration callback. The ca= llback is > > > > > * used to fill in members of the struct > > > > > rte_event_crypto_adapter_conf, this @@ -392,10 +464,9 @@ > > > > rte_event_crypto_adapter_free(uint8_t id); > > > > > * Cryptodev queue pair identifier. If queue_pair_id is set -1, > > > > > * adapter adds all the pre configured queue pairs to the insta= nce. > > > > > * > > > > > - * @param event > > > > > - * if HW supports cryptodev queue pair to event queue binding, > > > > > application is > > > > > - * expected to fill in event information, else it will be NULL. > > > > > - * @see > > > RTE_EVENT_CRYPTO_ADAPTER_CAP_INTERNAL_PORT_QP_EV_BIND > > > > > + * @param conf > > > > > + * Additional configuration structure of type > > > > > + * *rte_event_crypto_adapter_queue_conf* > > > > > * > > > > > * @return > > > > > * - 0: Success, queue pair added correctly. > > > > > @@ -405,7 +476,7 @@ int > > > > > rte_event_crypto_adapter_queue_pair_add(uint8_t id, > > > > > uint8_t cdev_id, > > > > > int32_t queue_pair_id, > > > > > - const struct rte_event *event); > > > > > + const struct > rte_event_crypto_adapter_queue_conf > > > > > *conf); > > > > > > > > > > /** > > > > > * Delete a queue pair from an event crypto adapter. > > > > > @@ -523,6 +594,26 @@ > > > > > rte_event_crypto_adapter_service_id_get(uint8_t > > > > > id, uint32_t *service_id); int > > > > > rte_event_crypto_adapter_event_port_get(uint8_t > > > > > id, uint8_t *event_port_id); > > > > > > > > > > +/** > > > > > + * Retrieve vector limits for a given event dev and crypto dev p= air. > > > > > + * @see rte_event_crypto_adapter_vector_limits > > > > > + * > > > > > + * @param dev_id > > > > > + * Event device identifier. > > > > > + * @param cdev_id > > > > > + * Crypto device identifier. > > > > > + * @param [out] limits > > > > > + * A pointer to rte_event_crypto_adapter_vector_limits > > > > > +structure that has to > > > > > + * be filled. > > > > Space missing before "be filled" > > > Ack > > > > > > > > + * > > > > > + * @return > > > > > + * - 0: Success. > > > > > + * - <0: Error code on failure. > > > > > + */ > > > > > +int rte_event_crypto_adapter_vector_limits_get( > > > > > + uint8_t dev_id, uint16_t cdev_id, > > > > > + struct rte_event_crypto_adapter_vector_limits *limits); > > > > > + > > > > > /** > > > > > * Enqueue a burst of crypto operations as event objects > > > > > supplied in > > > > > *rte_event* > > > > > * structure on an event crypto adapter designated by its event > > > > > *dev_id* through diff --git > > > > > a/lib/eventdev/rte_event_eth_rx_adapter.h > > > > > b/lib/eventdev/rte_event_eth_rx_adapter.h > > > > > index 3608a7b2cf..c8f2936866 100644 > > > > > --- a/lib/eventdev/rte_event_eth_rx_adapter.h > > > > > +++ b/lib/eventdev/rte_event_eth_rx_adapter.h > > > > > @@ -457,7 +457,8 @@ int rte_event_eth_rx_adapter_free(uint8_t id)= ; > > > > > * @see RTE_EVENT_ETH_RX_ADAPTER_CAP_MULTI_EVENTQ > > > > > * > > > > > * @param conf > > > > > - * Additional configuration structure of type > > > > > *rte_event_eth_rx_adapter_conf* > > > > > + * Additional configuration structure of type > > > > > + * *rte_event_eth_rx_adapter_queue_conf* > > > > These changes are not relevant. Please consider sending separate pa= tch. > > > > > > > Ack, Will send this change as a separate patch. > > > > > > > > * > > > > > * @return > > > > > * - 0: Success, Receive queue added correctly. > > > > > diff --git a/lib/eventdev/rte_eventdev.h > > > > > b/lib/eventdev/rte_eventdev.h index > > > > > 6a6f6ea4c1..1a737bf851 100644 > > > > > --- a/lib/eventdev/rte_eventdev.h > > > > > +++ b/lib/eventdev/rte_eventdev.h > > > > > @@ -1203,6 +1203,9 @@ struct rte_event_vector { #define > > > > > RTE_EVENT_TYPE_ETH_RX_ADAPTER_VECTOR > > > \ > > > > > (RTE_EVENT_TYPE_VECTOR | RTE_EVENT_TYPE_ETH_RX_ADAPTER) > > > /**< The > > > > > event vector generated from eth Rx adapter. */ > > > > > +#define RTE_EVENT_TYPE_CRYPTODEV_VECTOR > \ > > > > > + (RTE_EVENT_TYPE_VECTOR | RTE_EVENT_TYPE_CRYPTODEV) > /**< The > > > > > event > > > > > +vector generated from cryptodev adapter. */ > > > > > > > > > > #define RTE_EVENT_TYPE_MAX 0x10 > > > > > /**< Maximum number of event types */ @@ -1420,6 +1423,11 @@ > > > > > rte_event_timer_adapter_caps_get(uint8_t dev_id, uint32_t *caps); > > > > > * the private data information along with the crypto session. > > > > > */ > > > > > > > > > > +#define RTE_EVENT_CRYPTO_ADAPTER_CAP_EVENT_VECTOR 0x10 > > > > > +/**< Flag indicates HW is capable of aggregating processed > > > > > + * crypto operations into rte_event_vector. > > > > > + */ > > > > > + > > > > > /** > > > > > * Retrieve the event device's crypto adapter capabilities for t= he > > > > > * specified cryptodev device > > > > > -- > > > > > 2.25.1