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 6E921A034F; Mon, 11 Oct 2021 18:58:36 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 37CC8410EF; Mon, 11 Oct 2021 18:58:36 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 453F3410E2 for ; Mon, 11 Oct 2021 18:58:34 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.16.1.2/8.16.1.2) with SMTP id 19BEHaOW030538; Mon, 11 Oct 2021 09:58:28 -0700 Received: from nam12-dm6-obe.outbound.protection.outlook.com (mail-dm6nam12lp2169.outbound.protection.outlook.com [104.47.59.169]) by mx0b-0016f401.pphosted.com with ESMTP id 3bmpv4rm73-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 11 Oct 2021 09:58:28 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=HM/F0+iSD1IQYQMCWq3Ti98R7vDGacmVHq1oHVWJ5+CpKjzgxYRw06rXArDOXRV87OhDdMTx8L9LPU3Nz3+bvZ3n3/jNCd0IxU3AJHAHjZoxu+O4QLM3shgOr4eKH5vAl5beLDumYcmFniI/26zCtmyWNtY8Ij9gp01uYbzVEBVq/WhfCv3apqic3lx97TN9VYrUObqFAeQO/HKdfqE6i9FdLQp68CD/E6/ABX2BLfVRYW3EwGaOJsfK/xZjdX5hRb6DJVfeD3BjzoB/YV2DtgbFZHVOp7y3JWBoXPJDk0PduksL7M1CCNfKRIG+fz8NkmWe0nCjT/BQPZJGTarg/A== 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=sKJGC3OjJ1TESp5MXFttoHroQN5DOpfP3f5yvl4/pq0=; b=dV8Z3S/nzhSNup1X44Uyrv224tZkTo5HuVm+G3uZynVJiIrnd23Fr1QhdVJKxveaYxm1UVP4vfbGQrHgkxmYVuzSNYvpmF7QPLmPfCYE3fjDsjTY+XAYDmeSjmm+vmRE6+sBmdsO7xbr4Uo3UIHAju2Btw+BpcbdzLYM2WUGBBcEt10vHdN1tfCRaUb8JQ4KZGWsGpdk8RGf/BsZt6oMFj/CjWNZVmpN82OeSr3BWhPithhIk4guvI/T4i905P5sS6aO7bSe6UxYaFWD4IhhRaFyeVQyQyZoLwJlY5CzHqp/oLh5SRCkUBFfgeouGHI7x6+ERWT5J9ztziFDxc3dFg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=marvell.com; dmarc=pass action=none header.from=marvell.com; dkim=pass header.d=marvell.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.onmicrosoft.com; s=selector1-marvell-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=sKJGC3OjJ1TESp5MXFttoHroQN5DOpfP3f5yvl4/pq0=; b=JBi5bUYUkBO9QfA7VuSP6voXpACrCXERx8/g6VaTy1taMcryIVU7AbK+2FyDSynzQjTPpLSQMN88TbldL8ESYg8xbdL7Oa+JlSzR1P85+8OiR6PDKaoYPJHgBox3UZ/CBQ9FTkR2v6DDgY8Xtp3OCmVyvwqAqWlPsudvEY/6FkQ= Received: from CO6PR18MB4484.namprd18.prod.outlook.com (2603:10b6:5:359::9) by CO1PR18MB4651.namprd18.prod.outlook.com (2603:10b6:303:e6::15) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4587.22; Mon, 11 Oct 2021 16:58:25 +0000 Received: from CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::c41e:707:3f91:71b8]) by CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::c41e:707:3f91:71b8%8]) with mapi id 15.20.4587.026; Mon, 11 Oct 2021 16:58:25 +0000 From: Akhil Goyal To: Thomas Monjalon CC: "dev@dpdk.org" , "david.marchand@redhat.com" , "hemant.agrawal@nxp.com" , Anoob Joseph , "pablo.de.lara.guarch@intel.com" , "fiona.trahe@intel.com" , "declan.doherty@intel.com" , "matan@nvidia.com" , "g.singh@nxp.com" , "roy.fan.zhang@intel.com" , "jianjay.zhou@huawei.com" , "asomalap@amd.com" , "ruifeng.wang@arm.com" , "konstantin.ananyev@intel.com" , "radu.nicolau@intel.com" , "ajit.khaparde@broadcom.com" , Nagadheeraj Rottela , Ankur Dwivedi , "ciara.power@intel.com" , Stephen Hemminger , "ray.kinsella@intel.com" , "bruce.richardson@intel.com" Thread-Topic: [EXT] Re: [PATCH v2 3/3] security: add reserved bitfields Thread-Index: AQHXvIV3wHmAuXgUlUGBHcAT2EMzxKvNe94AgACJd9A= Date: Mon, 11 Oct 2021 16:58:24 +0000 Message-ID: References: <20210731181327.660296-1-gakhil@marvell.com> <20211008204516.3497060-1-gakhil@marvell.com> <20211008204516.3497060-3-gakhil@marvell.com> <2663373.9MILaYa0Np@thomas> In-Reply-To: <2663373.9MILaYa0Np@thomas> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: fa72fe08-8bf9-4c5c-32db-08d98cd85717 x-ms-traffictypediagnostic: CO1PR18MB4651: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: XCbiWNATm3UlpxqtKxH41eaYti1tP/YXhzqINiNMGYBn5rQo7C0Fr/DV0GFUbtdz27Jaox/2qUUuTbpch4UfiU9ZvpX1tvjDf5qm66mUExtcoAXl2mtoEy1nS06lFekvfcKyxyG4S7koLtzrNBcZJ+kXFKugCAcc1N8jd12eHjfB4zi7Y2lTEU/DQYphlnSallYYmg13agwDNnNHgsKBwjBBPmSr+8psbARc3dg5YbgdUx2bzZ0m3eJSPSCO/uj5+IutjOgoYhhxmDmIWRXoXsG8PFAiazVv7qTKJpbFhRoItjjVmJmZaba207vCSwQhcSGG87muym0wgWABBzihQ3hrssJCdAvXQZWOepM6wsD3JZwFNA9EVOKFlCQ61ZDAhwY6nJ1cpZTR2tYdttSzpLkTbpvM7n09jYmbY1m3x2Gt2+9Rrn+NH3NB2TLqJnFG+k9lO12WHpwVcTmWhMU+ToOM/htFWU1w+Z9lPIjh+U6+/ccVRIhTXMTvCxX+jDc/UpjCqekNDhghRpsxNjWR2Kp2CKnHgKRLeo9RD8D9lx4zqu+8yD5be7w2eL18O5tdEKCQOz8mXELo38zCgWormd6NdrTIePmmVnU9IA8TuNvBCQ8ibIRm9d8S3V/NmepCZsOUCkfPzSz+UZBp3F1mIzO5Q+I6Y4rtVCnSxN72wIZJcShEq4WMC/bSziaw/o61BFhv7Lx2NsDxeiPufSxQvw== x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CO6PR18MB4484.namprd18.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(186003)(7416002)(66476007)(55016002)(54906003)(64756008)(66556008)(9686003)(66946007)(26005)(15650500001)(6506007)(8936002)(8676002)(316002)(55236004)(5660300002)(7696005)(38070700005)(52536014)(83380400001)(38100700002)(6916009)(2906002)(122000001)(66446008)(33656002)(76116006)(71200400001)(4326008)(86362001)(508600001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?bEwv4zI601d9OaXRclzoxVOswqYaKzhCYTlv0juugAwK8tF/1gVu3kCbgkuQ?= =?us-ascii?Q?h5Hd+TZiRIJJ8scFggojpwoH9twI6hzSwEuzWm6ekISPVdoEquYHWMT1znqG?= =?us-ascii?Q?zX+/YZJNLIvJtgjvyc89DqXgzZCsqztZZ/wXjQR173lt0f5HvwrAekJaeuHt?= =?us-ascii?Q?DNpz0NoUzBDDd3VaWZGv3o9nDG7nSfiks/VB/lHdtHJJrPaW9n46QhHNPJAr?= =?us-ascii?Q?9rRHsd3wsbzcbAdQG5c/3+fQkXk3GSK4U2Ju7BiKf7tlwgmxrKw8rACNGSsL?= =?us-ascii?Q?ifRZig8PcTe/8pS5x0a1hOLUXEp4SSq1HU4OknqR5s72D6sDySccy5Ahbb52?= =?us-ascii?Q?Osc2zmS55PTcu/xd0lqwLN2moj1kZP2hb1oJpiEMAIk2RFIg2tw9swd4s4gI?= =?us-ascii?Q?h2lHC1BYIEkqJwHIewOJMzbsUwAOsGlkz95FCoRasFTXV/udQHPmtlF+6mU9?= =?us-ascii?Q?yf5uXL0SSnKC5Gjqny9DGuOrMIvxUlGQ2PkZEMpvEEIr9b8VuaGpiNj8aygM?= =?us-ascii?Q?vDq0D7ZWUQz5sGVODCi5RK7oa8Cx6/mpuvhVRj+k5VaFqftgI5oRP1daC3Zw?= =?us-ascii?Q?S5e9bVAp1b9Pl6dDfEsCX2mgMVfgFzWOXMIokUTq/oFh792anegnxe9CnC5p?= =?us-ascii?Q?XbNzOPwJ7nLBDOsYAeMMJR/Qsvr5IhBJkovQTTP3aZVK33UnOfiRY68yOk+r?= =?us-ascii?Q?1lO411eqiCFOOozlUQIqMsRgRkIGzeMLuX1gDsyfhpFco0ImqCy6tj/HIk4b?= =?us-ascii?Q?W4t/7dUkMCnO8XeZFvainVHVt8sLXe+mF7K6JS6G15kOPKlPVoWctoxliS2l?= =?us-ascii?Q?Aqpsr7Iz/LFJdxZYvZrxHDHQjwB0Jx4sSGY2Z+e7wmizoT1fs6DwzlUsOB4B?= =?us-ascii?Q?X1U/AsVE9js8rom40L/I4f2Kuy4yq2SzFUjqFIfl8VSeJnI6O8tWBLGt20qw?= =?us-ascii?Q?9Ffj2XkdmF92VCmOGoAmm877uOP6q7J3cp7q0kxUw9TooPAjyW+adv7r1RP7?= =?us-ascii?Q?xpnE4vrXpL9AwNv/zCCofFj5ImBgACXXfMxi3tf0Vgqs16Sz+7/R9Gs66SpE?= =?us-ascii?Q?Sm3kRAgmxRDpSw5q3W3cyZwp6ytAsxCp8qJlSuJhgCMYbS4trfXtfC4wWVkB?= =?us-ascii?Q?U9WiUrJSYZ+yzUQkYm2/MAoQCayxzTUzTilhquJjB0vInXZqwKTFYg+We2Y9?= =?us-ascii?Q?rWvrCieqY024vhjHYyib8IxQzAiVPtwpEdsC0ivAY71wMTh5AqzqlWZIC6UU?= =?us-ascii?Q?jM12g4qLRO+3xxTC6peIPmHNQpa145CrpFKbs+ROLr+k44ANVOAWbyxS0Fx7?= =?us-ascii?Q?3s/xciSSjFgd07kiv4XKO9jb?= Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 X-OriginatorOrg: marvell.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: CO6PR18MB4484.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: fa72fe08-8bf9-4c5c-32db-08d98cd85717 X-MS-Exchange-CrossTenant-originalarrivaltime: 11 Oct 2021 16:58:25.0015 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 70e1fb47-1155-421d-87fc-2e58f638b6e0 X-MS-Exchange-CrossTenant-mailboxtype: HOSTED X-MS-Exchange-CrossTenant-userprincipalname: +5EtP/ZlpeGdZILa2ZIiHBkcXQXY9HRZwP1KHekIrn/nDJIPtn2c9FfaM5SlVVn4sDgCrOkMoTVQ6+PMFexAXQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: CO1PR18MB4651 X-Proofpoint-ORIG-GUID: DN5AFR7FIoqQFfKaIyBCgS-YBJHTpbsI X-Proofpoint-GUID: DN5AFR7FIoqQFfKaIyBCgS-YBJHTpbsI X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.182.1,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.0.607.475 definitions=2021-10-11_06,2021-10-11_01,2020-04-07_01 Subject: Re: [dpdk-dev] [EXT] Re: [PATCH v2 3/3] security: add reserved bitfields 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" > 08/10/2021 22:45, Akhil Goyal: > > In struct rte_security_ipsec_sa_options, for every new option > > added, there is an ABI breakage, to avoid, a reserved_opts > > bitfield is added to for the remaining bits available in the > > structure. > > Now for every new sa option, these reserved_opts can be reduced > > and new option can be added. >=20 > How do you make sure this field is initialized to 0? >=20 Struct rte_security_ipsec_xform Is part of rte_security_capability as well As a configuration structure in session create. User, should ensure that if a device support that option(in capability), th= en only these options will take into effect or else it will be don't care for = the PMD. The initial values of capabilities are set by PMD statically based on the f= eatures that it support. So if someone sets a bit in reserved_opts, it will work only if PMD support= it And sets the corresponding field in capabilities. But yes, if a new field is added in future, and user sets the reserved_opts= by mistake And the PMD supports that feature as well, then that feature will be enable= d. This may or may not create issue depending on the feature which is enabled. Should I add a note in the comments to clarify that reserved_opts should be= set as 0 And future releases may change this without notice(But reserved in itself s= uggest that)? Adding an explicit check in session_create does not make sense to me. What do you suggest? Regards, Akhil