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 743A2428D7; Thu, 6 Apr 2023 10:12:55 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 1098D40F18; Thu, 6 Apr 2023 10:12:55 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 9EE6940DF6 for ; Thu, 6 Apr 2023 10:12:53 +0200 (CEST) Received: from pps.filterd (m0045851.ppops.net [127.0.0.1]) by mx0b-0016f401.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 3367eKPV004873; Thu, 6 Apr 2023 01:12:53 -0700 Received: from nam10-bn7-obe.outbound.protection.outlook.com (mail-bn7nam10lp2102.outbound.protection.outlook.com [104.47.70.102]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3psg3ras21-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Thu, 06 Apr 2023 01:12:52 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=VjEZS8AgqWlvhSGydiMdY6W1QySU8l8g5vL4KFhMLSIjAShmzzEw/CHxchUwMvM1eh+KorrHJNzCNWzt5qt/76LUnXJJIr35oGW1mrxwSf/5lT/z2oqEDROwezqc8ZbpXltenSx5w1KbdZ7FFHV1shAt7gHXw6TxGBqMKxd1tqMu65CYR6zMy57y64LAVBYXCmpS22nRLKfqK4u8z3oSVQhUnkxNxXKjl+J3IPSoJRS3EkiLkd6oK7zrl3GeM1XAdvtOzCuJalB3bk4a40cN+Gk4omWC8JeeJGuTE4hm+3uJR7Ea7o45sf8MZVZaob2oeNGhK4Kr4sQBNlfvMID7hw== 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=mkbMp9ZX6RRNtX4oMlkA0S51HIvBTcYbDHVOys8bqlc=; b=VvSMi2tqVDIYvxTPPx+MeJ6K47WSqDsKIsjzmRWY2l7kFO3H5R18bFYvZNQ+8fRGOgaBavMOYfehFo96/+3snJvyI/i6ZV+MYHB56wmg7LNpkxXIp6eA/hhb/tXZyVClBOESIHqsF5GZ/ACJ44a36FOeLt3DdMtQ9qDpY6p8ZQ53fuF6VwiqMbfsx8uEmYJSS7sJyV5zoSusUGzWtcp4Wq7pGnANdjsvS+mDRKajOBRzCWQsJJ1KziUJ+ZUpOq2+qn0mEm6+jDxSPmdFX+VfTbgUP6ulJ0kfgo/E7mA2Puun2cKg3h8IpZh+Bjq25lsmmfWEqxzadrEjhEg08uRoow== 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=mkbMp9ZX6RRNtX4oMlkA0S51HIvBTcYbDHVOys8bqlc=; b=ATZr25s9JbBc2J8a8CK4MN1Fy9h2YRCCYBxAjb7yNqJjzXqS22FIIK1qzzpibF/6fC440irU7e0YwG2qzI/2BtUQbkL8ERswfXwR4rsYdbnRzwfYIhF6VhbXxFchN37MZz5G04/TLOu2E47SquEI+eg2jKvri2/wqZHns4lfdAI= Received: from PH0PR18MB4086.namprd18.prod.outlook.com (2603:10b6:510:3::9) by BYAPR18MB2903.namprd18.prod.outlook.com (2603:10b6:a03:10d::26) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6277.30; Thu, 6 Apr 2023 08:12:49 +0000 Received: from PH0PR18MB4086.namprd18.prod.outlook.com ([fe80::2e10:cd72:2bc4:b4bb]) by PH0PR18MB4086.namprd18.prod.outlook.com ([fe80::2e10:cd72:2bc4:b4bb%4]) with mapi id 15.20.6254.028; Thu, 6 Apr 2023 08:12:49 +0000 From: Pavan Nikhilesh Bhagavatula To: "Naga Harish K, S V" , Jerin Jacob Kollanukkaran CC: "dev@dpdk.org" Subject: RE: [PATCH v2] eventdev/eth_rx: fix invalid memory access Thread-Topic: [PATCH v2] eventdev/eth_rx: fix invalid memory access Thread-Index: AQHZZ6McsxPO3x3tlEWvOjDiRFEwUa8d5bqAgAAJlJA= Date: Thu, 6 Apr 2023 08:12:49 +0000 Message-ID: References: <20230405093959.2052-1-pbhagavatula@marvell.com> <20230405094332.2862-1-pbhagavatula@marvell.com> In-Reply-To: Accept-Language: en-IN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-dg-ref: =?us-ascii?Q?PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNccGJoYWdhdmF0?= =?us-ascii?Q?dWxhXGFwcGRhdGFccm9hbWluZ1wwOWQ4NDliNi0zMmQzLTRhNDAtODVlZS02?= =?us-ascii?Q?Yjg0YmEyOWUzNWJcbXNnc1xtc2ctY2ZiNWJiYjUtZDQ1Mi0xMWVkLWI2Y2It?= =?us-ascii?Q?NGMwMzRmNWZhYTM2XGFtZS10ZXN0XGNmYjViYmI3LWQ0NTItMTFlZC1iNmNi?= =?us-ascii?Q?LTRjMDM0ZjVmYWEzNmJvZHkudHh0IiBzej0iNDExOCIgdD0iMTMzMjUyNDIz?= =?us-ascii?Q?NjY1NTY0Mjk2IiBoPSJMcUJRTzUwQi9EKzZ4NjlsaTlMMCtSc1NueTg9IiBp?= =?us-ascii?Q?ZD0iIiBibD0iMCIgYm89IjEiIGNpPSJjQUFBQUVSSFUxUlNSVUZOQ2dVQUFI?= =?us-ascii?Q?WUlBQUNJdWd1U1gyalpBUlN3MDE1NW55T21GTERUWG5tZkk2WU5BQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBSEFBQUFBR0NBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBRUFBUUFCQUFBQTFGSDNhQUFBQUFBQUFBQUFBQUFBQUo0QUFBQmhB?= =?us-ascii?Q?R1FBWkFCeUFHVUFjd0J6QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFFQUFBQUFBQUFBQWdBQUFBQUFuZ0FBQUdNQWRRQnpBSFFBYndCdEFG?= =?us-ascii?Q?OEFjQUJsQUhJQWN3QnZBRzRBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFB?= =?us-ascii?Q?QUNBQUFBQUFDZUFBQUFZd0IxQUhNQWRBQnZBRzBBWHdCd0FHZ0Fid0J1QUdV?= =?us-ascii?Q?QWJnQjFBRzBBWWdCbEFISUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQkFBQUFBQUFBQUFJQUFBQUFBSjRBQUFC?= =?us-ascii?Q?akFIVUFjd0IwQUc4QWJRQmZBSE1BY3dCdUFGOEFaQUJoQUhNQWFBQmZBSFlB?= =?us-ascii?Q?TUFBeUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-refone: =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUVBQUFBQUFBQUFBZ0FBQUFBQW5n?= =?us-ascii?Q?QUFBR01BZFFCekFIUUFid0J0QUY4QWN3QnpBRzRBWHdCckFHVUFlUUIzQUc4?= =?us-ascii?Q?QWNnQmtBSE1BQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFRQUFBQUFBQUFBQ0FBQUFBQUNlQUFBQVl3QjFBSE1BZEFC?= =?us-ascii?Q?dkFHMEFYd0J6QUhNQWJnQmZBRzRBYndCa0FHVUFiQUJwQUcwQWFRQjBBR1VB?= =?us-ascii?Q?Y2dCZkFIWUFNQUF5QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFCQUFB?= =?us-ascii?Q?QUFBQUFBQUlBQUFBQUFKNEFBQUJqQUhVQWN3QjBBRzhBYlFCZkFITUFjd0J1?= =?us-ascii?Q?QUY4QWN3QndBR0VBWXdCbEFGOEFkZ0F3QURJQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBRUFBQUFBQUFBQUFnQUFBQUFB?= =?us-ascii?Q?bmdBQUFHUUFiQUJ3QUY4QWN3QnJBSGtBY0FCbEFGOEFZd0JvQUdFQWRBQmZB?= =?us-ascii?Q?RzBBWlFCekFITUFZUUJuQUdVQVh3QjJBREFBTWdBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFBQUFBQ2VBQUFBWkFCc0FIQUFY?= =?us-ascii?Q?d0J6QUd3QVlRQmpBR3NBWHdCakFHZ0FZUUIwQUY4QWJRQmxBSE1BY3dCaEFH?= =?us-ascii?Q?Y0FaUUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-reftwo: =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUJBQUFBQUFBQUFBSUFBQUFBQUo0QUFBQmtBR3dB?= =?us-ascii?Q?Y0FCZkFIUUFaUUJoQUcwQWN3QmZBRzhBYmdCbEFHUUFjZ0JwQUhZQVpRQmZB?= =?us-ascii?Q?R1lBYVFCc0FHVUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFFQUFBQUFBQUFBQWdBQUFBQUFuZ0FBQUdVQWJRQmhBR2tBYkFCZkFHRUFa?= =?us-ascii?Q?QUJrQUhJQVpRQnpBSE1BQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQURBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFBQUNB?= =?us-ascii?Q?QUFBQUFDZUFBQUFiUUJoQUhJQWRnQmxBR3dBYkFCZkFIQUFjZ0J2QUdvQVpR?= =?us-ascii?Q?QmpBSFFBWHdCakFHOEFaQUJsQUhNQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQkFBQUFBQUFBQUFJQUFBQUFBSjRBQUFCdEFH?= =?us-ascii?Q?RUFjZ0IyQUdVQWJBQnNBRjhBZEFCbEFISUFiUUJwQUc0QWRRQnpBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUVBQUFBQUFBQUFBZ0FBQUFBQSIvPjwvbWV0YT4=3D?= x-dg-rorf: true x-ms-publictraffictype: Email x-ms-traffictypediagnostic: PH0PR18MB4086:EE_|BYAPR18MB2903:EE_ x-ms-office365-filtering-correlation-id: 3ccbb227-83bc-4314-b99d-08db3676b623 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Tl/V16/pS8GNYr2LtNFJLj7X4OWBvAE1f7NHUn23D/zuax2Q4PHeFS9WexWZJkZ/f5UKEsE7H7IRUX821A+WoqcO5iG6uRuL94JodEuKBSS/xKa+P+rFiv3g63iOyiqJMOkd9PcVUSwlvzie7P40ExV83jVQvwQhrdDqtdU84m/GeMK0K6NPQXJSYXhv5AKPgdmi9WgRab9MDJMlyA0/wWiIcmm2gRnpp066krbvEMkLfrxLmfgufTv6MA9knL/pnhVZ5yloe/kDsFMepCG//iPEqZdLRwKa0c0Dl3CuboGhRRxnQHNeUSTaAt4O/aUxgu7VnLbUBWKeLvCHklGAKbci3FP5HXd8ZMDxY00Xh+QACa6MNnaPZOiMzAxN+yBX4m0loHkNcHYzrpnEX+HDVlqJbpHulFrWT7734yoQP60l7A4s/vdfOGW1aC/EwaqI78D6U4CO/LpCV0ZzqyHpyV69g2w1gRO4DGC4IqKUEpCsuPerf8d9ymhukJRyx/qiGBIvwExY2FVwqC2VOjxO9wTh9jsEyz1WR8W3zYflFyT7UbKNpn5wYItTMg3oQK0X0OtvV4Z3r/dKJWikymkM/3M79gXRtcvlgp2qnT/pTptAsS/Lkjwi2yiaMO1Lt1eb x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:PH0PR18MB4086.namprd18.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(346002)(396003)(39860400002)(376002)(136003)(366004)(451199021)(83380400001)(316002)(7696005)(71200400001)(186003)(478600001)(9686003)(6506007)(26005)(6636002)(53546011)(110136005)(5660300002)(66556008)(2906002)(52536014)(33656002)(38100700002)(122000001)(4326008)(66476007)(64756008)(66446008)(41300700001)(86362001)(55016003)(76116006)(66946007)(8676002)(8936002)(38070700005); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?ao9xffht2ik0zcBBvJ1WeQSqWaxJgbUCCVFD0iJ1lT4JVPOdyZYH1Q92sNYj?= =?us-ascii?Q?oXjM9fu9X8DoJbPJQpaEae8BROe8zOhr6ohLPlLrqf6mb6WLB5PCfWvJtiZO?= =?us-ascii?Q?QlR3Q7lfq9MCcBdTR0HEBgg1JVg7ECnLMpSfJZ5J+VTaIrtXLSBPHWDvONr0?= =?us-ascii?Q?2/xvVzr+d9INk4WeQyXgA7uvaaA8Nw3yP+uOe8nX1TfoKqErd7HwKDLh1nVu?= =?us-ascii?Q?DHpM7EfBzbDDmksGCwL8Ugdo9UHrYprzAhHDNPF2EbBmKlcfUjf0OiXWLDqj?= =?us-ascii?Q?5OTMyCG2KW9McOIKbNRT1fFFRaPdBc3Ee2DepICFwEChDpXaOrKt9EnN5E1g?= =?us-ascii?Q?3Cb3hrRgID8h6UB5Fw3/gB29WJaFzwxWBlylnju2v3h0dlhBPNTrAaCHY20W?= =?us-ascii?Q?OwpFgHDhC1Cw/clU9AWHJIUgt5XAooU4TySp2jLCz/cZgQcXXhx9qlb2/05g?= =?us-ascii?Q?20eKUESQcPCPezq7dY5/thdXaCief5alsQqG2z4unQENmNAAYbD0XMRsEBA3?= =?us-ascii?Q?r0Fw4IP5s49dLrSsabTf+rmpJjaqyLIaP77Gvtd4yT2MO/3XmPFpuECHADsG?= =?us-ascii?Q?taMl/JrUumvW6G2Tza0e6bMLF44IWEZ1YmyJJhyC3QyVzB05f5AOQbGwufq/?= =?us-ascii?Q?uFLltAScUO0YqNBts6R9nnXd/ms5fz4CygMwtCjBrT2/5loxvk8WwHCt0IrK?= =?us-ascii?Q?H4kyaWfJzalPN5I4KT4kZPVi+sJT+5N/Ds9F1R8m8V24YxyTkTUQox1j3WQJ?= =?us-ascii?Q?CS6JlgllUPoDoJrSXAp8adkJzod3VJehHRl1PVoMsBElDJwrnAIMMABIUwYh?= =?us-ascii?Q?KKNaOuRUhZSn+NnMkj8Yp59UN3NjwjTfp4L9wmW19b3PehkCeqzcMM/e6eSS?= =?us-ascii?Q?OwQPge64Cc7R8P2W/xtIxDmtiJ0WZkrxzvH/D+Y9n8awSCmsL1mMiXB57/aq?= =?us-ascii?Q?BIpB7z0HT+NM5zs7xS9OOmYt52VPizy41HLGZyqmvxXRsMJhBXJ9o6dMFYG1?= =?us-ascii?Q?dKe8YkSrF5BNY1ykL07JaEtDKc7jtqxpR4UIGzEqk2uBs0rVX57DpcX+qoGw?= =?us-ascii?Q?o9/UtmwseCITcY9xEczRKZWp6HuuQFiJnhjR97WA0tdrWHw/tMAByvPgLdpr?= =?us-ascii?Q?ud9lzkWhcs4bEvrNzEDsuX2X4AO3BtJ2IvQz8CZ5maUjEgC71ScV9c8T35ln?= =?us-ascii?Q?gWmXF3LcBDVuMwAzATSrRG5K/JlIEyzQ/QIvDwlebzgflJZYsrhoBCmi7izr?= =?us-ascii?Q?3iKjWfT/cnVz3382qTA2jv1UuVJdnYbSFhaxfGYwxLc3PWAXzpIp+JUU6fut?= =?us-ascii?Q?C//jfTGs0kLNp8M325fU4hzXmaIiIiBS5hFyWLWDQsLUouacC4R/8G3Zlmj4?= =?us-ascii?Q?ffFOgNeWAsnyheT280Tu83iHGihag5CY5x8IXjAMVyNGm3q8x3B3W6Qj+rkn?= =?us-ascii?Q?SS6gXdH1AnIJGodbiZCfnP0FjgKDaDcTUf4GSmCNgG/Ga5BIUHBz69U0mL6F?= =?us-ascii?Q?W+AhfIerc/mlqHy3Pe/8VX5WyR6k5bv869mbv/lRBh1rS2/QIvupQFd/6pU2?= =?us-ascii?Q?6umm5N0OAyaYD8SBxEfE4bLPb52uWmMvSCzk/b8R?= 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: PH0PR18MB4086.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3ccbb227-83bc-4314-b99d-08db3676b623 X-MS-Exchange-CrossTenant-originalarrivaltime: 06 Apr 2023 08:12:49.2768 (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: zi/HW1WmKI3mhCVbXCrhr+dgliuYuSYMx7UDnhZr1RxeQI3ezhEZwbgRXmJP4sik+IUOPaH11MShcTdhxv58zl5s/l1T0xM9fbcO3pOHrYs= X-MS-Exchange-Transport-CrossTenantHeadersStamped: BYAPR18MB2903 X-Proofpoint-ORIG-GUID: F9FQ0PfWcj5CjdLsvaw3w1ftMbr3bQtK X-Proofpoint-GUID: F9FQ0PfWcj5CjdLsvaw3w1ftMbr3bQtK X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.942,Hydra:6.0.573,FMLib:17.11.170.22 definitions=2023-04-06_03,2023-04-05_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 > -----Original Message----- > From: Naga Harish K, S V > Sent: Thursday, April 6, 2023 1:06 PM > To: Pavan Nikhilesh Bhagavatula ; Jerin Jacob > Kollanukkaran > Cc: dev@dpdk.org > Subject: [EXT] RE: [PATCH v2] eventdev/eth_rx: fix invalid memory access >=20 > External Email >=20 > ---------------------------------------------------------------------- > Hi, >=20 > > -----Original Message----- > > From: pbhagavatula@marvell.com > > Sent: Wednesday, April 5, 2023 3:14 PM > > To: jerinj@marvell.com; Naga Harish K, S V > > Cc: dev@dpdk.org; Pavan Nikhilesh > > Subject: [PATCH v2] eventdev/eth_rx: fix invalid memory access > > >=20 > Which invalid memory access are you referring to? rxa->eth_rx_poll[0] is NULL when internal port is used. Also, currently it doesn't account for case where the same Rx adapter is us= ed for=20 both internal and non-internal ports, this patch fixes that case too. >=20 > > From: Pavan Nikhilesh > > > > Rx adapter internal service structures are only initialized when non-in= ternal > > port is added to the adapter. > > Check if Rx adapter service is initialized to determine if runtime set/= get > > parameters are supported. > > > > Fixes: 3716f52186af ("eventdev/eth_rx: support runtime set/get > > parameters") > > > > Signed-off-by: Pavan Nikhilesh > > --- > > v2 Changes: > > ---------- > > - Remove unused variables. > > - Fix spell check. > > > > app/test/test_event_eth_rx_adapter.c | 2 ++ > > lib/eventdev/rte_event_eth_rx_adapter.c | 24 ++++-------------------- > > 2 files changed, 6 insertions(+), 20 deletions(-) > > > > diff --git a/app/test/test_event_eth_rx_adapter.c > > b/app/test/test_event_eth_rx_adapter.c > > index cfe6052d7e..b1775e6af1 100644 > > --- a/app/test/test_event_eth_rx_adapter.c > > +++ b/app/test/test_event_eth_rx_adapter.c > > @@ -1230,6 +1230,8 @@ adapter_get_set_params(void) > > /* Case 1: Get the default value of mbufs processed by Rx adapter */ > > err =3D > > rte_event_eth_rx_adapter_runtime_params_get(TEST_INST_ID, > > &out_params); > > + if (err =3D=3D -ENOTSUP) >=20 > The queue needs to be deleted from the adapter instance before > returning, otherwise > teardown fails as adapter_free fails because all the queues are not > deleted. I will add teardown in next version. Thanks. >=20 > > + return TEST_SKIPPED; > > TEST_ASSERT(err =3D=3D 0, "Expected 0 got %d", err); > > > > /* Case 2: Set max_nb_rx =3D 32 (=3DBATCH_SEIZE) */ diff --git > > a/lib/eventdev/rte_event_eth_rx_adapter.c > > b/lib/eventdev/rte_event_eth_rx_adapter.c > > index c1cd124002..a8a5564b65 100644 > > --- a/lib/eventdev/rte_event_eth_rx_adapter.c > > +++ b/lib/eventdev/rte_event_eth_rx_adapter.c > > @@ -3466,30 +3466,14 @@ > > rte_event_eth_rx_adapter_instance_get(uint16_t eth_dev_id, static int > > rxa_caps_check(struct event_eth_rx_adapter *rxa) { > > - uint16_t eth_dev_id; > > - uint32_t caps =3D 0; > > - int ret; > > - > > if (!rxa->nb_queues) > > return -EINVAL; > > > > - /* The eth_dev used is always of same type. > > - * Hence eth_dev_id is taken from first entry of poll array. > > - */ > > - eth_dev_id =3D rxa->eth_rx_poll[0].eth_dev_id; > > - ret =3D rte_event_eth_rx_adapter_caps_get(rxa->eventdev_id, > > - eth_dev_id, > > - &caps); > > - if (ret) { > > - RTE_EDEV_LOG_ERR("Failed to get adapter caps edev %" > > PRIu8 > > - "eth port %" PRIu16, rxa->eventdev_id, eth_dev_id); > > - return ret; > > - } > > - > > - if (caps & RTE_EVENT_ETH_RX_ADAPTER_CAP_INTERNAL_PORT) > > - return -ENOTSUP; > > + /* Check if there is at least one non-internal ethernet port. */ > > + if (rxa->service_inited) > > + return 0; > > > > - return 0; > > + return -ENOTSUP; > > } > > > > int > > -- > > 2.25.1