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 56F5142D0E; Wed, 21 Jun 2023 06:47:05 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id E7E9C40DFB; Wed, 21 Jun 2023 06:47:04 +0200 (CEST) Received: from mx0b-0016f401.pphosted.com (mx0b-0016f401.pphosted.com [67.231.156.173]) by mails.dpdk.org (Postfix) with ESMTP id 9E6C940DDC for ; Wed, 21 Jun 2023 06:47:03 +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 35L0ZggM006728; Tue, 20 Jun 2023 21:47:02 -0700 Received: from nam10-mw2-obe.outbound.protection.outlook.com (mail-mw2nam10lp2100.outbound.protection.outlook.com [104.47.55.100]) by mx0b-0016f401.pphosted.com (PPS) with ESMTPS id 3rbkup10xe-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 20 Jun 2023 21:47:01 -0700 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=MTYyB9VKfE8LiHCuA7ei3sBkcbwCUBV/vp1HiVqvh4GC+PJVRk6AcvlxKarR7AvrUFIOVUKwVOghpzeCtbX5w9Ae6rreC9uonLjeeLp09/9slLN0nG+7agRnhJybLSF+SJVE04Fr/Rxu9jyED/OYmEOpNuyEGGF0rnE/Er6ydS1TjI+MOSif6+wz2LBdNGup+x1v1tiLiuEZSRDv9adXFWEWyjpHyLIiaqNu0qIfV5uspZZllvAzWuO5idSMB/4b6nsj5aoAMngF3veFZo/WdQFE8EepF6Glk0qlrJ+sJgWNH+OivyCUX4qPYu09s1wvQeV1HA9d1tEp1PwLHJiiFw== 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=5iva2X6ggZpeuN+NKr/EZQnmla1k/6/Mio6fQpA8GrE=; b=lTW/rwdysgsnAFS/tMXP9X1rpwpehXvq0TK8OtlRO01gURH0JMOvR0uDkFLhz31Z5FJQH4fzBjgBIKf7fRo8zOWsWf5IOnJxJrngQ4OCNvJelp+L2xpIxs3QmYwOUmFbyTdgmyyYzs5d6/yGSaAv+DnqIWoo8moz4tuDCCPWgcBjC/qSrmBrU/rtzKNnqP/j+tJR860T2Tz3+6z2Pwml0l7TeUlW7Q8Q4bf6LhTNH/Ovo/iFJFd1knL8pOL0ALAafWjHCIAysK8Zsv+w1XGY071IuyPnEvDJB93Zc3O6OXPpp/z9ZJKj+TPTiFEBtZh0OMttLGEiFG/X7VZ9HgC9qQ== 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=5iva2X6ggZpeuN+NKr/EZQnmla1k/6/Mio6fQpA8GrE=; b=gU5YHMKnRMZyKo5m4HrM4c4URramZEHEUxi2/K0Q2rnZxw+mkiSgNruA2In+oCm35Ard/AOjG9Pz1YMY13ZvsONf9+pVw8KnyjsU2OHAeOLkTn5gZDSbugD9gyxiOPkIK+iy7IPBFGTd1XAJbgZ8WNBHjfWHlQECpl8LKHicaWk= Received: from CO1PR18MB4540.namprd18.prod.outlook.com (2603:10b6:303:ff::13) by PH0PR18MB4442.namprd18.prod.outlook.com (2603:10b6:510:ec::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6521.23; Wed, 21 Jun 2023 04:46:58 +0000 Received: from CO1PR18MB4540.namprd18.prod.outlook.com ([fe80::d975:c5fa:8025:f521]) by CO1PR18MB4540.namprd18.prod.outlook.com ([fe80::d975:c5fa:8025:f521%6]) with mapi id 15.20.6521.020; Wed, 21 Jun 2023 04:46:58 +0000 From: Ashwin Sekhar T K To: Thierry Herbelot , "dev@dpdk.org" CC: Thomas Monjalon , Nithin Kumar Dabilpuram , Satha Koteswara Rao Kottidi , Jerin Jacob Kollanukkaran Subject: RE: [EXT] [PATCH v3] common/cnxk/roc_npa: unlock mbox in error cases Thread-Topic: [EXT] [PATCH v3] common/cnxk/roc_npa: unlock mbox in error cases Thread-Index: AQHZo1u4EpXJCGq89ESVd9kAiV887a+Ur73w Date: Wed, 21 Jun 2023 04:46:58 +0000 Message-ID: References: <20230620093950.3127383-1-thierry.herbelot@6wind.com> <20230620094337.3127595-1-thierry.herbelot@6wind.com> In-Reply-To: <20230620094337.3127595-1-thierry.herbelot@6wind.com> Accept-Language: en-GB, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-dg-rorf: true x-dg-ref: =?us-ascii?Q?PG1ldGE+PGF0IG5tPSJib2R5LnR4dCIgcD0iYzpcdXNlcnNcYXNla2hhclxh?= =?us-ascii?Q?cHBkYXRhXHJvYW1pbmdcMDlkODQ5YjYtMzJkMy00YTQwLTg1ZWUtNmI4NGJh?= =?us-ascii?Q?MjllMzViXG1zZ3NcbXNnLWEzZjgyOWJjLTBmZWUtMTFlZS05NGNkLWEwYTRj?= =?us-ascii?Q?NTk3MjdmNFxhbWUtdGVzdFxhM2Y4MjliZS0wZmVlLTExZWUtOTRjZC1hMGE0?= =?us-ascii?Q?YzU5NzI3ZjRib2R5LnR4dCIgc3o9IjIyNzQiIHQ9IjEzMzMxNzk2NDEzMjgw?= =?us-ascii?Q?MzU5NCIgaD0iTldHZk9QaDY0aXRwMVpmVlZSVXdUeDFiREtRPSIgaWQ9IiIg?= =?us-ascii?Q?Ymw9IjAiIGJvPSIxIiBjaT0iY0FBQUFFUkhVMVJTUlVGTkNnVUFBTjRQQUFB?= =?us-ascii?Q?S1pGQm0rNlBaQVdYVlM4dnNka2ZoWmRWTHkreDJSK0VaQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUhBQUFBQnVEd0FBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUVBQVFFQkFBQUFJN3FUcEFDQUFRQUFBQUFBQUFBQUFKNEFBQUJoQUdRQVpB?= =?us-ascii?Q?QnlBR1VBY3dCekFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?RUFBQUFBQUFBQUFnQUFBQUFBbmdBQUFHTUFkUUJ6QUhRQWJ3QnRBRjhBY0FC?= =?us-ascii?Q?bEFISUFjd0J2QUc0QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFB?= =?us-ascii?Q?QUFBQ2VBQUFBWXdCMUFITUFkQUJ2QUcwQVh3QndBR2dBYndCdUFHVUFiZ0Ix?= =?us-ascii?Q?QUcwQVlnQmxBSElBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUJBQUFBQUFBQUFBSUFBQUFBQUo0QUFBQmpBSFVB?= =?us-ascii?Q?Y3dCMEFHOEFiUUJmQUhNQWN3QnVBRjhBWkFCaEFITUFhQUJmQUhZQU1BQXlB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-refone: =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFFQUFBQUFBQUFBQWdBQUFBQUFuZ0FBQUdN?= =?us-ascii?Q?QWRRQnpBSFFBYndCdEFGOEFjd0J6QUc0QVh3QnJBR1VBZVFCM0FHOEFjZ0Jr?= =?us-ascii?Q?QUhNQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBUUFBQUFBQUFBQUNBQUFBQUFDZUFBQUFZd0IxQUhNQWRBQnZBRzBB?= =?us-ascii?Q?WHdCekFITUFiZ0JmQUc0QWJ3QmtBR1VBYkFCcEFHMEFhUUIwQUdVQWNnQmZB?= =?us-ascii?Q?SFlBTUFBeUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQkFBQUFBQUFB?= =?us-ascii?Q?QUFJQUFBQUFBSjRBQUFCakFIVUFjd0IwQUc4QWJRQmZBSE1BY3dCdUFGOEFj?= =?us-ascii?Q?d0J3QUdFQVl3QmxBRjhBZGdBd0FESUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUVBQUFBQUFBQUFBZ0FBQUFBQW5nQUFB?= =?us-ascii?Q?R1FBYkFCd0FGOEFjd0JyQUhrQWNBQmxBRjhBWXdCb0FHRUFkQUJmQUcwQVpR?= =?us-ascii?Q?QnpBSE1BWVFCbkFHVUFYd0IyQURBQU1nQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFRQUFBQUFBQUFBQ0FBQUFBQUNlQUFBQVpBQnNBSEFBWHdCekFH?= =?us-ascii?Q?d0FZUUJqQUdzQVh3QmpBR2dBWVFCMEFGOEFiUUJsQUhNQWN3QmhBR2NBWlFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-reftwo: =?us-ascii?Q?QUFBQUFBQUFBQUFCQUFBQUFBQUFBQUlBQUFBQUFKNEFBQUJrQUd3QWNBQmZB?= =?us-ascii?Q?SFFBWlFCaEFHMEFjd0JmQUc4QWJnQmxBR1FBY2dCcEFIWUFaUUJmQUdZQWFR?= =?us-ascii?Q?QnNBR1VBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBRUFB?= =?us-ascii?Q?QUFBQUFBQUFnQUFBQUFBbmdBQUFHVUFiUUJoQUdrQWJBQmZBR0VBWkFCa0FI?= =?us-ascii?Q?SUFaUUJ6QUhNQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFDUUFBQUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFBQUFB?= =?us-ascii?Q?Q2VBQUFBYlFCaEFISUFkZ0JsQUd3QVh3QndBSElBYndCcUFHVUFZd0IwQUY4?= =?us-ascii?Q?QWJnQmhBRzBBWlFCekFGOEFZd0J2QUc0QVpnQnBBR1FBWlFCdUFIUUFhUUJo?= =?us-ascii?Q?QUd3QVh3QmhBR3dBYndCdUFHVUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUJBQUFBQUFBQUFBSUFBQUFBQUo0QUFBQnRBR0VBY2dC?= =?us-ascii?Q?MkFHVUFiQUJmQUhBQWNnQnZBR29BWlFCakFIUUFYd0J1QUdFQWJRQmxBSE1B?= =?us-ascii?Q?WHdCeUFHVUFjd0IwQUhJQWFRQmpBSFFBWlFCa0FGOEFZUUJzQUc4QWJnQmxB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFF?= =?us-ascii?Q?QUFBQUFBQUFBQWdBQUFBQUFuZ0FBQUcwQVlRQnlBSFlBWlFCc0FGOEFjQUJ5?= =?us-ascii?Q?QUc4QWFnQmxBR01BZEFCZkFHNEFZUUJ0QUdVQWN3QmZBSElBWlFCekFIUUFj?= =?us-ascii?Q?Z0JwQUdNQWRBQmxBR1FBWHdCb0FHVUFlQUJqQUc4QVpBQmxBSE1BQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFBQUNBQUFB?= =?us-ascii?Q?QUFDZUFBQUFiUUJoQUhJQWRnQmxBR3dBYkFCZkFHRUFjZ0J0QUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-refthree: =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQkFBQUFBQUFBQUFJ?= =?us-ascii?Q?QUFBQUFBSjRBQUFCdEFHRUFjZ0IyQUdVQWJBQnNBRjhBWndCdkFHOEFad0Jz?= =?us-ascii?Q?QUdVQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUVBQUFBQUFBQUFBZ0FBQUFBQW5nQUFBRzBB?= =?us-ascii?Q?WVFCeUFIWUFaUUJzQUd3QVh3QndBSElBYndCcUFHVUFZd0IwQUY4QVl3QnZB?= =?us-ascii?Q?R1FBWlFCekFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFRQUFBQUFBQUFBQ0FBQUFBQUNlQUFBQWJRQmhBSElBZGdCbEFHd0Fi?= =?us-ascii?Q?QUJmQUhBQWNnQnZBR29BWlFCakFIUUFYd0JqQUc4QVpBQmxBSE1BWHdCa0FH?= =?us-ascii?Q?a0FZd0IwQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFCQUFBQUFBQUFB?= =?us-ascii?Q?QUlBQUFBQUFKNEFBQUJ0QUdFQWNnQjJBR1VBYkFCc0FGOEFjQUJ5QUc4QWFn?= =?us-ascii?Q?QmxBR01BZEFCZkFHNEFZUUJ0QUdVQWN3QmZBR01BYndCdUFHWUFhUUJrQUdV?= =?us-ascii?Q?QWJnQjBBR2tBWVFCc0FGOEFiUUJoQUhJQWRnQmxBR3dBYkFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBRUFBQUFBQUFBQUFnQUFBQUFBbmdBQUFH?= =?us-ascii?Q?MEFZUUJ5QUhZQVpRQnNBR3dBWHdCd0FISUFid0JxQUdVQVl3QjBBRjhBYmdC?= =?us-ascii?Q?aEFHMEFaUUJ6QUY4QVl3QnZBRzRBWmdCcEFHUUFaUUJ1QUhRQWFRQmhBR3dB?= =?us-ascii?Q?WHdCdEFHRUFjZ0IyQUdVQWJBQnNBRjhBYndCeUFGOEFZUUJ5QUcwQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= x-dg-reffour: =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQVFBQUFBQUFBQUFDQUFBQUFBQ2VB?= =?us-ascii?Q?QUFBYlFCaEFISUFkZ0JsQUd3QWJBQmZBSEFBY2dCdkFHb0FaUUJqQUhRQVh3?= =?us-ascii?Q?QnVBR0VBYlFCbEFITUFYd0JqQUc4QWJnQm1BR2tBWkFCbEFHNEFkQUJwQUdF?= =?us-ascii?Q?QWJBQmZBRzBBWVFCeUFIWUFaUUJzQUd3QVh3QnZBSElBWHdCbkFHOEFid0Ju?= =?us-ascii?Q?QUd3QVpRQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUJBQUFBQUFBQUFBSUFBQUFBQUo0QUFBQnRBR0VBY2dCMkFH?= =?us-ascii?Q?VUFiQUJzQUY4QWNBQnlBRzhBYWdCbEFHTUFkQUJmQUc0QVlRQnRBR1VBY3dC?= =?us-ascii?Q?ZkFISUFaUUJ6QUhRQWNnQnBBR01BZEFCbEFHUUFYd0J0QUdFQWNnQjJBR1VB?= =?us-ascii?Q?YkFCc0FBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFFQUFB?= =?us-ascii?Q?QUFBQUFBQWdBQUFBQUFuZ0FBQUcwQVlRQnlBSFlBWlFCc0FHd0FYd0J3QUhJ?= =?us-ascii?Q?QWJ3QnFBR1VBWXdCMEFGOEFiZ0JoQUcwQVpRQnpBRjhBY2dCbEFITUFkQUJ5?= =?us-ascii?Q?QUdrQVl3QjBBR1VBWkFCZkFHMEFZUUJ5QUhZQVpRQnNBR3dBWHdCdkFISUFY?= =?us-ascii?Q?d0JoQUhJQWJRQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBUUFBQUFBQUFBQUNBQUFBQUFD?= =?us-ascii?Q?ZUFBQUFiUUJoQUhJQWRnQmxBR3dBYkFCZkFIUUFaUUJ5QUcwQWFRQnVBSFVB?= =?us-ascii?Q?Y3dBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQkFBQUFBQUFBQUFJQUFBQUFBSjRBQUFCdEFHRUFjZ0Iy?= =?us-ascii?Q?QUdVQWJBQnNBRjhBZHdCdkFISUFaQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFB?= =?us-ascii?Q?QUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUFBQUVB?= =?us-ascii?Q?QUFBQUFBQUFBZ0FBQUFBQSIvPjwvbWV0YT4=3D?= x-ms-publictraffictype: Email x-ms-traffictypediagnostic: CO1PR18MB4540:EE_|PH0PR18MB4442:EE_ x-ms-office365-filtering-correlation-id: e330e2e4-67a6-4991-7d73-08db72128bc6 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: +4wzJaLQ3Tbgv8TtOojhwCOpjplNfytAP4I6o6n9wtGAJubnHa9q2z9PcmxnKwLuGdhigugIp+VkR59FjXY5PSKZrsHhUpa8AljM+VSRHGYyjuvI35q7oHXi0jwZ/I6ia7+VegBmP9p2u45BCZ7Awco9cpaj8027uFplRUoBGFeDRrxIcnnKyX499HCRPaWdFuvwkVtwWIuQKpqoCJlDTOA7GuI+QWSUD5IrobSRcGhhw7W7PZKqufZeY77IU86WVBg781EhmdykGh7wk6nt2F0WG4XHR00wOcltopRCTrsXceF7EuTGSYoskBnJumWQjGfqQYlQXCPi+YvmK/XNWJcByDrKe12QqCz7jOzfrkMYanSQ7+7WP6hCc/3VrXXQcQg71bXt4pbz1ovfle0TxYE47ScZCeh+CWBwki3IPHWkSGciJHTJ/bDSPuw6Txg4IQHbWAnmcawsZQ65omc4gGlfOCECHNkeatITNNSoIYMO+5vBwV06JRxeD5g9yXaKdbJZcYMNTJkr3WeCAeBjVw9pRfvIIFnOBXJ1nsmTsKOQflCL8w3lX2pL0iQm5+mu5RZu+lCzwRMCgChsm9ESsueDE1Gi6/upf2K1dIJoTXgXv/JpM1G8fxgNHBZ3jE1H x-forefront-antispam-report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:CO1PR18MB4540.namprd18.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230028)(4636009)(39860400002)(396003)(346002)(366004)(136003)(376002)(451199021)(76116006)(64756008)(66446008)(66946007)(66476007)(66556008)(4326008)(186003)(122000001)(38100700002)(107886003)(86362001)(38070700005)(53546011)(6506007)(9686003)(83380400001)(33656002)(7696005)(478600001)(71200400001)(55016003)(110136005)(54906003)(8936002)(8676002)(5660300002)(52536014)(2906002)(316002)(41300700001); DIR:OUT; SFP:1101; x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?RLUQeVZsYVWJc6gGx9bfCL0dfbDyTGE+hwL+qYlGq7UwYHoF6npovcVjEnN+?= =?us-ascii?Q?qgmKFI+lomqryFd3hDQqfgeyo/vAMlX2jVuPezw8P3qWkKg44YoazFRtr7lF?= =?us-ascii?Q?li9LOxUfv3KTH1F+55Jx71nz3Zo3JlWXY4Z+y6/ryyFhKl0zJOWe2pmTE3sB?= =?us-ascii?Q?uCNoDffF11pIFTOQdgBEEaqcQMRKiM+w8EKWxmbHMxvk+HUmJhPWknvHk/98?= =?us-ascii?Q?WsU5ON4uw4GgYApud1a/wOtNf/07XLtu3nKoJoNGu6u1WkZAB6fjmtdeU8OI?= =?us-ascii?Q?DXrwitVofc/+fzMGeS9k09uhzsyHZ4MyuZE9HnbmpAtrtSL/SQDdHytXXJOE?= =?us-ascii?Q?Bl9xFWsTi1JJ+GLRzBkUgBZf7OUuZ2o4Zwq3X8Qr8RsChYQKh27pJAwjPyDq?= =?us-ascii?Q?rm1vx0CrB1nQrOHkr5R/zECq/H5X6zcBx5SKDETPaMe51MjDLp2r7k7wQSla?= =?us-ascii?Q?GGd50jbx+jBTPBJFsiq0QxBlqU/34vCE/BESCFYJQwH4p94dbKAgYe1rypV9?= =?us-ascii?Q?s1P+1PRmfZFQ5QhSqVXuUhnF2FwhYzJO6MRtE3FrYYXrbBp88PLZqUwzpmPQ?= =?us-ascii?Q?/WDODLapy5XXqcS8BU5oppAVHEkSWsZjcRiTL81JAbRwuEI9adIQiNYqRVWk?= =?us-ascii?Q?7EibxZKlNwa6da+SIWdLDCfhhcTC99jCJ55cSTZyTpNFM4SpB+FPUJMI2qz3?= =?us-ascii?Q?Hc2FGn08tlVex3YqJ/8awQGkxGRrkJIog6EWrPMRDk1JfDZz9jO53DvKRsLt?= =?us-ascii?Q?Y2/wmh3jrpPkfCs3Gl2AH3wVuQ0nr9i5H7XS3PEmhIJmFfYJrzwZiQLYAv5s?= =?us-ascii?Q?4Lu09Ddg2822yXj8bI8atj4fFpLpLWXWH80zfXI/MPjKA+9cBt77WGzM0gjA?= =?us-ascii?Q?CI1iFEMNMXmNQu4K0IzAoAcGFTU0KE8cy7BhCGPzVrl4Sqyfe3ugWNwRIeoz?= =?us-ascii?Q?vQNWec6NOrv0cnNvwrk+AWd4J9Co6fN/JV3fPRwcdwnqjcLc3J70C+7XxAXw?= =?us-ascii?Q?DRZjHkk/yOsuRHkTTk0DBnCY87+FaE+twRd0QPnnlOVy9zgAkfGjsUQ+5mVV?= =?us-ascii?Q?Zn9xvAa7p6Uy6YhWregUKdVLumrKfuyon5ugbNrRy+i9RYvCSpqfYJrGGsl7?= =?us-ascii?Q?hmR3eqoJgzdalXd9ndW0MKOoohN1/9TkioZqq20gOk8yQK9/cgF9ZF5O9oeh?= =?us-ascii?Q?Tpx2zgx8YBYMtkXYs8/a+YIG21oHoM2D5kpRMY6tVX+mxPsbcGq/cFvD/OeG?= =?us-ascii?Q?CLt7+mTuNs7occDvxBa4aESQyAjYcQM1yIjXigTcTwgcDW9ZFssZEAwOIO11?= =?us-ascii?Q?aELLWxp26HKVggixvffBGZV+uIJQRG0O8GVl9H93H86OyWtgIf455/C78iId?= =?us-ascii?Q?zoWugPhdQn32fjNSb57/FTUUCNR/FORwyBg/W64mb1N6KYaSwPn78OAs8WmS?= =?us-ascii?Q?30hKS3y8LwUZ01h4pcUw7chsr8GaIxoCUKwYVc0YyicvEqVPKTBX8zovYrIh?= =?us-ascii?Q?89YpeC6XShsWQZ1ypzejyQuIdeBTezVlsyKwDKVlnHsiAUJDkWn77BKljBCH?= =?us-ascii?Q?X/yIdqWPiKN44jeTmsZ3Qx+772UZHlbuFwqBqkNQuHKwtkaSHEEj3bdnxxEG?= =?us-ascii?Q?/+Okh8tjLfxXIuF48nqA70mmsSyret9YiGDeeJhNoZUk?= 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: CO1PR18MB4540.namprd18.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: e330e2e4-67a6-4991-7d73-08db72128bc6 X-MS-Exchange-CrossTenant-originalarrivaltime: 21 Jun 2023 04:46:58.2747 (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: ULdsbwO+DUV3HBjpCcOhPsLLgVlDTuvKoOHhBVy/S3OlCL+NAyVCqFngcjtH6AvofOeqpGjdOTENGPCSn7bw4g== X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR18MB4442 X-Proofpoint-GUID: xD7NxONTEdZSrZRaDXaSzxhXfTuapKKk X-Proofpoint-ORIG-GUID: xD7NxONTEdZSrZRaDXaSzxhXfTuapKKk X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.254,Aquarius:18.0.957,Hydra:6.0.591,FMLib:17.11.176.26 definitions=2023-06-21_03,2023-06-16_01,2023-05-22_02 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 Acked-by: Ashwin Sekhar T K > -----Original Message----- > From: Thierry Herbelot > Sent: Tuesday, June 20, 2023 3:14 PM > To: dev@dpdk.org > Cc: Thierry Herbelot ; Thomas Monjalon > ; Nithin Kumar Dabilpuram > ; Satha Koteswara Rao Kottidi > ; Jerin Jacob Kollanukkaran > > Subject: [EXT] [PATCH v3] common/cnxk/roc_npa: unlock mbox in error > cases >=20 > External Email >=20 > ---------------------------------------------------------------------- > In npa_aura_pool_fini and in npa_aura_fini, Unlock mbox when an error is > detected. >=20 > Fixes: ea4d70cc4f0b46d5 ('common/cnxk: add NPA aura create/destroy ROC > APIs') > Signed-off-by: Thierry Herbelot > -- > V2: also patch npa_aura_pool_fini > V3: add missing space > --- > drivers/common/cnxk/roc_npa.c | 12 ++++++++---- > 1 file changed, 8 insertions(+), 4 deletions(-) >=20 > diff --git a/drivers/common/cnxk/roc_npa.c > b/drivers/common/cnxk/roc_npa.c index 377439c2baaf..3b9a70028ba2 > 100644 > --- a/drivers/common/cnxk/roc_npa.c > +++ b/drivers/common/cnxk/roc_npa.c > @@ -197,8 +197,10 @@ npa_aura_pool_fini(struct mbox *m_box, uint32_t > aura_id, uint64_t aura_handle) > off =3D mbox->rx_start + pool_rsp->hdr.next_msgoff; > aura_rsp =3D (struct npa_aq_enq_rsp *)((uintptr_t)mdev->mbase + > off); >=20 > - if (aura_rsp->hdr.rc !=3D 0 || pool_rsp->hdr.rc !=3D 0) > - return NPA_ERR_AURA_POOL_FINI; > + if (aura_rsp->hdr.rc !=3D 0 || pool_rsp->hdr.rc !=3D 0) { > + rc =3D NPA_ERR_AURA_POOL_FINI; > + goto exit; > + } >=20 > /* Sync NDC-NPA for LF */ > ndc_req =3D mbox_alloc_msg_ndc_sync_op(mbox); @@ -245,8 > +247,10 @@ npa_aura_fini(struct mbox *m_box, uint32_t aura_id) > if (rc < 0) > goto exit; >=20 > - if (aura_rsp->hdr.rc !=3D 0) > - return NPA_ERR_AURA_POOL_FINI; > + if (aura_rsp->hdr.rc !=3D 0) { > + rc =3D NPA_ERR_AURA_POOL_FINI; > + goto exit; > + } >=20 > /* Sync NDC-NPA for LF */ > ndc_req =3D mbox_alloc_msg_ndc_sync_op(mbox); > -- > 2.39.2