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 058BEA0524; Wed, 14 Apr 2021 09:59:04 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 979271617C6; Wed, 14 Apr 2021 09:59:04 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id 57B241617C3 for ; Wed, 14 Apr 2021 09:59:02 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 13E7tRCW005344; Wed, 14 Apr 2021 00:58:58 -0700 Received: from nam12-dm6-obe.outbound.protection.outlook.com (mail-dm6nam12lp2172.outbound.protection.outlook.com [104.47.59.172]) by mx0a-0016f401.pphosted.com with ESMTP id 37wn4ws6u6-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 14 Apr 2021 00:58:58 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=QZNXgLnNG4xhOo3+7mrJ6JVnmfCPZnNz7xpljyRGi88hvag64fyRjrhvR41A36jr7Su5rmuwVhHHusn2wdGK9tJc8DeJN1kIqOP7cEe26YkdX3CAD3Q2HYfQazzlq2Rxnm5duSA8XKeWmeRwuiMaacaWdbCVDOeBYJQms1/Wubc0S7QsyI7qqxaBl5IFrbFvyTVumxugfJ/FJEK2v9Db6iRjQg3XEEEtK2BP0oEdTDgcNCikjH26TICyXjbj8jPWkp7EjS2kVVI7Fhjh51Nt5frvER89h6LBK/fwkkPkkpC16Xs34RGsl/P5xZanJY/iE7+339fL1FCkqq2N0OwOdg== 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-SenderADCheck; bh=05mM+Szo/bEtp2iLWOBdjIVd0mD6R+lR0DiagYM2czQ=; b=jFUrN9vXg2llT2i5ybKfNoItW9uieFVFuaVYl7QVNtFI48ZvgGN+PtDP5kM8jCLcBI5SHFkrOs0Np0jZCrpiO13n1fC7PBjQtChT+k334rtQI7umt+XIwH+icJSHLsuc7MNf9MOtKP/21EM7M8CPV2sX3KPBkhq1UUY+J3aTN4bVuiZ2UkeqS44dlLtNDgSF/047pw77p6+XwSyDoRocICB2iz2eXKlAl+eRVFvBPXBo93c34cxfNF+s8Tfskwn6XiLeBLC6Xkj5vuipBHtlqkUkwNQtDj3r6mIsexHQ2OajyXtnKIY/SiI5+PWwZSGsRQYqPNaVXiFSwd1t29TLkA== 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=05mM+Szo/bEtp2iLWOBdjIVd0mD6R+lR0DiagYM2czQ=; b=Bc4XyQ3mMMqbRP4BZpbEgnomWo0w9YKU7y+oJsHTYBn8xp3gbFahn69DHUJH8VheChrk8c4s0mGobma+15zPPQiBno/8nw/XaW8w4njUKgdK3FvSfLIOSOqGy7B+zQ8bnOwPKS0OGG3dg0KnMTq+zIMKr6kvftG57gV8hJM57bs= Received: from MW2PR18MB2284.namprd18.prod.outlook.com (2603:10b6:907:10::16) by MWHPR1801MB1984.namprd18.prod.outlook.com (2603:10b6:301:6c::32) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.16; Wed, 14 Apr 2021 07:58:56 +0000 Received: from MW2PR18MB2284.namprd18.prod.outlook.com ([fe80::3168:cb00:6607:743f]) by MW2PR18MB2284.namprd18.prod.outlook.com ([fe80::3168:cb00:6607:743f%7]) with mapi id 15.20.4042.016; Wed, 14 Apr 2021 07:58:56 +0000 From: Akhil Goyal To: Jerin Jacob Kollanukkaran , "dev@dpdk.org" , Ray Kinsella , David Marchand , NBU-Contact-Thomas Monjalon CC: "abhinandan.gujjar@intel.com" , "hemant.agrawal@nxp.com" , "nipun.gupta@nxp.com" , "sachin.saxena@oss.nxp.com" , Anoob Joseph , "matan@nvidia.com" , "roy.fan.zhang@intel.com" , "g.singh@nxp.com" , "erik.g.carrillo@intel.com" , "jay.jayatheerthan@intel.com" , Pavan Nikhilesh Bhagavatula , "harry.van.haaren@intel.com" , Shijith Thotton Thread-Topic: [PATCH v8 1/3] eventdev: introduce crypto adapter enqueue API Thread-Index: AQHXME/wQthqHceUbkCNS9EClab02aqznsiAgAAGv6A= Date: Wed, 14 Apr 2021 07:58:55 +0000 Message-ID: References: <561328fb16b18404a30a2401c6bfb3a7f373a0f4.1618309291.git.sthotton@marvell.com> In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [182.69.47.6] x-ms-publictraffictype: Email x-ms-office365-filtering-correlation-id: 4bd89e95-587a-4509-2df2-08d8ff1b2755 x-ms-traffictypediagnostic: MWHPR1801MB1984: x-ms-exchange-transport-forked: True x-microsoft-antispam-prvs: x-ms-oob-tlc-oobclassifiers: OLM:9508; x-ms-exchange-senderadcheck: 1 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: uI821sbW1uFMky+NdjgXaxMluxFUrk63jThtuRRzhSue19u4dNvdtyzQLG165YfTIbnAVwK823iW/rOGyYtf5B2OwVtmyDO21fQA7Mv7dVxOaLMNLtat+FjnRGvyM/W9ilRvOa9eAnLvFli5xuBNBbDF6Ii/DyIAfsGmYAYths1Iye2QpFlBBP2W5EJfyTZoJt6+lJrH65HYQuYd2frvyvFoWG5JLyGrvRSjizyBnGrhQ75AuJ/N2ggKQJHMCZ1ebFzKfuDv1kPWjSSpaOORMSdz3UD+aBUTJmDpI+UWCe99cWkW61tu5w4TvT4u0jXKywusiq2Ebss5lIBJ0YQG2OOeBebP1ks4PZB2+M9Qa9Fs38h0SQfj16zseUYgncWqiSlNxVxS3iNN//vnPgaRhhSc3O17X47EMNeZnDxSJQ2Pb/mecTHoCpXsPCzRZMvELfptrOMA2fLBPErzY2vw4YoSOck0YJ9v925K1jBjPLRbFG0f48v0+w7MsWeVeuosGEpjtwFlcZbUpVJelukZefcY/RM0R62djC3XtqUbvIZidCQxl07n3uhGtXesr90+y7w9FfQlHR1kTI1x3hz6Q8t5dDUYUgPlBBPoX0zjmtY= x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:MW2PR18MB2284.namprd18.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(136003)(346002)(396003)(39850400004)(366004)(376002)(71200400001)(8936002)(122000001)(107886003)(5660300002)(7416002)(26005)(478600001)(66476007)(6506007)(33656002)(66556008)(52536014)(76116006)(66946007)(64756008)(66446008)(186003)(9686003)(54906003)(2906002)(86362001)(4326008)(55016002)(83380400001)(316002)(110136005)(7696005)(8676002)(38100700002); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata: =?us-ascii?Q?s4sClAby6D3G/Zpd4Fo+ydaWTephmI9z9G8rHpRITZkgBvt8JYFMd3CRjLh3?= =?us-ascii?Q?TV6i3p9IWm1GUb2j9gzVtkf2E4yCl7WGPOSlscCuU1YcGfTswd3N6XjMqzxH?= =?us-ascii?Q?G28+tRpOrznTQbMoYCjOAWrqpMP1JHc3jCsDX5nivX4HAnp8vJrCsBYGRviW?= =?us-ascii?Q?s2DAtPV42q0xPVBytsO87fiX/mZAKgV+PNwDuY6xzLBWFleDOJ2zzgdpX3Lk?= =?us-ascii?Q?yXtK3xmbxk2v9ltEM/k5veJA8AT9qS+roH/waoXdcfvUi01qHvSXEROEsKN5?= =?us-ascii?Q?wcguz96Vlje/bBZ+NdODLi55OY3cIhreYbBwi1vsfAMfOMtQ1zSg6M1noYPo?= =?us-ascii?Q?3Rf58orK0AMmjxsZoZqbPNnE8xMoUdr6t8iUYZ9u5On+8v9Qxnhy6a/x1qBd?= =?us-ascii?Q?hf4J5anHJE41P8mjEFNiBu9nOC5qYNvrTjZeb1Oq981mEPsB53lo3c6p2AfE?= =?us-ascii?Q?CNs/umFTYXNJ+ipYuugqBC5zbkvEsnA+I4YJbJ3VLsVO36gsRO9aKvfuOOlk?= =?us-ascii?Q?EeAB0mYEFSPddyAzsaWbC2qSVBakuJdrQp+htSQDWZPwyw4q+YBL8fBDZ/KE?= =?us-ascii?Q?JjCoRb8NIfkxxfedrJ/HDucvcFR8aLfTqSKTNyO53Z11GJoMgWvSsVlQ70li?= =?us-ascii?Q?mubjjx3lnU0eX6Itp+MYmSvyy8Ced4e7Lc4VyCFItNDeo4sdIWcePvPDtFLz?= =?us-ascii?Q?+KnlJfczFjRcgycS+vdXLsHnPqdo0DJbRbMC+WjjkdP9fMNby96wPiX4DyiS?= =?us-ascii?Q?lDq5KzUwevjJImaYeeV6bJazKlsTLSQDhY/q8/oywgGt5gooATVyQhVaQNzi?= =?us-ascii?Q?5fX3KcymxKQx2BZMADgEDKmszlMPoelaB3CenrAjZw5FBCGMngiAxiZwnK2Q?= =?us-ascii?Q?wCz3Wm2HMyZXUF9Zs1+HgUbyYCwzHqP6ujZTPXNulUo62n8zoJof481DV+T1?= =?us-ascii?Q?i5hqPhrL4jNGgrk8XMwdnoKsHsC8sR9QEblGmIuFLYHEE3FgqCRAHab8JfgJ?= =?us-ascii?Q?38K/l5+BcZhdh0sc8eGqG9wkcPAsYiKYLS3MarpaIsVdKYF9eVgXK3HReSJf?= =?us-ascii?Q?KkQ73HENUlljD3ZEruf6OACo0L/ahZJrsEYhA45VJdt8Dg3jQhan2wSqyvFR?= =?us-ascii?Q?FKXO+B8uJH8T8uZOsXktjpKMq7CcMLgQ8/jjPk6XINLIn5zK+ouDHhH8txGB?= =?us-ascii?Q?Lsbp9ddEBlAU+OhHw47oJuspnksVtqnrs+dI+brNjSjjHRuveh9snCZIewTm?= =?us-ascii?Q?9N1SezlIU/P8RFPubY6KZzg0OBKsJTNePI/pNlgaRhIN5NN3Mb4qD07wld7C?= =?us-ascii?Q?tO4=3D?= 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: MW2PR18MB2284.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4bd89e95-587a-4509-2df2-08d8ff1b2755 X-MS-Exchange-CrossTenant-originalarrivaltime: 14 Apr 2021 07:58:55.9979 (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: 13U9Ul43Vh3aTd7oMCL1M44yGtZkVYORmF9M6recoCbgFX8OaTi7M3WIdzFBbuB/EKi5o5ChuFppZVHhPsauBw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR1801MB1984 X-Proofpoint-ORIG-GUID: O0blN7VajCUiPxut9gFSD9qP4whKQT3q X-Proofpoint-GUID: O0blN7VajCUiPxut9gFSD9qP4whKQT3q X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.391, 18.0.761 definitions=2021-04-14_03:2021-04-14, 2021-04-14 signatures=0 Subject: Re: [dpdk-dev] [PATCH v8 1/3] eventdev: introduce crypto adapter enqueue API 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" Hi, > > + > > #define RTE_EVENTDEV_NAME_MAX_LEN (64) > > /**< @internal Max length of name of event PMD */ > > > > @@ -1423,6 +1427,8 @@ struct rte_eventdev { > > */ > > event_tx_adapter_enqueue txa_enqueue; > > /**< Pointer to PMD eth Tx adapter enqueue function. */ > > + event_crypto_adapter_enqueue ca_enqueue; > > + /**< Pointer to PMD crypto adapter enqueue function. */ > > struct rte_eventdev_data *data; > > /**< Pointer to device data */ > > struct rte_eventdev_ops *dev_ops; > > @@ -1435,7 +1441,7 @@ struct rte_eventdev { > > /**< Flag indicating the device is attached */ > > > > uint64_t reserved_64s[4]; /**< Reserved for future fields */ > > - void *reserved_ptrs[4]; /**< Reserved for future fields */ > > + void *reserved_ptrs[3]; /**< Reserved for future fields */ > > } __rte_cache_aligned; >=20 >=20 > This change has following ABI breakage[1]. >=20 > Could you move ca_enqueue at end of struct to avoid the ABI breakage. Als= o, > please update depreciation notice to move ca_enqueue to above(to align > with function pointers) in 21.11 release. >=20 > [1] > [C]'function rte_eventdev* rte_event_pmd_allocate(const char*, int)' at > rte_eventdev.c:1467:1 has some indirect sub-type changes: > return type changed: > in pointed to type 'struct rte_eventdev' at rte_eventdev.h:1411:1: > type size hasn't changed > 1 data member insertion: > 'event_crypto_adapter_enqueue rte_eventdev::ca_enqueue', at off= set > 512 (in bits) at rte_eventdev.h:1430:1 > 5 data member changes: > 'rte_eventdev_data* rte_eventdev::data' offset changed from 512 = to > 576 (in bits) (by +64 bits) > 'rte_eventdev_ops* rte_eventdev::dev_ops' offset changed from 57= 6 to > 640 (in bits) (by +64 bits) > 'rte_device* rte_eventdev::dev' offset changed from 640 to 704 (= in bits) > (by +64 bits) > 'uint64_t rte_eventdev::reserved_64s[4]' offset changed from 768= to > 832 (in bits) (by +64 bits) > type of 'void* rte_eventdev::reserved_ptrs[4]' changed: > type name changed from 'void*[4]' to 'void*[3]' > array type size changed from 256 to 192 > array type subrange 1 changed length from 4 to 3 > and offset changed from 1024 to 1088 (in bits) (by +64 bits) >=20 >=20 Yes my bad, it should be added in the end. But abi script will still shout for 'void*[4]' to 'void*[3]' conversion. We may need to add something in the devtools/libabigail.abignore So that, CI is not broken when reserved fields are changed. Otherwise, it does not make sense to introduce reserve fields. Can we have something generic for reserved fields? Any suggestions? Regards, Akhil