From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 658ACC77B7F for ; Tue, 16 May 2023 15:31:09 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 9B34C406B6; Tue, 16 May 2023 17:31:08 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by mails.dpdk.org (Postfix) with ESMTP id E449240689 for ; Tue, 16 May 2023 17:31:06 +0200 (CEST) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 34G7Di0a021663; Tue, 16 May 2023 08:31:04 -0700 Received: from nam12-dm6-obe.outbound.protection.outlook.com (mail-dm6nam12lp2170.outbound.protection.outlook.com [104.47.59.170]) by mx0a-0016f401.pphosted.com (PPS) with ESMTPS id 3qkvbmm1cf-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 16 May 2023 08:31:03 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=PbVbPqjl/OtvlzPsvMjIZ4BecyD5MgVQ/yGU//KIWefqkkG1S9+hJKQ+ldn4PjxHAE44FKF8a5lDM5CDfMNn1Z0prnRvTjF0gcgaoAvbXV2YIMWXGpFAkMkpEfnX/gjpgdPzqLmtXQ6y7hctncuLA5LvOQSsPGj+t+9oEG9GAPSKcRj/9ZqAVcNgiId9vTM4LHZ4QdLAlR9ThgHBTIjA7yPS9Y2cazW4Gz9tsDZ6RRGx4sLwDhbJdK35EMALsX4Hj7gCGb/vrTKmDJJv/CraO7KBUCkagqRdvCsYsvS5vWhRtXzZUHVP8PDy+qFjIBA+9iQviyo+VrqOpsvWAqgnTQ== 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=5B/+emrB6w/lgfRUdVrsFR2o5ABtalEuXm8RUl/zEVw=; b=APxboBqfqgZnDyO0Yspge5sHJKrQCgY8feZS8DiE5LLxb5Z01hFbQiYv3vXAK0cjmMie1+vNn0fYAaamLm1L902r1UwBCryIJ+TnCy91EcBj3GrKysdSzjvCbSD7OuDmpWgvf2pIY1DLuKomprrFQPUSxM1mZV1p0LGOsj+LniAm+i5UoPfOgrtbEyef/xUU6UNqBJyGW+qVPJ7potpsOx7rSO7r5YMgxzEKnewsNqaT0ux5mxEysYgznGcJIcXxB1hXzXbsnVXKYTxnIspPkzqr/mEuA/c42DPGkidZqK7ftJpqDR0rUezD9fMCE01FWE4YKyHNmdW07e6G81M5Ew== 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=5B/+emrB6w/lgfRUdVrsFR2o5ABtalEuXm8RUl/zEVw=; b=gAuKpZOH9DKR6wYWkK0Q8fuc4IYd7GPIxmiaSlWkFV/hJnR+n5i3fybUwPvFFBIQdhtJ/6UE0ZGWMIOt6fL6bzxdrg7Tk2W4J9DqTjVfUZO8taaoL0iE4mNqtriUCeJAr8ioRcFRwJkMStlOCxYOcw5cpvTmUpdnsxU5SmY20Vs= Received: from CO6PR18MB4484.namprd18.prod.outlook.com (2603:10b6:5:359::9) by MW2PR18MB2089.namprd18.prod.outlook.com (2603:10b6:907:10::11) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6387.30; Tue, 16 May 2023 15:30:59 +0000 Received: from CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::831b:85d2:9a45:8489]) by CO6PR18MB4484.namprd18.prod.outlook.com ([fe80::831b:85d2:9a45:8489%6]) with mapi id 15.20.6387.033; Tue, 16 May 2023 15:30:57 +0000 From: Akhil Goyal To: Anoob Joseph , Thomas Monjalon , Jerin Jacob Kollanukkaran , Konstantin Ananyev , Bernard Iremonger CC: Hemant Agrawal , =?iso-8859-1?Q?Mattias_R=F6nnblom?= , Kiran Kumar Kokkilagadda , Volodymyr Fialko , "dev@dpdk.org" , Olivier Matz Subject: RE: [PATCH v2 02/22] lib: add pdcp protocol Thread-Topic: [PATCH v2 02/22] lib: add pdcp protocol Thread-Index: AQHZbvjvvmj4ZOLr0kOu2Nfgh9ofla9dI8uQ Date: Tue, 16 May 2023 15:30:57 +0000 Message-ID: References: <20221222092522.1628-1-anoobj@marvell.com> <20230414174512.642-1-anoobj@marvell.com> <20230414174512.642-3-anoobj@marvell.com> In-Reply-To: <20230414174512.642-3-anoobj@marvell.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-dg-rorf: true x-dg-ref: =?iso-8859-1?Q?PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNcZ2FraGlsXG?= =?iso-8859-1?Q?FwcGRhdGFccm9hbWluZ1wwOWQ4NDliNi0zMmQzLTRhNDAtODVlZS02Yjg0?= =?iso-8859-1?Q?YmEyOWUzNWJcbXNnc1xtc2ctYTU0OGM2YTUtZjNmZS0xMWVkLWEwOTMtOT?= =?iso-8859-1?Q?RlNmY3NjYxOTA3XGFtZS10ZXN0XGE1NDhjNmE2LWYzZmUtMTFlZC1hMDkz?= =?iso-8859-1?Q?LTk0ZTZmNzY2MTkwN2JvZHkudHh0IiBzej0iMjM1NTIiIHQ9IjEzMzI4Nz?= =?iso-8859-1?Q?I0NjU1NDE1NTkyNSIgaD0iWU5OWEpUb1o5eFowQnJVVzU1TWdrQUhOcjdz?= =?iso-8859-1?Q?PSIgaWQ9IiIgYmw9IjAiIGJvPSIxIiBjaT0iY0FBQUFFUkhVMVJTUlVGTk?= =?iso-8859-1?Q?NnVUFBTWdNQUFDVlMvTm5DNGpaQWQrVWxJekJiREtpMzVTVWpNRnNNcUlV?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUhBQUFBQllEQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUVBQVFFQkFBQUE0K1V0REFDQUFRQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFKNEFBQUJoQUdRQVpBQnlBR1VBY3dCekFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFFQUFBQUFBQUFBQUFBQUFBRUFBQUFBQUFBQUFnQUFBQUFBbmdBQU?= =?iso-8859-1?Q?FHTUFkUUJ6QUhRQWJ3QnRBRjhBY0FCbEFISUFjd0J2QUc0QUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFBQUFBQ2VBQUFBWXdCMUFI?= =?iso-8859-1?Q?TUFkQUJ2QUcwQVh3QndBR2dBYndCdUFHVUFiZ0IxQUcwQVlnQmxBSElBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBSkFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUJBQUFBQUFBQUFBSUFBQUFBQUo0QUFBQmpBSFVBY3dCMEFHOE?= =?iso-8859-1?Q?FiUUJmQUhNQWN3QnVBRjhBWkFCaEFITUFhQUJmQUhZQU1BQXlBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-refone: =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFFQUFBQUFBQUFBQWdBQUFBQUFuZ0?= =?iso-8859-1?Q?FBQUdNQWRRQnpBSFFBYndCdEFGOEFjd0J6QUc0QVh3QnJBR1VBZVFCM0FH?= =?iso-8859-1?Q?OEFjZ0JrQUhNQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFBQUNBQUFBQUFDZUFBQUFZd0Ix?= =?iso-8859-1?Q?QUhNQWRBQnZBRzBBWHdCekFITUFiZ0JmQUc0QWJ3QmtBR1VBYkFCcEFHME?= =?iso-8859-1?Q?FhUUIwQUdVQWNnQmZBSFlBTUFBeUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQkFBQUFBQUFBQUFJQUFBQUFBSjRBQUFCakFIVUFjd0IwQU?= =?iso-8859-1?Q?c4QWJRQmZBSE1BY3dCdUFGOEFjd0J3QUdFQVl3QmxBRjhBZGdBd0FESUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUVBQUFBQUFBQUFBZ0FBQUFBQW5nQUFBR1FBYkFCd0FGOEFjd0JyQUhr?= =?iso-8859-1?Q?QWNBQmxBRjhBWXdCb0FHRUFkQUJmQUcwQVpRQnpBSE1BWVFCbkFHVUFYd0?= =?iso-8859-1?Q?IyQURBQU1nQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFRQUFB?= =?iso-8859-1?Q?QUFBQUFBQ0FBQUFBQUNlQUFBQVpBQnNBSEFBWHdCekFHd0FZUUJqQUdzQV?= =?iso-8859-1?Q?h3QmpBR2dBWVFCMEFGOEFiUUJsQUhNQWN3QmhBR2NBWlFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-reftwo: =?iso-8859-1?Q?QUFBQUFBQUFBQUFCQUFBQUFBQUFBQUlBQUFBQUFKNEFBQUJrQUd3QWNBQm?= =?iso-8859-1?Q?ZBSFFBWlFCaEFHMEFjd0JmQUc4QWJnQmxBR1FBY2dCcEFIWUFaUUJmQUdZ?= =?iso-8859-1?Q?QWFRQnNBR1VBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBRUFBQUFBQUFBQUFnQUFBQUFBbmdBQUFHVUFiUUJoQUdrQWJBQmZB?= =?iso-8859-1?Q?R0VBWkFCa0FISUFaUUJ6QUhNQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBd0FBQUFBQUFBQUFBQUFBQVFB?= =?iso-8859-1?Q?QUFBQUFBQUFDQUFBQUFBQ2VBQUFBYlFCaEFISUFkZ0JsQUd3QVh3QndBSE?= =?iso-8859-1?Q?lBYndCcUFHVUFZd0IwQUY4QWJnQmhBRzBBWlFCekFGOEFjZ0JsQUhNQWRB?= =?iso-8859-1?Q?QnlBR2tBWXdCMEFHVUFaQUJmQUdFQWJBQnZBRzRBWlFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUJBQUFBQUFBQU?= =?iso-8859-1?Q?FBSUFBQUFBQUo0QUFBQnRBR0VBY2dCMkFHVUFiQUJmQUhBQWNnQnZBR29B?= =?iso-8859-1?Q?WlFCakFIUUFYd0J1QUdFQWJRQmxBSE1BWHdCeUFHVUFjd0IwQUhJQWFRQm?= =?iso-8859-1?Q?pBSFFBWlFCa0FGOEFhQUJsQUhnQVl3QnZBR1FBWlFCekFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFFQUFBQUFBQUFBQWdBQUFB?= =?iso-8859-1?Q?QUFuZ0FBQUcwQVlRQnlBSFlBWlFCc0FHd0FYd0JoQUhJQWJRQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFBQUNBQUFBQUFDZUFBQU?= =?iso-8859-1?Q?FiUUJoQUhJQWRnQmxBR3dBYkFCZkFIQUFjZ0J2QUdvQVpRQmpBSFFBWHdC?= =?iso-8859-1?Q?akFHOEFaQUJsQUhNQUFBQUFBQUFB?= x-dg-refthree: =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQkFBQUFBQU?= =?iso-8859-1?Q?FBQUFJQUFBQUFBSjRBQUFCdEFHRUFjZ0IyQUdVQWJBQnNBRjhBY0FCeUFH?= =?iso-8859-1?Q?OEFhZ0JsQUdNQWRBQmZBR01BYndCa0FHVUFjd0JmQUdRQWFRQmpBSFFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUVBQUFBQUFBQUFBZ0FB?= =?iso-8859-1?Q?QUFBQW5nQUFBRzBBWVFCeUFIWUFaUUJzQUd3QVh3QndBSElBYndCcUFHVU?= =?iso-8859-1?Q?FZd0IwQUY4QWJnQmhBRzBBWlFCekFGOEFjZ0JsQUhNQWRBQnlBR2tBWXdC?= =?iso-8859-1?Q?MEFHVUFaQUJmQUcwQVlRQnlBSFlBWlFCc0FHd0FBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFRQUFBQUFBQUFBQ0FBQUFBQUNlQU?= =?iso-8859-1?Q?FBQWJRQmhBSElBZGdCbEFHd0FiQUJmQUhBQWNnQnZBR29BWlFCakFIUUFY?= =?iso-8859-1?Q?d0J1QUdFQWJRQmxBSE1BWHdCeUFHVUFjd0IwQUhJQWFRQmpBSFFBWlFCa0?= =?iso-8859-1?Q?FGOEFiUUJoQUhJQWRnQmxBR3dBYkFCZkFHOEFjZ0JmQUdFQWNnQnRBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFCQUFBQUFBQUFBQUlBQUFBQUFKNEFBQUJ0QUdF?= =?iso-8859-1?Q?QWNnQjJBR1VBYkFCc0FGOEFkQUJsQUhJQWJRQnBBRzRBZFFCekFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBRUFBQUFBQUFBQUFnQUFBQUFBbmdBQUFHMEFZUUJ5QUhZQV?= =?iso-8859-1?Q?pRQnNBR3dBWHdCM0FHOEFjZ0JrQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQU?= =?iso-8859-1?Q?FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?iso-8859-1?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-reffour: QUFBQUFBQUFDQUFBQUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFBQUFBQT0iLz48L21ldGE+ x-ms-publictraffictype: Email x-ms-traffictypediagnostic: CO6PR18MB4484:EE_|MW2PR18MB2089:EE_ x-ms-office365-filtering-correlation-id: d3f8ab0f-2194-4ac7-6758-08db56228bc1 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: znW+uVKdptF07YYeBLjxIyDsC7gZTjW91kXSDyzAYqOyABCI71fXA5TPCPdh7CoTVXTiQxGBRkhwehNVqoMgCW0zXEhM497VfZ1nCjgtglq6faAFQx+hZ7hnd2yUHGZSiiGHL+tAqj0NmRA7S8QiPLUSPYEQQMP/YHg31WjGPpoPSy6qorq7NIcxJz1gQ5X4arqqN5D0zo8uiI1/lopIHXjTXc7i0rfzgNeAAW1VnFOiyvH601xxKcrFaPTvr4MhDfroKnPcd3fNwriCtkdLu6gcOI72o/QXMXTZUKJF1qoOj+FhLAYtUIvkKai9eArrAtXt47ziE375SEdEQIOYopSP0dY1c2HxTJDR3SJtQJzW0NpCbS23X40SDRVSw115WVzev5dAXjD4hGQ9VsONM1mAaNehQ0jypMra3LadtR2BUV3ti8rYQYjOpFrZf+C2Rz6Up5u7G31Lq4H7nePAbdjf3Fb0aW5wEoTk9kSTj90q0HgkP3uf9lPudIomN/b3hCny2Dz1jW1orMbBA2pPRWe7W8EEdHfd2IQzjlwkcUqw+o/zRzXMv1bhV1X35VwwqN819ZAoMJZqdd/lrlWDvaKlJU+8bAvwzKRghD6UfysPXDK/qge4XwusRMwMZge/ 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:(13230028)(4636009)(396003)(39860400002)(346002)(376002)(366004)(136003)(451199021)(66476007)(66556008)(110136005)(64756008)(4326008)(66946007)(76116006)(478600001)(66446008)(54906003)(6506007)(186003)(38100700002)(38070700005)(55236004)(83380400001)(9686003)(8676002)(8936002)(41300700001)(2906002)(55016003)(71200400001)(7696005)(316002)(33656002)(86362001)(122000001)(5660300002)(30864003)(52536014)(26005)(579004); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?nPd0eLZMEfG4/p8WeMfzUx7m/1xW/TZssqf3gi5/j3yfBGW3q04t8Ku5Xo?= =?iso-8859-1?Q?p4JGYWYxy4AlLLzrwwkBNc/HF52BsjRqTdhfa4qlLf07Xz9Ss89UPV4sw9?= =?iso-8859-1?Q?UflHh83n/n/wOf7cRf4T9IgLZ9I7ftBWSnB5hLSxoijvoepe/ZMMSIFSw9?= =?iso-8859-1?Q?FgYOv06VP4vyD5RRX3JPgf+2sqjJRvSXECnRH3aCpnDXwJbNOq+DY5ZQ3u?= =?iso-8859-1?Q?dyFy9PlU7pXTR5TsOfnZjjTtyPN7Di40evxEP0W33PtTsGwrtfuHj8QKH9?= =?iso-8859-1?Q?b/vwOqdeI2tOYijc9r/inJoZ/UtR3zHcsx1bvXxsTOFch5gcitBA7hxpPX?= =?iso-8859-1?Q?ZvItaXnj4ggWz+SQo+48Zb20fyUEJiGjilptXP97h9nNe9I0378eAHHvX1?= =?iso-8859-1?Q?D2BvaalUCWKBPfGhWk5O0yk2XJdAYb0m+/p0iYWV3yCXS399wYsV01LWls?= =?iso-8859-1?Q?4dNPmNH8miQ6qB5GncHBhLkd5AtD2r/yVx/SOAELUBODIRx6E1DpbMlt4v?= =?iso-8859-1?Q?VpwWJ5zd3dMY2ZEC+CywmDztc5yn5WE8xCSftHzA1vhvbszPBJiaAZfmir?= =?iso-8859-1?Q?Yf7JXd8PcLIMXtHCYXhgG2/Z8JzZ9sDwAGX7OWshVe2SC1ZdedBo+9qtpu?= =?iso-8859-1?Q?6ZYAv5cxTrpBlVS7MSmkDaGX+w7PXrodgXOe3tu9NsaizNJLs8ZjENcO6n?= =?iso-8859-1?Q?dP+hqo3Iy86BrjmSj/Q8HczHwQEimxcL30/XBE72jyn3YQVTQvlxj+e7Nl?= =?iso-8859-1?Q?CokYfzZw6taFwQrtRuo2djajHfGZGrF9prlwOm6qGtkKjdBk29jWlANPBx?= =?iso-8859-1?Q?0ioPtg/uSIlHWqfQwzApPL9hUyWSmqPRpm+52S4amPWCzd5JudTIHZNFpu?= =?iso-8859-1?Q?vHhGiUqVFlRMcFIL/KokLVxuNrE6fw1IJ+hjNg1//ENysL23XetxhHisqm?= =?iso-8859-1?Q?5G9JGviea6Lq55KqSkvNe+g1eTBrb1EFh/wVAS1DDHfASsSiHdFT6QYraP?= =?iso-8859-1?Q?Qpk+kn9wgaXJTGmRGlGCEkvCDq1RFxEjjD6sF2Kdjbs9bfkyczKHzGu1MF?= =?iso-8859-1?Q?9w6PIAxLYOB8VDX8+peNZqxx4J2jUwjacMXrtM1JtzeQvyJ9fgYuFxKoA0?= =?iso-8859-1?Q?NZmB4LlLKSJKr0NguvytWdROp6O1Zyu0fkhek/POTtDkRbuXpDiCanzjNK?= =?iso-8859-1?Q?/7jre10ceny6JoKn43Po6Chx4P/HPqJeGaBN3rDv8vAuQTaNYjSjZcb5ob?= =?iso-8859-1?Q?GapsUnz1dkAtskyM57sKWZwPJ7H+VyQe8XHqx+EwofGLmAPr6AnXDVT1m/?= =?iso-8859-1?Q?RhCxKizBIXpfyDD5m9kUeoePV27yEN+9SS8BnONg2GMlXDrinpP/5Mxtfn?= =?iso-8859-1?Q?m2DNueHv1aZuPzGOuF3DO4kLXVdNqr0QguqGL9hmwm8n2Ezi16YFSSkD/i?= =?iso-8859-1?Q?tZz8T0bsVlkx0Qrtz332OZMqtiD7YG8hbB7h9UURPqxQI28HXOe0wISe/R?= =?iso-8859-1?Q?HHoFiXndH3JhRJ9jFw814jmgYBScs3MTEIWYi0POTwWLLrMngJ4Yf2mENE?= =?iso-8859-1?Q?9cgQN5ZpDQwldaUIUo0LRJt2303at0QdYwPXfjqZr6V4QVfQn7rSPDGeOy?= =?iso-8859-1?Q?UxDzY2kZ866W8xOc4a6ANQ4LzBvG54YLu/?= Content-Type: text/plain; charset="iso-8859-1" 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: d3f8ab0f-2194-4ac7-6758-08db56228bc1 X-MS-Exchange-CrossTenant-originalarrivaltime: 16 May 2023 15:30:57.6646 (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: Avfta/Nqn9gXoa7YZNdfml+9mOWfbtdZyP72dJ7TqbDTBUjKRkjeZBMxczCYVRAHpEh7WlOvbPsQVl5k51FYbw== X-MS-Exchange-Transport-CrossTenantHeadersStamped: MW2PR18MB2089 X-Proofpoint-GUID: ySJoIFRlSkD1qKR79kYT7LCo1XfkuPWd X-Proofpoint-ORIG-GUID: ySJoIFRlSkD1qKR79kYT7LCo1XfkuPWd X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.573,FMLib:17.11.170.22 definitions=2023-05-16_08,2023-05-16_01,2023-02-09_01 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 Anoob, Fix check patch issues and please see some inline comments. > Subject: [PATCH v2 02/22] lib: add pdcp protocol >=20 > Add Packet Data Convergence Protocol (PDCP) processing library. >=20 > The library is similar to lib_ipsec which provides IPsec processing > capabilities in DPDK. >=20 > PDCP would involve roughly the following options, > 1. Transfer of user plane data > 2. Transfer of control plane data > 3. Header compression > 4. Uplink data compression > 5. Ciphering and integrity protection >=20 > PDCP library provides following control path APIs that is used to > configure various PDCP entities, > 1. rte_pdcp_entity_establish() > 2. rte_pdcp_entity_suspend() > 3. rte_pdcp_entity_release() >=20 > Signed-off-by: Anoob Joseph > Signed-off-by: Kiran Kumar K > Signed-off-by: Volodymyr Fialko > --- > doc/api/doxy-api-index.md | 3 +- > doc/api/doxy-api.conf.in | 1 + > lib/meson.build | 1 + > lib/pdcp/meson.build | 17 +++++ > lib/pdcp/pdcp_crypto.c | 21 +++++ > lib/pdcp/pdcp_crypto.h | 15 ++++ > lib/pdcp/pdcp_entity.h | 95 +++++++++++++++++++++++ > lib/pdcp/pdcp_process.c | 138 +++++++++++++++++++++++++++++++++ > lib/pdcp/pdcp_process.h | 13 ++++ > lib/pdcp/rte_pdcp.c | 138 +++++++++++++++++++++++++++++++++ > lib/pdcp/rte_pdcp.h | 157 ++++++++++++++++++++++++++++++++++++++ > lib/pdcp/version.map | 10 +++ > 12 files changed, 608 insertions(+), 1 deletion(-) > create mode 100644 lib/pdcp/meson.build > create mode 100644 lib/pdcp/pdcp_crypto.c > create mode 100644 lib/pdcp/pdcp_crypto.h > create mode 100644 lib/pdcp/pdcp_entity.h > create mode 100644 lib/pdcp/pdcp_process.c > create mode 100644 lib/pdcp/pdcp_process.h > create mode 100644 lib/pdcp/rte_pdcp.c > create mode 100644 lib/pdcp/rte_pdcp.h > create mode 100644 lib/pdcp/version.map >=20 > diff --git a/doc/api/doxy-api-index.md b/doc/api/doxy-api-index.md > index debbe4134f..cd7a6cae44 100644 > --- a/doc/api/doxy-api-index.md > +++ b/doc/api/doxy-api-index.md > @@ -128,7 +128,8 @@ The public API headers are grouped by topics: > [eCPRI](@ref rte_ecpri.h), > [L2TPv2](@ref rte_l2tpv2.h), > [PPP](@ref rte_ppp.h), > - [PDCP hdr](@ref rte_pdcp_hdr.h) > + [PDCP hdr](@ref rte_pdcp_hdr.h), > + [PDCP](@ref rte_pdcp.h) >=20 > - **QoS**: > [metering](@ref rte_meter.h), > diff --git a/doc/api/doxy-api.conf.in b/doc/api/doxy-api.conf.in > index d230a19e1f..58789308a9 100644 > --- a/doc/api/doxy-api.conf.in > +++ b/doc/api/doxy-api.conf.in > @@ -62,6 +62,7 @@ INPUT =3D @TOPDIR@/doc/api/doxy-api- > index.md \ > @TOPDIR@/lib/net \ > @TOPDIR@/lib/pcapng \ > @TOPDIR@/lib/pci \ > + @TOPDIR@/lib/pdcp \ > @TOPDIR@/lib/pdump \ > @TOPDIR@/lib/pipeline \ > @TOPDIR@/lib/port \ > diff --git a/lib/meson.build b/lib/meson.build > index 0812ce6026..d217c04ea9 100644 > --- a/lib/meson.build > +++ b/lib/meson.build > @@ -64,6 +64,7 @@ libraries =3D [ > 'flow_classify', # flow_classify lib depends on pkt framework ta= ble lib > 'graph', > 'node', > + 'pdcp', # pdcp lib depends on crypto and security > ] >=20 > optional_libs =3D [ > diff --git a/lib/pdcp/meson.build b/lib/pdcp/meson.build > new file mode 100644 > index 0000000000..ccaf426240 > --- /dev/null > +++ b/lib/pdcp/meson.build > @@ -0,0 +1,17 @@ > +# SPDX-License-Identifier: BSD-3-Clause > +# Copyright(C) 2023 Marvell. > + > +if is_windows > + build =3D false > + reason =3D 'not supported on Windows' > + subdir_done() > +endif > + > +sources =3D files( > + 'pdcp_crypto.c', > + 'pdcp_process.c', > + 'rte_pdcp.c', > + ) > +headers =3D files('rte_pdcp.h') > + > +deps +=3D ['mbuf', 'net', 'cryptodev', 'security'] > diff --git a/lib/pdcp/pdcp_crypto.c b/lib/pdcp/pdcp_crypto.c > new file mode 100644 > index 0000000000..755e27ec9e > --- /dev/null > +++ b/lib/pdcp/pdcp_crypto.c > @@ -0,0 +1,21 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#include > + > +#include "pdcp_crypto.h" > + > +int > +pdcp_crypto_sess_create(struct rte_pdcp_entity *entity, const struct > rte_pdcp_entity_conf *conf) > +{ > + RTE_SET_USED(entity); > + RTE_SET_USED(conf); > + return 0; > +} > + > +void > +pdcp_crypto_sess_destroy(struct rte_pdcp_entity *entity) > +{ > + RTE_SET_USED(entity); > +} > diff --git a/lib/pdcp/pdcp_crypto.h b/lib/pdcp/pdcp_crypto.h > new file mode 100644 > index 0000000000..6563331d37 > --- /dev/null > +++ b/lib/pdcp/pdcp_crypto.h > @@ -0,0 +1,15 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#ifndef PDCP_CRYPTO_H > +#define PDCP_CRYPTO_H > + > +#include > + > +int pdcp_crypto_sess_create(struct rte_pdcp_entity *entity, > + const struct rte_pdcp_entity_conf *conf); > + > +void pdcp_crypto_sess_destroy(struct rte_pdcp_entity *entity); > + > +#endif /* PDCP_CRYPTO_H */ > diff --git a/lib/pdcp/pdcp_entity.h b/lib/pdcp/pdcp_entity.h > new file mode 100644 > index 0000000000..ca1d56b516 > --- /dev/null > +++ b/lib/pdcp/pdcp_entity.h > @@ -0,0 +1,95 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#ifndef PDCP_ENTITY_H > +#define PDCP_ENTITY_H > + > +#include > +#include > +#include > +#include > +#include > + > +struct entity_priv; > + > +/* IV generation function based on the entity configuration */ > +typedef void (*iv_gen_t)(struct rte_crypto_op *cop, const struct entity_= priv > *en_priv, > + uint32_t count); > + > +struct entity_state { > + uint32_t rx_next; > + uint32_t tx_next; > + uint32_t rx_deliv; > + uint32_t rx_reord; > +}; > + > +/* > + * Layout of PDCP entity: [rte_pdcp_entity] [entity_priv] [entity_dl/ul] > + */ > + > +struct entity_priv { > + /** Crypto sym session. */ > + struct rte_cryptodev_sym_session *crypto_sess; > + /** Entity specific IV generation function. */ > + iv_gen_t iv_gen; > + /** Entity state variables. */ > + struct entity_state state; > + /** Flags. */ > + struct { > + /** PDCP PDU has 4 byte MAC-I. */ > + uint64_t is_authenticated : 1; > + /** Cipher offset & length in bits. */ > + uint64_t is_ciph_in_bits : 1; > + /** Auth offset & length in bits. */ > + uint64_t is_auth_in_bits : 1; > + /** Is UL/transmitting PDCP entity. */ > + uint64_t is_ul_entity : 1; > + /** Is NULL auth. */ > + uint64_t is_null_auth : 1; > + } flags; > + /** Crypto op pool. */ > + struct rte_mempool *cop_pool; > + /** PDCP header size. */ > + uint8_t hdr_sz; > + /** PDCP AAD size. For AES-CMAC, additional message is prepended for > the operation. */ > + uint8_t aad_sz; > + /** Device ID of the device to be used for offload. */ > + uint8_t dev_id; > +}; > + > +struct entity_priv_dl_part { > + /* NOTE: when in-order-delivery is supported, post PDCP packets would > need to cached. */ > + uint8_t dummy; > +}; > + > +struct entity_priv_ul_part { > + /* > + * NOTE: when re-establish is supported, plain PDCP packets & COUNT > values need to be > + * cached. > + */ > + uint8_t dummy; > +}; > + > +static inline struct entity_priv * > +entity_priv_get(const struct rte_pdcp_entity *entity) { > + return RTE_PTR_ADD(entity, sizeof(struct rte_pdcp_entity)); > +} > + > +static inline struct entity_priv_dl_part * > +entity_dl_part_get(const struct rte_pdcp_entity *entity) { > + return RTE_PTR_ADD(entity, sizeof(struct rte_pdcp_entity) + > sizeof(struct entity_priv)); > +} > + > +static inline struct entity_priv_ul_part * > +entity_ul_part_get(const struct rte_pdcp_entity *entity) { > + return RTE_PTR_ADD(entity, sizeof(struct rte_pdcp_entity) + > sizeof(struct entity_priv)); > +} > + > +static inline int > +pdcp_hdr_size_get(enum rte_security_pdcp_sn_size sn_size) > +{ > + return RTE_ALIGN_MUL_CEIL(sn_size, 8) / 8; > +} > + > +#endif /* PDCP_ENTITY_H */ > diff --git a/lib/pdcp/pdcp_process.c b/lib/pdcp/pdcp_process.c > new file mode 100644 > index 0000000000..d4b158536d > --- /dev/null > +++ b/lib/pdcp/pdcp_process.c > @@ -0,0 +1,138 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#include > +#include > +#include > +#include > +#include > +#include > + > +#include "pdcp_crypto.h" > +#include "pdcp_entity.h" > +#include "pdcp_process.h" > + > +static int > +pdcp_crypto_xfrm_get(const struct rte_pdcp_entity_conf *conf, struct > rte_crypto_sym_xform **c_xfrm, > + struct rte_crypto_sym_xform **a_xfrm) > +{ > + *c_xfrm =3D NULL; > + *a_xfrm =3D NULL; > + > + if (conf->crypto_xfrm =3D=3D NULL) > + return -EINVAL; > + > + if (conf->crypto_xfrm->type =3D=3D RTE_CRYPTO_SYM_XFORM_CIPHER) { > + *c_xfrm =3D conf->crypto_xfrm; > + *a_xfrm =3D conf->crypto_xfrm->next; > + } else if (conf->crypto_xfrm->type =3D=3D > RTE_CRYPTO_SYM_XFORM_AUTH) { > + *a_xfrm =3D conf->crypto_xfrm; > + *c_xfrm =3D conf->crypto_xfrm->next; > + } else { > + return -EINVAL; > + } > + > + return 0; > +} > + > +static int > +pdcp_entity_priv_populate(struct entity_priv *en_priv, const struct > rte_pdcp_entity_conf *conf) > +{ > + struct rte_crypto_sym_xform *c_xfrm, *a_xfrm; > + int ret; > + > + /** > + * flags.is_authenticated > + * > + * MAC-I would be added in case of control plane packets and when > authentication > + * transform is not NULL. > + */ > + > + if (conf->pdcp_xfrm.domain =3D=3D > RTE_SECURITY_PDCP_MODE_CONTROL) > + en_priv->flags.is_authenticated =3D 1; This check should be added after getting the xfrm. If domain =3D=3D control and a_xfrm is NULL, then it should be error, right= ? > + > + ret =3D pdcp_crypto_xfrm_get(conf, &c_xfrm, &a_xfrm); > + if (ret) > + return ret; > + > + if (a_xfrm !=3D NULL) > + en_priv->flags.is_authenticated =3D 1; > + > + /** > + * flags.is_ciph_in_bits > + * > + * For ZUC & SNOW3G cipher algos, offset & length need to be provided > in bits. > + */ > + > + if ((c_xfrm->cipher.algo =3D=3D RTE_CRYPTO_CIPHER_SNOW3G_UEA2) || > + (c_xfrm->cipher.algo =3D=3D RTE_CRYPTO_CIPHER_ZUC_EEA3)) > + en_priv->flags.is_ciph_in_bits =3D 1; > + > + /** > + * flags.is_auth_in_bits > + * > + * For ZUC & SNOW3G authentication algos, offset & length need to be > provided in bits. > + */ > + > + if (a_xfrm !=3D NULL) { > + if ((a_xfrm->auth.algo =3D=3D RTE_CRYPTO_AUTH_SNOW3G_UIA2) > || > + (a_xfrm->auth.algo =3D=3D RTE_CRYPTO_AUTH_ZUC_EIA3)) > + en_priv->flags.is_auth_in_bits =3D 1; > + } > + > + /** > + * flags.is_ul_entity > + * > + * Indicate whether the entity is UL/transmitting PDCP entity. > + */ > + if (conf->pdcp_xfrm.pkt_dir =3D=3D RTE_SECURITY_PDCP_UPLINK) > + en_priv->flags.is_ul_entity =3D 1; > + > + /** > + * flags.is_null_auth > + * > + * For NULL auth, 4B zeros need to be added by lib PDCP. Indicate that > + * algo is NULL auth to perform the same. > + */ > + if (a_xfrm !=3D NULL && a_xfrm->auth.algo =3D=3D > RTE_CRYPTO_AUTH_NULL) > + en_priv->flags.is_null_auth =3D 1; > + > + /** > + * hdr_sz > + * > + * PDCP header size of the entity > + */ > + en_priv->hdr_sz =3D pdcp_hdr_size_get(conf->pdcp_xfrm.sn_size); > + > + /** > + * aad_sz > + * > + * For AES-CMAC, additional message is prepended for processing. Need > to be trimmed after > + * crypto processing is done. > + */ > + if (a_xfrm !=3D NULL && a_xfrm->auth.algo =3D=3D > RTE_CRYPTO_AUTH_AES_CMAC) > + en_priv->aad_sz =3D 8; > + else > + en_priv->aad_sz =3D 0; > + > + return 0; > +} > + > +int > +pdcp_process_func_set(struct rte_pdcp_entity *entity, const struct > rte_pdcp_entity_conf *conf) > +{ > + struct entity_priv *en_priv; > + int ret; > + > + if (entity =3D=3D NULL || conf =3D=3D NULL) > + return -EINVAL; > + > + en_priv =3D entity_priv_get(entity); > + > + ret =3D pdcp_entity_priv_populate(en_priv, conf); > + if (ret) > + return ret; > + > + return 0; > +} > diff --git a/lib/pdcp/pdcp_process.h b/lib/pdcp/pdcp_process.h > new file mode 100644 > index 0000000000..fd53fff0aa > --- /dev/null > +++ b/lib/pdcp/pdcp_process.h > @@ -0,0 +1,13 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#ifndef PDCP_PROCESS_H > +#define PDCP_PROCESS_H > + > +#include > + > +int > +pdcp_process_func_set(struct rte_pdcp_entity *entity, const struct > rte_pdcp_entity_conf *conf); > + > +#endif /* PDCP_PROCESS_H */ > diff --git a/lib/pdcp/rte_pdcp.c b/lib/pdcp/rte_pdcp.c > new file mode 100644 > index 0000000000..8914548dbd > --- /dev/null > +++ b/lib/pdcp/rte_pdcp.c > @@ -0,0 +1,138 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#include > +#include > +#include > + > +#include "pdcp_crypto.h" > +#include "pdcp_entity.h" > +#include "pdcp_process.h" > + > +static int > +pdcp_entity_size_get(const struct rte_pdcp_entity_conf *conf) > +{ > + int size; > + > + size =3D sizeof(struct rte_pdcp_entity) + sizeof(struct entity_priv); > + > + if (conf->pdcp_xfrm.pkt_dir =3D=3D RTE_SECURITY_PDCP_DOWNLINK) > + size +=3D sizeof(struct entity_priv_dl_part); > + else if (conf->pdcp_xfrm.pkt_dir =3D=3D RTE_SECURITY_PDCP_UPLINK) > + size +=3D sizeof(struct entity_priv_ul_part); > + else > + return -EINVAL; > + > + return RTE_ALIGN_CEIL(size, RTE_CACHE_LINE_SIZE); > +} > + > +struct rte_pdcp_entity * > +rte_pdcp_entity_establish(const struct rte_pdcp_entity_conf *conf) > +{ > + struct rte_pdcp_entity *entity =3D NULL; > + struct entity_priv *en_priv; > + int ret, entity_size; > + > + if (conf =3D=3D NULL || conf->cop_pool =3D=3D NULL) { > + rte_errno =3D -EINVAL; > + return NULL; > + } errnos are normally set as positive values. > + > + if (conf->pdcp_xfrm.en_ordering || conf- > >pdcp_xfrm.remove_duplicates || conf->is_slrb || > + conf->en_sec_offload) { > + rte_errno =3D -ENOTSUP; > + return NULL; > + } > + > + /* > + * 6.3.2 PDCP SN > + * Length: 12 or 18 bits as indicated in table 6.3.2-1. The length of t= he > PDCP SN is > + * configured by upper layers (pdcp-SN-SizeUL, pdcp-SN-SizeDL, or sl- > PDCP-SN-Size in > + * TS 38.331 [3]) > + */ > + if ((conf->pdcp_xfrm.sn_size !=3D RTE_SECURITY_PDCP_SN_SIZE_12) && > + (conf->pdcp_xfrm.sn_size !=3D RTE_SECURITY_PDCP_SN_SIZE_18)) { > + rte_errno =3D -ENOTSUP; > + return NULL; > + } Check for PDCP crypto algos may also be added. As only 4 cipher and 4 auth algos are supported in case of PDCP. > + > + if (conf->pdcp_xfrm.hfn || conf->pdcp_xfrm.hfn_threshold) { > + rte_errno =3D -EINVAL; > + return NULL; > + } What is the reason to set errno as EINVAL when HFN is set? > + > + entity_size =3D pdcp_entity_size_get(conf); > + if (entity_size < 0) { > + rte_errno =3D -EINVAL; > + return NULL; > + } > + > + entity =3D rte_zmalloc_socket("pdcp_entity", entity_size, > RTE_CACHE_LINE_SIZE, SOCKET_ID_ANY); > + if (entity =3D=3D NULL) { > + rte_errno =3D -ENOMEM; > + return NULL; > + } > + > + en_priv =3D entity_priv_get(entity); > + > + en_priv->state.rx_deliv =3D conf->count; > + en_priv->state.tx_next =3D conf->count; > + en_priv->cop_pool =3D conf->cop_pool; > + > + /* Setup crypto session */ > + ret =3D pdcp_crypto_sess_create(entity, conf); > + if (ret) > + goto entity_free; > + > + ret =3D pdcp_process_func_set(entity, conf); > + if (ret) > + goto crypto_sess_destroy; > + > + return entity; > + > +crypto_sess_destroy: > + pdcp_crypto_sess_destroy(entity); > +entity_free: > + rte_free(entity); > + rte_errno =3D ret; > + return NULL; > +} > + > +int > +rte_pdcp_entity_release(struct rte_pdcp_entity *pdcp_entity, struct rte_= mbuf > *out_mb[]) > +{ > + if (pdcp_entity =3D=3D NULL) > + return -EINVAL; > + > + /* Teardown crypto sessions */ > + pdcp_crypto_sess_destroy(pdcp_entity); > + > + rte_free(pdcp_entity); > + > + RTE_SET_USED(out_mb); > + return 0; > +} > + > +int > +rte_pdcp_entity_suspend(struct rte_pdcp_entity *pdcp_entity, > + struct rte_mbuf *out_mb[]) > +{ > + struct entity_priv *en_priv; > + > + if (pdcp_entity =3D=3D NULL) > + return -EINVAL; > + > + en_priv =3D entity_priv_get(pdcp_entity); > + > + if (en_priv->flags.is_ul_entity) { > + en_priv->state.tx_next =3D 0; > + } else { > + en_priv->state.rx_next =3D 0; > + en_priv->state.rx_deliv =3D 0; > + } > + > + RTE_SET_USED(out_mb); > + > + return 0; > +} > diff --git a/lib/pdcp/rte_pdcp.h b/lib/pdcp/rte_pdcp.h > new file mode 100644 > index 0000000000..33c355b05a > --- /dev/null > +++ b/lib/pdcp/rte_pdcp.h > @@ -0,0 +1,157 @@ > +/* SPDX-License-Identifier: BSD-3-Clause > + * Copyright(C) 2023 Marvell. > + */ > + > +#ifndef RTE_PDCP_H > +#define RTE_PDCP_H > + > +/** > + * @file rte_pdcp.h > + * > + * RTE PDCP support. > + * > + * librte_pdcp provides a framework for PDCP protocol processing. A framework for PDCP protocol processing. > + */ > + > +#include > +#include > +#include > +#include > + > +#ifdef __cplusplus > +extern "C" { > +#endif > + > +/** > + * PDCP entity. You can probably explain more on what a PDCP entity is. > + */ > +struct rte_pdcp_entity { > + /** > + * PDCP entities may hold packets for purposes of in-order delivery (in > + * case of receiving PDCP entity) and re-transmission (in case of > + * transmitting PDCP entity). > + * > + * For receiving PDCP entity, it may hold packets when in-order > + * delivery is enabled. The packets would be cached until either a > + * packet that completes the sequence arrives or when t-Reordering > timer > + * expires. > + * > + * When post-processing of PDCP packet which completes a sequence is > + * done, the API may return more packets than enqueued. Application is > + * expected to provide *rte_pdcp_pkt_post_process()* with *out_mb* > + * which can hold maximum number of packets which may be returned. > + */ The above comment explains the need for holding the packets. But it does not talk about the parameter it is explaining. This explanation should be part of programmer's guide and not the API guide= . > + uint32_t max_pkt_cache; > + /** User area for saving application data. */ > + uint64_t user_area[2]; Is it being used right now in the patches? If not, can we add it later? And if really needed now, can we rename to user_data > +} __rte_cache_aligned; > + > +/** > + * PDCP entity configuration to be used for establishing an entity. > + */ > +/* Structure rte_pdcp_entity_conf 8< */ > +struct rte_pdcp_entity_conf { > + /** PDCP transform for the entity. */ > + struct rte_security_pdcp_xform pdcp_xfrm; > + /** Crypto transform applicable for the entity. */ > + struct rte_crypto_sym_xform *crypto_xfrm; > + /** Mempool for crypto symmetric session. */ > + struct rte_mempool *sess_mpool; > + /** Crypto op pool.*/ > + struct rte_mempool *cop_pool; > + /** > + * 32 bit count value (HFN + SN) to be used for the first packet. > + * pdcp_xfrm.hfn would be ignored as the HFN would be derived from > this value. > + */ If the HFN is to be ignored, then why to add a check in entity establish an= d return EINVAL? It should be silently ignored in that case with a debug print at max. > + uint32_t count; > + /** Indicate whether the PDCP entity belongs to Side Link Radio Bearer. > */ > + bool is_slrb; > + /** Enable security offload on the device specified. */ > + bool en_sec_offload; > + /** Device on which security/crypto session need to be created. */ > + uint8_t dev_id; > + /** Reverse direction during IV generation. Can be used to simulate UE > crypto processing.*/ > + bool reverse_iv_direction; > +}; > +/* >8 End of structure rte_pdcp_entity_conf. */ > + > +/** > + * @warning > + * @b EXPERIMENTAL: this API may change without prior notice > + * > + * 5.1.1 PDCP entity establishment > + * > + * Establish PDCP entity based on provided input configuration. > + * > + * @param conf > + * Parameters to be used for initializing PDCP entity object. > + * @return > + * - Valid handle if success > + * - NULL in case of failure. rte_errno will be set to error code > + */ > +__rte_experimental > +struct rte_pdcp_entity * > +rte_pdcp_entity_establish(const struct rte_pdcp_entity_conf *conf); > + > +/** > + * @warning > + * @b EXPERIMENTAL: this API may change without prior notice > + * > + * 5.1.3 PDCP entity release > + * > + * Release PDCP entity. > + * > + * For UL/transmitting PDCP entity, all stored PDCP SDUs would be droppe= d. > + * For DL/receiving PDCP entity, the stored PDCP SDUs would be returned = in > + * *out_mb* buffer. The buffer should be large enough to hold all cached > + * packets in the entity. > + * > + * @param pdcp_entity > + * Pointer to the PDCP entity to be released. > + * @param[out] out_mb > + * The address of an array that can hold up to > *rte_pdcp_entity.max_pkt_cache* > + * pointers to *rte_mbuf* structures. > + * @return > + * - 0: Success and no cached packets to return > + * - >0: Success and the number of packets returned in out_mb > + * - <0: Error code in case of failures > + */ > +__rte_experimental > +int > +rte_pdcp_entity_release(struct rte_pdcp_entity *pdcp_entity, > + struct rte_mbuf *out_mb[]); > + > +/** > + * @warning > + * @b EXPERIMENTAL: this API may change without prior notice > + * > + * 5.1.4 PDCP entity suspend > + * > + * Suspend PDCP entity. > + * > + * For DL/receiving PDCP entity, the stored PDCP SDUs would be returned = in > + * *out_mb* buffer. The buffer should be large enough to hold all cached > + * packets in the entity. > + * > + * For UL/transmitting PDCP entity, *out_mb* buffer would be unused. > + * > + * @param pdcp_entity > + * Pointer to the PDCP entity to be suspended. > + * @param[out] out_mb > + * The address of an array that can hold up to > *rte_pdcp_entity.max_pkt_cache* > + * pointers to *rte_mbuf* structures. > + * @return > + * - 0: Success and no cached packets to return > + * - >0: Success and the number of packets returned in out_mb > + * - <0: Error code in case of failures > + */ > +__rte_experimental > +int > +rte_pdcp_entity_suspend(struct rte_pdcp_entity *pdcp_entity, > + struct rte_mbuf *out_mb[]); > + > +#ifdef __cplusplus > +} > +#endif > + > +#endif /* RTE_PDCP_H */ > diff --git a/lib/pdcp/version.map b/lib/pdcp/version.map > new file mode 100644 > index 0000000000..923e165f3f > --- /dev/null > +++ b/lib/pdcp/version.map > @@ -0,0 +1,10 @@ > +EXPERIMENTAL { > + global: > + > + # added in 23.07 > + rte_pdcp_entity_establish; > + rte_pdcp_entity_release; > + rte_pdcp_entity_suspend; > + > + local: *; > +}; > -- > 2.25.1