Received-SPF: pass (google.com: domain of linux-kernel+bounces-117027-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: Subbaraya Sundeep Bhatta <sbhatta@marvell.com>
To: Su Hui <suhui@nfschina.com>, Sunil Kovvuri Goutham <sgoutham@marvell.com>,
        Geethasowjanya Akula <gakula@marvell.com>,
        Hariprasad Kelam
	<hkelam@marvell.com>,
        "davem@davemloft.net" <davem@davemloft.net>,
        "edumazet@google.com" <edumazet@google.com>,
        "kuba@kernel.org"
	<kuba@kernel.org>,
        "pabeni@redhat.com" <pabeni@redhat.com>
CC: Sai Krishna Gajula <saikrishnag@marvell.com>,
        "netdev@vger.kernel.org"
	<netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org"
	<linux-kernel@vger.kernel.org>,
        "kernel-janitors@vger.kernel.org"
	<kernel-janitors@vger.kernel.org>
Subject: RE: [EXTERNAL] [PATCH 1/2] octeontx2-pf: check negative error code in
 otx2_open()
Thread-Topic: [EXTERNAL] [PATCH 1/2] octeontx2-pf: check negative error code
 in otx2_open()
Thread-Index: AQHafocejEkZK72mrkOj39rY0uPDDrFIZiIA
Date: Mon, 25 Mar 2024 12:42:05 +0000
Message-ID: 
 <CO1PR18MB4666EC4FB52BAF4A1A8EA021A1362@CO1PR18MB4666.namprd18.prod.outlook.com>
References: <20240325073549.823832-1-suhui@nfschina.com>
In-Reply-To: <20240325073549.823832-1-suhui@nfschina.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: 
 =?us-ascii?Q?wd7dmSaiBT+GAZtuHBGG/od/rxLaVx/MhiMHwXM7K3WBjRhuCILHFei4ZnSE?=
 =?us-ascii?Q?q5e2IskCHH6M4Kn84xcOR4iIprs+LZ6EEcFXKQh7VeclxNPATb1Ec16g3GeJ?=
 =?us-ascii?Q?+gjydJwGWaocNXqx296+P/m6Dzt9rI8q73mRf/nH3accIXkoCWGrCQYZkRBv?=
 =?us-ascii?Q?B8X7n+vZQBr/Y6cJ6H7a7U4NGSiREyN0Ng9yqDRIpYYYlHyLfBnyjKXbARPB?=
 =?us-ascii?Q?Ht1vKES/BpkQ6u+5F8jCBCiF8MmZS5uRUHBnbWTPsVa2/ax/s68695I3gWfN?=
 =?us-ascii?Q?6g1gwM4IUFkqJagiHF2ljbL4bz3HgI6upWYwrqaBvRZE5QAwV3Ku6OJ08zV1?=
 =?us-ascii?Q?eTFJwUjieujlz7l1mOpARSDyJm6koO/oZf+CLXcdOpPbpFfV61a/2l7cxo9E?=
 =?us-ascii?Q?mMcgnLH/u0yC6AI9LdIwg+haqEUJGKWTpGIwUtY2gRM3fA1HJiejVa916+NN?=
 =?us-ascii?Q?unuzREPmqwezZtRVU5p/ay6D8bEMZHwQ6eRXPNs9BQtHaAaMXDIyZtucHteG?=
 =?us-ascii?Q?cJqDGBEtTXi7iccq6IoqIkwDp2KCmj7Pg76pQGC+/9vpaSeVaMrb8pLPL1Jw?=
 =?us-ascii?Q?7HdFyDOslvDrBXZvlW1W13RFKinn1bujHOkI0eTHtAwyto3xzP/sOftkRwHF?=
 =?us-ascii?Q?4G3jbX311FTvWv2d3Yt5B8YXjFi6fqFZM6e5GEuDnJFQaGwq7jSZaqz2i/dB?=
 =?us-ascii?Q?zKJXWXMXFvIi2j51SsKTXXgZJQPSmC422suSGJIIw0pB2TX6nwhw1QgkfBn3?=
 =?us-ascii?Q?dqFOQ/RSrJXY7wGUuza8gTTsS3tHSM3sGZX/QE5izEyUXvPEmSKP6WBXY9H+?=
 =?us-ascii?Q?MG91TR8hgYqQj+GQqQNL+ix5m51fWaNVX+rdEwCowwXplrE7v1xwWaG2HCit?=
 =?us-ascii?Q?trf6kvx+tRqkareTGczYLsEEAvVk3xxGBMYCMNO+KSHXHXCHxkeO/zjkSMOA?=
 =?us-ascii?Q?kOkA1+RBwsuAlWJqfC/JV9XS/EWnZ70Yw/bl73LptT9tzsqLIYk78VdgzRq4?=
 =?us-ascii?Q?2s+PAPPc+I5OKhnNshUONOeq7EIcK2sUqebR/4itCJhb3Z6PD/xomtbLnuqT?=
 =?us-ascii?Q?v65HfOg9/C63150lD8FeWjsCGBiLs/hde8da2Rxn8nG6uG5StvxyEpa/vQve?=
 =?us-ascii?Q?zWf5IWUA86OjELCcRrVgH2sJjhLHEs7sdDzXVTD/tHRf9O1e1IwLp9eS5WZe?=
 =?us-ascii?Q?cNOWYfc1Jyhx5zil/9TM2n1o2rb/EZrkMMBo5QWupgEmooprefjdQK5nyeZz?=
 =?us-ascii?Q?PGXBF2JCgeTUuo3sWsHwvIPDq9GFL3jEVa8tuU52ZIjp751p6vBfiypodmwn?=
 =?us-ascii?Q?D+zRdiudUhZLXGHgRnt80vCwYdj5NKUzeQYx8Go3j04Y2TldgavYPk9Smn3/?=
 =?us-ascii?Q?WZZuTsPiaNtY7JPsJAain8Sycgeg4oLwGx2rhG7ZCMWzXWK9prF2yaU+Q95R?=
 =?us-ascii?Q?DKOcHkZOp6NehyPuTZnftaxj+ArzjTSb5q/JlYs1fM5CW9gI28pOzPI/6koe?=
 =?us-ascii?Q?9NM1P2Eq1cmg9agyi4MLbZLaa3OPfAn52C5cYWhUkS8baKTykhxHPAMGLQ?=
 =?us-ascii?Q?=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
X-Mailing-List: linux-kernel@vger.kernel.org
List-Id: <linux-kernel.vger.kernel.org>
List-Subscribe: <mailto:linux-kernel+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-kernel+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-OriginatorOrg: marvell.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CO1PR18MB4666.namprd18.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b43c63f0-2803-4284-7af1-08dc4cc8fa00
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Mar 2024 12:42:05.1226
 (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: e3Wl3Blj/XaoHHoZyOU9jU5QSjdrcr90Ij+K+PlpSUzVx/AxO6q7zzk/K0Q22Wpy9JOePI+VOF3rzYmFIuExCQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN4PR18MB4855
X-Proofpoint-GUID: pGJt-F_25vXM5sZ03ASdEZGRoKFTtgll
X-Proofpoint-ORIG-GUID: pGJt-F_25vXM5sZ03ASdEZGRoKFTtgll
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-03-25_08,2024-03-21_02,2023-05-22_02


>-----Original Message-----
>From: Su Hui <suhui@nfschina.com>
>Sent: Monday, March 25, 2024 1:06 PM
>To: Sunil Kovvuri Goutham <sgoutham@marvell.com>; Geethasowjanya Akula
><gakula@marvell.com>; Subbaraya Sundeep Bhatta <sbhatta@marvell.com>;
>Hariprasad Kelam <hkelam@marvell.com>; davem@davemloft.net;
>edumazet@google.com; kuba@kernel.org; pabeni@redhat.com
>Cc: Su Hui <suhui@nfschina.com>; Sai Krishna Gajula
><saikrishnag@marvell.com>; netdev@vger.kernel.org; linux-
>kernel@vger.kernel.org; kernel-janitors@vger.kernel.org
>Subject: [PATCH 1/2] octeontx2-pf: check negative error code in
>otx2_open()
>
>otx2_rxtx_enable() return negative error code such as -EIO,
>check -EIO rather than EIO to fix this problem.
>
>Fixes: c926252205c4 ("octeontx2-pf: Disable packet I/O for graceful exit")
>Signed-off-by: Su Hui <suhui@nfschina.com>

Reviewed-by: Subbaraya Sundeep <sbhatta@marvell.com>