From patchwork Mon Sep 13 13:01:32 2021 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= X-Patchwork-Id: 510241 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-16.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_CR_TRAILER, INCLUDES_PATCH, MAILING_LIST_MULTI, MSGID_FROM_MTA_HEADER, SPF_HELO_NONE, SPF_PASS, URIBL_BLOCKED, USER_AGENT_GIT autolearn=unavailable autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id CB186C433FE for ; Mon, 13 Sep 2021 13:02:24 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B1403610A3 for ; Mon, 13 Sep 2021 13:02:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236918AbhIMNDh (ORCPT ); Mon, 13 Sep 2021 09:03:37 -0400 Received: from mail-dm6nam12on2088.outbound.protection.outlook.com ([40.107.243.88]:14912 "EHLO NAM12-DM6-obe.outbound.protection.outlook.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S236893AbhIMNDf (ORCPT ); Mon, 13 Sep 2021 09:03:35 -0400 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SEUwUVH5b8oOJgqnH7rhS5BXiYv0lhkWJYQevswXm6GgtlyMjX38N1ISwKWqNHsLp5blm4eEtePLhFuLD+93oPsYsYlJB7HQZDo4AgRjKlcvmO8hhJvxYL0vI8nY6K/2uWjM/I6n/39cFTKMF1LlretTLPyOVWQxwiIREeGcJV9IFMbH56rZhVlkq5rq301twTZvigIUc82I74epbaeVVOSYOlAZOqtHMxd1F7zVADQ+MakRSc2zC7LTip0//jCLALVGuoLGv68LTWTRhvBtzRt7+0wMzmr0ZM/QxBvIAVOZ++RkTE3pQNX8QN/kBqSY1avWLoHLWLqoGxHaNL4TqQ== 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; bh=Fib8Y8j+Fw/sN2NX69u9T+P9uNpyVHksmDfyGgxLK4M=; b=C97lMExusJSKCp9gP1nmzDJzwL/S2zWLnBp7i587Jz3HuNOQeiVjr3BCXENieqZ5h+liz8W/Ph913e/IcuaNKMGEsn2BsRexY1rwfGLKbLlRzcpqCwEl6ohMn1KQs9pNpoUzg0Y32EHvBYsqP9t/cAtPCEDXWl2tc+iivPLqpQeaxDy8iDSbWWZpDFLFUkuNsDNbaAdD4udGam4qTpfF+eKmrcj7/2yfQ2HnX06e8FYN3K0itxRMYQQCqsfYhHTnTTpbiz0GHEimDLwvXBUqWNqm2HBakBnBGzCyYVOWRHf/Pzq8xl4oXXVBqgAOUa75/YGsmHrMUYllWhyCc0RGig== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=silabs.com; dmarc=pass action=none header.from=silabs.com; dkim=pass header.d=silabs.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=silabs.onmicrosoft.com; s=selector2-silabs-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Fib8Y8j+Fw/sN2NX69u9T+P9uNpyVHksmDfyGgxLK4M=; b=dIKNzk7yEJ2Zx2dqAAi4U7B5fphy7OuyV/eX/DaLEeEOWBFZkBP/tkJCNHyy+0U3It1hmGi6ijd3F+9/P9Xw4kPVLIw1mkDX4YbT846v4duBX9uBin2k3nHizRI+B83zWAKEeI5qhNRyDVN1MSzN7sEQliNXY8fKN38okHQqET8= Authentication-Results: driverdev.osuosl.org; dkim=none (message not signed) header.d=none; driverdev.osuosl.org; dmarc=none action=none header.from=silabs.com; Received: from SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) by SN6PR11MB2894.namprd11.prod.outlook.com (2603:10b6:805:d7::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.17; Mon, 13 Sep 2021 13:02:18 +0000 Received: from SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd]) by SN6PR11MB2718.namprd11.prod.outlook.com ([fe80::7050:a0a:415:2ccd%7]) with mapi id 15.20.4500.017; Mon, 13 Sep 2021 13:02:18 +0000 From: Jerome Pouiller To: devel@driverdev.osuosl.org, linux-wireless@vger.kernel.org Cc: netdev@vger.kernel.org, linux-kernel@vger.kernel.org, Greg Kroah-Hartman , Kalle Valo , "David S . Miller" , =?utf-8?b?SsOpcsO0bWUgUG91aWxsZXI=?= Subject: [PATCH v3 01/32] staging: wfx: use abbreviated message for "incorrect sequence" Date: Mon, 13 Sep 2021 15:01:32 +0200 Message-Id: <20210913130203.1903622-2-Jerome.Pouiller@silabs.com> X-Mailer: git-send-email 2.33.0 In-Reply-To: <20210913130203.1903622-1-Jerome.Pouiller@silabs.com> References: <20210913130203.1903622-1-Jerome.Pouiller@silabs.com> X-ClientProxiedBy: PR2P264CA0021.FRAP264.PROD.OUTLOOK.COM (2603:10a6:101::33) To SN6PR11MB2718.namprd11.prod.outlook.com (2603:10b6:805:63::18) MIME-Version: 1.0 Received: from pc-42.silabs.com (37.71.187.125) by PR2P264CA0021.FRAP264.PROD.OUTLOOK.COM (2603:10a6:101::33) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4500.14 via Frontend Transport; Mon, 13 Sep 2021 13:02:17 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 581b051f-27c1-45e9-1842-08d976b6b794 X-MS-TrafficTypeDiagnostic: SN6PR11MB2894: X-MS-Exchange-Transport-Forked: True X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:4125; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: 6PUY9Vz078mPn5SaCxoDe+OaiqDgNVHo7XtaekJ7fnvikufd6ylMWQx0ONDnLXeDNawfYh/nhakjXv+cE0Q9QHfXYg03GmkdYT735AdtATheORquw1NwIA8h6VZMHhVI5Q/k8vC4dt0hLlVM3I7QG8VLGgN3NRXF0aC7GlfQG8f9RCLWJ8olT1QZ0zYCLsCfnFLS05H6hE36rNNESrBB9zPoW571IwgVi4yOiu9fnQTprsQ6yBf73TLPPnxIRyuW7C+t1JtWwcjgdzV0jwFH4cRWpHSINW9Os57fCdUcnELHAyBOnhmWKg9W4s9Vwc1tPDbbV+ecnfWHR9UlsKslPvaWG8hs2uwgXoJpl4/ZiqHpXY2ozBClbZ91BfsOaMvu6uy5Z0RmvZyFhb/eJSuV8IMpWuaJ9SVRFI13Iozk+fjDmEfQbaDXFncJnJxljBSdYUcAPAA3aVXBTZYcRvnAiFAcpBFK6xIjsG53J4vk7xStjC8onjcOkvir7E4w+dSJm8d7kmkQ72jWDvghbqXqSDojjcrPv2JJ5l1cSVvfDTfdUCpC4cDJtWld1XABxq8cRFUU70w/+UObS5AdlNcVbkgTaoGCYGxcSjv1SCTG/PKcCx14xQF/RJTlpU3lSxyReYDe3kH3z+9dCt4nCxW5j7YdiRHjgXm/3/CJQeGYVyauo06qCipJnushO9jxXQR/Jt+tIwFj6LzA5mGG2n5aeA== X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SN6PR11MB2718.namprd11.prod.outlook.com; PTR:; CAT:NONE; SFS:(396003)(366004)(136003)(39850400004)(346002)(376002)(5660300002)(26005)(8936002)(6486002)(38350700002)(38100700002)(66946007)(7696005)(186003)(316002)(1076003)(8676002)(956004)(478600001)(2616005)(2906002)(52116002)(4326008)(36756003)(6666004)(15650500001)(54906003)(83380400001)(107886003)(66476007)(66556008)(86362001)(66574015); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?N8/Zr4Tzwtfp3oybsTTexAP0D?= =?utf-8?q?juOPQYRFCd9lrfnPEjX6UxB6sZ42H1cZyA25p0GkyP0Xq66cxK2xuTeV?= =?utf-8?q?UDpCX/ToqXVhyoJfVEDaitntWKgXlmRGU22EoP6yWYVTF8QASId0Qgbh?= =?utf-8?q?/U7U7Wj401J4C6ecNMPLp/g3Yx5yVH/JaPbocazAqS6ylaZkMmlL5Vd1?= =?utf-8?q?89GzrdyH/tw2+5faI+j7PsdnbY7fOQci2Qq50R6onUgigj/wAvIasoVr?= =?utf-8?q?qcReeK4KQXwURwVsQskXjua0ZRryEaJwgcCbQ26iCl/fVge/V6OwZvjX?= =?utf-8?q?NOlTLQXaeMl/COU7Lv2nS3KIS1MVYNlzry501uZcl43e4vy0IJsLNJUF?= =?utf-8?q?bi+bgZnbJEmkO+BNFFwSYOZP6IdHs5hSU2HQ8oeG4ZDNpaW4sqiAHSap?= =?utf-8?q?HncYf0u9mA67TOmlISe73b63b68t4CNIsdSLQ63PoPACm17Z1KY0DPGJ?= =?utf-8?q?NRhUHMjqgbAq7JsPdyrJxi7nCbFDd3XDNir+4JLh/sEiYAVqDJEEdZhU?= =?utf-8?q?Kqcoyl25yyke8BJVNs6n2R4v0xfbYEvmdVzbq+fzbgFWtmCfY4kHNJ36?= =?utf-8?q?y8f08Yel/I2cNy2NMRaXR91fAZ2LRwxaXHZaDWcQb2I6v5Lzk7u8ue8D?= =?utf-8?q?ZzbVksWGUZ3PFoKYuY3sWitTBgBsqEDNrIRyq/7XPcoD+Iy6WKjapWg6?= =?utf-8?q?KtKQJhH3QmCRRoiLZADmS3fZtde2B+X1RZANpm2jbJzOWCfhsSTIvb78?= =?utf-8?q?e7s92aM2sh0ppf4+kiIh8aT6rUt+h0ymG2cqq9EtisuGer4DFt1ohS0E?= =?utf-8?q?UmTX76/fV9KGBaA8i8XHM0aB0ju1nIhsLSIq4GfiB6fK+zUTWXmgUq37?= =?utf-8?q?vi798yt+4VrC/ojlhN7QEnnyh6JcJh/TkFrRVdT8AK9puxC+nAx7JjdM?= =?utf-8?q?dVr0RxDiYuk/GWZPRv1nTbSItuoH+Aknsar/nX9qkk4pE5uDio2pi3Gk?= =?utf-8?q?pNCiqHeReoqY51QRy2XVEUsu/v7fEPBnv4OBAjwTsqjlRorJ/typxu11?= =?utf-8?q?zP2djSOpm/wI1sGXqh2NnArTv03hFY9j1cWA7t4V290Irlx6e35SqaKE?= =?utf-8?q?ktb5zinncsdLiL16pPUxU5a4ydnbXLehuyxhu/ONdPWMjRinmom2K0yQ?= =?utf-8?q?2Es+HAlqnY1vPsOe9r1cSgrNot2b+LjCPtgtrxtdNRX5hj8zaCujEhhg?= =?utf-8?q?K4etIkczxhkFumLWtJikc+LF6meU8GBKUMf6kneEMDOFWu4o+EjdQ79P?= =?utf-8?q?67cM3cqztuP0q5+WH5BOZb+YANAszARlvrBjmo74pnrDpfw9aucZg5xm?= =?utf-8?q?oLt7atBumTZ8IQDs5nQU/DHEeDQha22?= X-OriginatorOrg: silabs.com X-MS-Exchange-CrossTenant-Network-Message-Id: 581b051f-27c1-45e9-1842-08d976b6b794 X-MS-Exchange-CrossTenant-AuthSource: SN6PR11MB2718.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 13 Sep 2021 13:02:18.7776 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 54dbd822-5231-4b20-944d-6f4abcd541fb X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: +kimGIzapP8qXiru8Rs7FYQD4Qy89CHpQ2ZrsaQU0wUXslk91+WDI8An6gIOguVpnKLa5JlAtVHG7gtCPywFoA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN6PR11MB2894 Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org From: Jérôme Pouiller The wfx driver checks carefully the coherency of of the DTIM notifications. We have noticed several times some small inconsistencies from the firmware on these notification. They have never been critical. However on the driver side they lead to big fat warnings. Worse, if these warning are displayed on UART console, they can be long to display (several hundreds of millisecs). Since, this warning is generated from a work queue, it can delay all the workqueue users. Especially, it can drastically slow down the frame management of the driver and then generate errors that are serious this time (eg. an overflow of the indication queue of the device). Signed-off-by: Jérôme Pouiller --- drivers/staging/wfx/sta.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/staging/wfx/sta.c b/drivers/staging/wfx/sta.c index cb7e8abdf43c..a236e5bb6914 100644 --- a/drivers/staging/wfx/sta.c +++ b/drivers/staging/wfx/sta.c @@ -631,8 +631,9 @@ void wfx_suspend_resume_mc(struct wfx_vif *wvif, enum sta_notify_cmd notify_cmd) { if (notify_cmd != STA_NOTIFY_AWAKE) return; - WARN(!wfx_tx_queues_has_cab(wvif), "incorrect sequence"); - WARN(wvif->after_dtim_tx_allowed, "incorrect sequence"); + if (!wfx_tx_queues_has_cab(wvif) || wvif->after_dtim_tx_allowed) + dev_warn(wvif->wdev->dev, "incorrect sequence (%d CAB in queue)", + wfx_tx_queues_has_cab(wvif)); wvif->after_dtim_tx_allowed = true; wfx_bh_request_tx(wvif->wdev); }