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 6396BA034F; Mon, 7 Feb 2022 15:51:01 +0100 (CET) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id F0AA1410F3; Mon, 7 Feb 2022 15:51:00 +0100 (CET) Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by mails.dpdk.org (Postfix) with ESMTP id 54E69410EA for ; Mon, 7 Feb 2022 15:50:59 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1644245459; x=1675781459; h=from:to:cc:subject:date:message-id:references: in-reply-to:content-transfer-encoding:mime-version; bh=ljMz0bfdBFI/GS2hyKEiXkaTuExwh7j4Q88jYGUuZRw=; b=UpdZHsnQcLpgsi/Hsif2Ob0llITMABydEgxy8jqileh4zXv/W1tjpIs5 qjcepu2gqKwIFJq27WgXhdVaa8T7MtIH88j1a/9+JhkTFHDtAtfqbsg/g cZjfBCMFNplyIRe74WhU3XvpTqxNR2RawH4OIE+xLRqDvlUMB8wp6+Meo ey+m9UMokyeJWpcjFlgU0xugQA0aH/0vcBlyuceVK6ih/m4dbzu45LpbP IuPpYL1mquMIpP1y8J38I4jDdB2QESIiKa//kFipcHIVqMaLIpcgh8f/r LpvgOFWrOcADJ2HlsY9bMDNSo8DlOCLrL10r+csLe/n6uVMU8u/UB2rrA A==; X-IronPort-AV: E=McAfee;i="6200,9189,10250"; a="309473527" X-IronPort-AV: E=Sophos;i="5.88,350,1635231600"; d="scan'208";a="309473527" Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga104.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 07 Feb 2022 06:50:54 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.88,350,1635231600"; d="scan'208";a="770653421" Received: from orsmsx605.amr.corp.intel.com ([10.22.229.18]) by fmsmga006.fm.intel.com with ESMTP; 07 Feb 2022 06:50:54 -0800 Received: from orsmsx603.amr.corp.intel.com (10.22.229.16) by ORSMSX605.amr.corp.intel.com (10.22.229.18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20; Mon, 7 Feb 2022 06:50:54 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2308.20 via Frontend Transport; Mon, 7 Feb 2022 06:50:53 -0800 Received: from NAM02-BN1-obe.outbound.protection.outlook.com (104.47.51.48) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2308.20; Mon, 7 Feb 2022 06:50:49 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=EuJAMTT62dD/YnucXsKZmIdv81sLCVHK29dOvancZNciTz/QQJ3Uo+/xbGggGeRPv+mzPWKfyNg8M27opYvvwcSObWvY5ycd4mND2IMkdKHzej/3/sohJknsHWqafqNQnKO9XrH9ZTzaS5qljzaPuezIOZAxnAy2BPXcE7BVz9UVtgXWopOpnDSw8Lfmo01ABuL8mxxxzooc5LPZb5JcH/TgE8zREfdmv9tEGC1W+K5AX24vLOasF94iK0q/1xBAg6Uo1VS2kOWzoSVIDc7xiLPubAUp4GXsdCboUCd2Ggo45N73FsDf4nKzrmMxApZBIf8geeWaNBw3VNXZzNStqQ== 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=xXZxdngK7dSfWKRz5MrYyGp+qwrVtEdKAdrhyFVM9A4=; b=dm6i4/hHsyxYfxCJypq3/gPU36J2e+7ISc5Q2cvlpK8mVkB6R3W4J0+/boQITdfTO8jHA5BxffK5je+KLM1IM9Kx/EQDV8T9xvZ92tb05thn0vqYKXkOKtiOfJ7piw0LnMVMgOXbrRxID4sE7ZXG6cm4jSWJ059SwLBmeXDlhzrF3IidaALezajRgkj2fH/Or0wTV18WU6HCim8+3cZEvoD3VuLi6xaMninRDf5B0l6afHdg2k4F9goN16MXPbmIe0UXTy4+6Qiw3ZtExe+TJeSU/+pLZ1rgHZnsW4mdugAGeBRTsJFwquC399oing71uUgk9DQTvcEtxQUmer9kMA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none Received: from MN2PR11MB3821.namprd11.prod.outlook.com (2603:10b6:208:f7::24) by CH0PR11MB5737.namprd11.prod.outlook.com (2603:10b6:610:112::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4951.12; Mon, 7 Feb 2022 14:50:48 +0000 Received: from MN2PR11MB3821.namprd11.prod.outlook.com ([fe80::4fd:a9c7:1eda:c93d]) by MN2PR11MB3821.namprd11.prod.outlook.com ([fe80::4fd:a9c7:1eda:c93d%6]) with mapi id 15.20.4951.019; Mon, 7 Feb 2022 14:50:48 +0000 From: "Power, Ciara" To: Akhil Goyal , "dev@dpdk.org" , "thomas@monjalon.net" CC: "Zhang, Roy Fan" , Anoob Joseph , "mdr@ashroe.eu" , "Doherty, Declan" Subject: RE: [EXT] [PATCH v3 4/4] crypto: modify return value for asym session create Thread-Topic: [EXT] [PATCH v3 4/4] crypto: modify return value for asym session create Thread-Index: AQHYGRfzXT73UQPhxUe0CwdFIggJTqyH0T4AgABZDoA= Date: Mon, 7 Feb 2022 14:50:47 +0000 Message-ID: References: <20220203160449.1638311-1-ciara.power@intel.com> <20220203160449.1638311-5-ciara.power@intel.com> In-Reply-To: Accept-Language: en-IE, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: dlp-version: 11.6.200.16 dlp-product: dlpe-windows dlp-reaction: no-action authentication-results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: c24238c1-bddd-4c2f-da48-08d9ea493a3d x-ms-traffictypediagnostic: CH0PR11MB5737:EE_ 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: pKxdVbHUX/fZ+0N0W/hmnQSshij16UFAUhakvszAHnZuULesOCkxogTYzJ7g3UkJGbfpFCg3LK5V5YodQYejvYDN29EyJ4YoqAHZTgoRaBbwUv9KMBcoipvdgY8tgUW/DgbXEluhUYwYrx0DjEf4RQA2j2rcmJVJCeBPwFL/YK6o4kvfujNjeJAIX9pUijtAE/GAyGoDZGx6EigXBLzwtF6iPW2WPAGdSvT0je3Phtv+B5GTBedZriWelI7diZVPGwMoUozXHbCvePBGP1I6n8il9ATyCsVK9VJJ576yAwYZj/FgNvNyG1IS/hpJPqZHibj1Q+OEtdi5ksUTBnwWFa0nUUt66GDQOetZqNJ73z2kGHX84zwm7DQSi3jKrugnq5VSuXjG+fSRg0XmufOAqsCTYFTz8zJOIHFvwAv2KcVKfT4981eQO2qXdTdyeFlC9JyYp2sJJC2FMVl6/DlNRAdkQAFs77tXXqUe0pU3MkhKhRc7vaeWCe6v9KEOI2YxxgImC8nKuDZIp0Zm2b3SBOyRgnk941SHS8EWEa26fJmTkkw+qVXjc/qXNyGUBAv3H3t0dtHofu7+SPkU5io3WVr76nOu5JqRXp+Oo7Ku5MjAMDeEJJbueZpTxjuTeJbS6eV9lmuYPS4Zw9+WPodJ5Vca7OV5Aq6fiYeiJNk7OEAAVvfgcDvZlInDE1myDCDBiHRoIDzXzHRKiKGL8u9qcQ== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MN2PR11MB3821.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230001)(366004)(186003)(5660300002)(26005)(107886003)(38070700005)(2906002)(508600001)(82960400001)(38100700002)(122000001)(66476007)(86362001)(76116006)(33656002)(71200400001)(316002)(52536014)(54906003)(6506007)(7696005)(110136005)(9686003)(55016003)(8676002)(64756008)(4326008)(8936002)(66946007)(66556008)(66446008)(83380400001); DIR:OUT; SFP:1102; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?t1GHzqGC0wWDRlRPUmxy2WaQb/4FkhCS6SJN7zn/Nmhzo/ogYXuqtiQbVbhL?= =?us-ascii?Q?qQyfW2S+WtyZdXR9VXg/SlolkCiObcp68gCHiKMZ7d/OvIcEzw9oZ73KaJML?= =?us-ascii?Q?G2lNDZ4Mn/m/dJ0i61RnRHaxc3AFqi98To9yG/Xw5L5k8TWBEULcvJmLMja6?= =?us-ascii?Q?Phdwgp4S2pBYbZftybC1Sb0NMACGSfIl64/hEyzLDFk/gujQfqnNa/JOuP/9?= =?us-ascii?Q?3imADG1X6pdbOG4ZoDeGEdinXJvMMZ57fzjjaydus/Po1YAbjfINCgFWLyOw?= =?us-ascii?Q?RaXNxdYFirt7U4eUts9940cEL2qPUyzfVXscsTXNHTnaBnz1nvs4IRQqPHHx?= =?us-ascii?Q?UkhrD64tgZ63/IOdPFWaC3zyyhSy5BxGpK0xVaw4eXwpC7Jqwi0DWJ/7ZW7O?= =?us-ascii?Q?Df+2RPdB+MI9cDHGRTjr02gK4luNujgzyRxMFCGGTk60IV/WvehvVbXMg+cE?= =?us-ascii?Q?vjYLgr6nB2WI4yDaTknFcp+PO1TWtyffFcKmBoxWFc2DZPOrHLDiqyGzdOVF?= =?us-ascii?Q?beLwDOTQ+ZKYKyFo5lnY+lbLQsJ3pIJnZy92UXXDk9YjnmneuB5KfYcQqOLs?= =?us-ascii?Q?BhIm/w7/QWM4JYpz3JTPHYvj8bKQ89gycpnAawgpjJyDCjSNwMT4zUTGMSMM?= =?us-ascii?Q?g9w9Rq+tZAWL0MeHJ4bYaq8jXYYi+HrxtIZ7UFKCzCBRIfUTxkmiN8gSaDFg?= =?us-ascii?Q?n3lnoJAGjJ5b0T5Jd4KQTAo9Gd1M/jRj76e1EmTfk4sCwcORWZKaxFt06XyY?= =?us-ascii?Q?8UGoeshyTOQpuV51xVLE3hKu4kkSCSCF8B24lEFGYcmmVQkxMYQSTme3KSyw?= =?us-ascii?Q?JHU9RX/sWAYAOnjg9K5qp+Nrg2J9lyuR3eZAseAv4lbWT2TDGclpiZ0MDYyC?= =?us-ascii?Q?Gab7S6XeOrzd4XSx/Rm5jrkKSZVL9ziZZfnCvFQZwGRy7VnIHK2vd2lni1Qm?= =?us-ascii?Q?eCDdFoJM7VVja4h6NpOk750yYEsypwcMZht00YxKrumWZlh5nHTQtCUznfNF?= =?us-ascii?Q?HKXigkbRCPnIjHd5lytTnpqaaxMGAbC0uKq97imuTBtNr3nm/39mdm/LkKB/?= =?us-ascii?Q?YKqgGdwH88xU4HrLkG/FECFHwvsyE46Pso6bG+IRo02cXwWFBcyUC3NhG63P?= =?us-ascii?Q?vhDOxTrqJZmY/zxbWOHsWiuj5cR6CdGFqNpCUDfstpBt+nX7xMZcGxVBEOx+?= =?us-ascii?Q?+bPYlETRjq/453LeFYrmYrLZwoIjF0Euy3WUt3WfubiWuQT4TeMiKcxhKbx1?= =?us-ascii?Q?sakxWzxYzsfIlSUTuu3yhaH7wnOTSu6iOwR7+3wn+78CgFzQAAMOKjArZ99G?= =?us-ascii?Q?WT/CjXOUA45C9di76gh6wmlnDQtwRIMG3zF2JG/zrtVmujIyUjCqo8FpdVjF?= =?us-ascii?Q?CPzVQNFypE+Yx90PwF3bjJcQqHMSCTn0JpQFtCQUSp3LIzCZCrE0PBQJDzZU?= =?us-ascii?Q?AXeHXc/CJdstwnEgNEionAPKI0oxcauG68h/tOJqyz7GeTrXgG4+tD4CvnpB?= =?us-ascii?Q?z0BZoxsuJqWMlDsLuI2HZo1rO+Qa4TkjEXW3AdJp+n//Q2aNkQso8XNxoLbA?= =?us-ascii?Q?PfdeoobxdE+WRcFtKX9yInPrtGJ9ywH+y+33p7CSowHoe6mAV6F3MWBUfLtw?= =?us-ascii?Q?OC85nxJQFSakIH4BbSp2ssM=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: MN2PR11MB3821.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: c24238c1-bddd-4c2f-da48-08d9ea493a3d X-MS-Exchange-CrossTenant-originalarrivaltime: 07 Feb 2022 14:50:47.9498 (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: rtV9UzTDP6ouCG6GKWb7tn0wfEWiVT7CY9UZlGqH2tfTQRE3/Tf7iZL8jBkNaOz+8D5PbVdtQuCvg5oZsIu8dg== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH0PR11MB5737 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 Hi Akhil, Some replies inline. Thanks, Ciara >-----Original Message----- >From: Akhil Goyal >Sent: Monday 7 February 2022 09:05 >To: Power, Ciara ; dev@dpdk.org; >thomas@monjalon.net >Cc: Zhang, Roy Fan ; Anoob Joseph >; mdr@ashroe.eu; Doherty, Declan > >Subject: RE: [EXT] [PATCH v3 4/4] crypto: modify return value for asym ses= sion >create > >> diff --git a/doc/guides/prog_guide/cryptodev_lib.rst >> b/doc/guides/prog_guide/cryptodev_lib.rst >> index 62bd3577f5..8e16461dc6 100644 >> --- a/doc/guides/prog_guide/cryptodev_lib.rst >> +++ b/doc/guides/prog_guide/cryptodev_lib.rst >> @@ -1236,10 +1236,10 @@ crypto operations is similar except change to >> respective op and xform setup). >> * Create asym crypto session and initialize it for the crypto devi= ce. >> * The session structure is hidden from the app, so void * is used. >> */ >> - void *asym_session; >> - asym_session =3D >> rte_cryptodev_asym_session_create(asym_session_pool, >> + void *asym_session =3D NULL; >> + ret =3D rte_cryptodev_asym_session_create(&asym_session, >> asym_session_pool, >> cdev_id, &modex_xform); >> - if (asym_session =3D=3D NULL) >> + if (ret < 0) >> rte_exit(EXIT_FAILURE, "Session could not be created\n"); > >Sample Code in the rst files is no more added. @Thomas: Could you please >confirm? >Probably a separate patch is required to clean this up. > [CP] I see Thomas replied on this - thanks. Will try find a section of exam= ple/test code that does the same thing as being shown here. >> diff --git a/lib/cryptodev/rte_cryptodev.c >> b/lib/cryptodev/rte_cryptodev.c index 0d816ed4a9..005f0e7952 100644 >> --- a/lib/cryptodev/rte_cryptodev.c >> +++ b/lib/cryptodev/rte_cryptodev.c >> @@ -1912,9 +1912,9 @@ rte_cryptodev_sym_session_create(struct >> rte_mempool *mp) >> return sess; >> } >> >> -void * >> -rte_cryptodev_asym_session_create(struct rte_mempool *mp, uint8_t >> dev_id, >> - struct rte_crypto_asym_xform *xforms) >> +int >> +rte_cryptodev_asym_session_create(void **session, struct rte_mempool >> *mp, >> + uint8_t dev_id, struct rte_crypto_asym_xform *xforms) > >Do you really need a double pointer for the session handle? > [CP] Yes I believe so, the return value used to be session, but now that we= have an int return value, the session needs to be passed in as a parameter= somehow. We need the double pointer because we need the call to rte_mempool_get() to= set the original session pointer that can be accessed outside of this func= tion, rather than just the local copy if it were a singular session pointer passe= d in as a parameter.