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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id E0CD5CDB474 for ; Mon, 16 Oct 2023 15:00:47 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233864AbjJPPAq (ORCPT ); Mon, 16 Oct 2023 11:00:46 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:43792 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233871AbjJPPAp (ORCPT ); Mon, 16 Oct 2023 11:00:45 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3D3A1E6 for ; Mon, 16 Oct 2023 08:00:43 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 4ADF0C433C9; Mon, 16 Oct 2023 15:00:42 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1697468442; bh=Zc5itXTGcrNkGJmVXSQ8NaEqyPRDB7z4Fr235s2jyiA=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=S2zvR6KGUi2MwiDuWy/c5FOSOwvHEecgZvvekMWuksq5+kCKMiMao05ArYhRwhMB0 07GlBERCjp/Sf/nx1C/LYwhKPRH4XzdVwWvJBMObfr7so0N29wn3IplJNPF7+I6M5p FD6uSs1vKuklCVJk6d6xW6W3jGS56nTo9CZVz9cY= Date: Mon, 16 Oct 2023 16:46:34 +0200 From: Greg Kroah-Hartman To: Haiyang Zhang Cc: "stable@vger.kernel.org" , "patches@lists.linux.dev" , Simon Horman , Shradha Gupta , Paolo Abeni , Sasha Levin Subject: Re: [PATCH 6.1 001/131] net: mana: Fix TX CQE error handling Message-ID: <2023101613-verbalize-runaround-f67f@gregkh> References: <20231016084000.050926073@linuxfoundation.org> <20231016084000.092429858@linuxfoundation.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Mon, Oct 16, 2023 at 02:35:15PM +0000, Haiyang Zhang wrote: > > > > -----Original Message----- > > From: Greg Kroah-Hartman > > Sent: Monday, October 16, 2023 4:40 AM > > To: stable@vger.kernel.org > > Cc: Greg Kroah-Hartman ; > > patches@lists.linux.dev; Haiyang Zhang ; Simon > > Horman ; Shradha Gupta > > ; Paolo Abeni ; > > Sasha Levin > > Subject: [PATCH 6.1 001/131] net: mana: Fix TX CQE error handling > > > > 6.1-stable review patch. If anyone has any objections, please let me know. > > > > ------------------ > > > > From: Haiyang Zhang > > > > [ Upstream commit b2b000069a4c307b09548dc2243f31f3ca0eac9c ] > > > > For an unknown TX CQE error type (probably from a newer hardware), > > still free the SKB, update the queue tail, etc., otherwise the > > accounting will be wrong. > > > > Also, TX errors can be triggered by injecting corrupted packets, so > > replace the WARN_ONCE to ratelimited error logging. > > > > Cc: stable@vger.kernel.org > > Fixes: ca9c54d2d6a5 ("net: mana: Add a driver for Microsoft Azure Network > > Adapter (MANA)") > > Signed-off-by: Haiyang Zhang > > Reviewed-by: Simon Horman > > Reviewed-by: Shradha Gupta > > Signed-off-by: Paolo Abeni > > Signed-off-by: Sasha Levin > > --- > > drivers/net/ethernet/microsoft/mana/mana_en.c | 16 ++++++++++------ > > 1 file changed, 10 insertions(+), 6 deletions(-) > > > > diff --git a/drivers/net/ethernet/microsoft/mana/mana_en.c > > b/drivers/net/ethernet/microsoft/mana/mana_en.c > > index 4f4204432aaa3..23ce26b8295dc 100644 > > --- a/drivers/net/ethernet/microsoft/mana/mana_en.c > > +++ b/drivers/net/ethernet/microsoft/mana/mana_en.c > > @@ -1003,16 +1003,20 @@ static void mana_poll_tx_cq(struct mana_cq > > *cq) > > case CQE_TX_VPORT_IDX_OUT_OF_RANGE: > > case CQE_TX_VPORT_DISABLED: > > case CQE_TX_VLAN_TAGGING_VIOLATION: > > - WARN_ONCE(1, "TX: CQE error %d: ignored.\n", > > - cqe_oob->cqe_hdr.cqe_type); > > + if (net_ratelimit()) > > + netdev_err(ndev, "TX: CQE error %d\n", > > + cqe_oob->cqe_hdr.cqe_type); > > + > > break; > > > > default: > > - /* If the CQE type is unexpected, log an error, assert, > > - * and go through the error path. > > + /* If the CQE type is unknown, log an error, > > + * and still free the SKB, update tail, etc. > > */ > > - WARN_ONCE(1, "TX: Unexpected CQE type %d: HW > > BUG?\n", > > - cqe_oob->cqe_hdr.cqe_type); > > + if (net_ratelimit()) > > + netdev_err(ndev, "TX: unknown CQE > > type %d\n", > > + cqe_oob->cqe_hdr.cqe_type); > > + > > return; > > This should be changed to "break", because we should "still free the SKB, update > the queue tail, etc., otherwise the accounting will be wrong": Is that an issue in Linus's tree, or is this unique to the stable backport? thanks, greg k-h