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 X-Spam-Level: X-Spam-Status: No, score=-3.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no 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 BE19CC433E0 for ; Sat, 13 Mar 2021 17:01:53 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 80DA964E3F for ; Sat, 13 Mar 2021 17:01:53 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234098AbhCMRBV (ORCPT ); Sat, 13 Mar 2021 12:01:21 -0500 Received: from paleale.coelho.fi ([176.9.41.70]:48766 "EHLO farmhouse.coelho.fi" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S233635AbhCMRBN (ORCPT ); Sat, 13 Mar 2021 12:01:13 -0500 X-Greylist: delayed 1718 seconds by postgrey-1.27 at vger.kernel.org; Sat, 13 Mar 2021 12:01:13 EST Received: from 91-156-6-193.elisa-laajakaista.fi ([91.156.6.193] helo=[192.168.100.150]) by farmhouse.coelho.fi with esmtpsa (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.93) (envelope-from ) id 1lL7Bj-004U2G-Pm; Sat, 13 Mar 2021 18:32:28 +0200 Message-ID: From: Luca Coelho To: Jiri Kosina , Kalle Valo Cc: Johannes Berg , Emmanuel Grumbach , linux-wireless@vger.kernel.org, linux-kernel@vger.kernel.org Date: Sat, 13 Mar 2021 18:32:26 +0200 In-Reply-To: References: <87h7lfbowr.fsf@tynnyri.adurom.net> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.38.3-1 MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [PATCH] iwlwifi: Fix softirq/hardirq disabling in iwl_pcie_enqueue_hcmd() Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Sat, 2021-03-13 at 16:43 +0100, Jiri Kosina wrote: > On Sat, 13 Mar 2021, Kalle Valo wrote: > > > > > > From: Jiri Kosina > > > > > > > > > > It's possible for iwl_pcie_enqueue_hcmd() to be called with hard IRQs > > > > > disabled (e.g. from LED core). We can't enable BHs in such a situation. > > > > > > > > > > Turn the unconditional BH-enable/BH-disable code into > > > > > hardirq-disable/conditional-enable. > > > > > > > > > > This fixes the warning below. > > > > > > > > Hi, > > > > > > > > friendly ping on this one ... > > > > > > Luca, > > > > > > Johannes is telling me that he merged this patch internally, but I have no > > > idea what is happening to it ... ? > > > > > > The reported splat is a clear bug, so it should be fixed one way or the > > > other. > > > > Should I take this to wireless-drivers? > > I can't speak for the maintainers, but as far as I am concerned, it > definitely is a 5.12 material, as it fixes real scheduling bug. Yes, please take this to w-d. We have a similar patch internally, but there's a backlog and it will take me some time to get to it. I'll resolve eventual conflicts when time comes. Thanks! -- Cheers, Luca.