Received: by 2002:a25:ad19:0:0:0:0:0 with SMTP id y25csp10425633ybi; Wed, 24 Jul 2019 23:00:42 -0700 (PDT) X-Google-Smtp-Source: APXvYqzh5BtCmMswfqtL1u54CMMsm7YqATCx4htNkbrbAJCiifBA6f0G79/fCHKpI+2Pn9aoXki5 X-Received: by 2002:a63:d315:: with SMTP id b21mr61136442pgg.326.1564034441747; Wed, 24 Jul 2019 23:00:41 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1564034441; cv=none; d=google.com; s=arc-20160816; b=Fb6CSzj8iv5T0YWFu40uMsYZ8PyyWYJMz2DgvPU0He0pPeI1jjvpactUOWAaAurs5q cWlsZ+VdBw/a2IgKcQ4EQS6/Pj4JcKMMW9YY/Y3gg9NNQof8foId1KrmVP7Aj0n0NkzO eXvEXfRHA4BfKwjNx46zK4IptD6tkNuZwDGvWJdHO1kTRXx5XZnjwUDDN7npD3rUBO2M XYCLi9SPcmGIYPVfh1L1QAnit2xwgcKN0j4mI6uhgSTm+nbu8qSLgZuFtCR71T01rwNR HSRNjshChn1plewOOJ7kep8JvZHbfqo6d7/6aJx0q5Sc5Jbsm2EQRTyLsLibyuME71bW jJwQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:content-transfer-encoding:mime-version :user-agent:references:in-reply-to:message-id:date:subject:cc:to :from:dkim-signature; bh=LHeb32hWOQFEzzBwQ7ezG3z0AAPGJDhbVRRve0R04As=; b=q5O9pOVSfAJS/DrcT+wOIMRB4I1HqwxBG85Y0Wx5QZevq55MSEQ25J4wcTsQhWEuJO KUREZrkKValJrzlbVxxNn5RxMVxgazNV+/cTNuVaBCud/Tbg/+inOs+T7BMczN7t8I7G 0bkCe4bbFM6EExPAZbkos3+0flA7f09Fk0gD+GGeg+a8U2pR8lUVkR29UEQEC8nZfRci d5ppC8DF5n+eLls7HVCdugKxscjH53jhenS7omz9VN7JN5rfRJC8+kBX9CIB7LrvbbNX xZfaUwXvudQwFPr6Oy5He8bK7rc6hOTA9NcAkaA/dWpZqEFV26rHsxROcvPUubYOdiXr xaCQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=b5XoCRgZ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id u38si15846761pgl.507.2019.07.24.23.00.26; Wed, 24 Jul 2019 23:00:41 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@kernel.org header.s=default header.b=b5XoCRgZ; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2391020AbfGYFnX (ORCPT + 99 others); Thu, 25 Jul 2019 01:43:23 -0400 Received: from mail.kernel.org ([198.145.29.99]:58296 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404913AbfGYFnW (ORCPT ); Thu, 25 Jul 2019 01:43:22 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 8E08C21880; Thu, 25 Jul 2019 05:43:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1564033401; bh=jCpDUQUInq4d0iQrHrClBp/uYmfBF+WVwvvFfEEPLIg=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=b5XoCRgZ0n0x4VWiJzPM6cSqSlvGy1K9Vud72RKpIUMwFGg41lIpRy7r3mJ4EPeyN uL3CaQRhXK7KYhzlxJcxiZfT//t/HOZvD1M18eFY2rF27IXJnLcsCwBi/fRMpvtqIX ppKX4RH1XXOHWeSod/+e/Wj0mvaBg+PFyzmNvKzk= From: Greg Kroah-Hartman To: linux-kernel@vger.kernel.org Cc: Greg Kroah-Hartman , stable@vger.kernel.org, Emmanuel Grumbach , Luca Coelho Subject: [PATCH 4.19 195/271] iwlwifi: pcie: dont service an interrupt that was masked Date: Wed, 24 Jul 2019 21:21:04 +0200 Message-Id: <20190724191711.811887990@linuxfoundation.org> X-Mailer: git-send-email 2.22.0 In-Reply-To: <20190724191655.268628197@linuxfoundation.org> References: <20190724191655.268628197@linuxfoundation.org> User-Agent: quilt/0.66 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Emmanuel Grumbach commit 3b57a10ca14c619707398dc58fe5ece18c95b20b upstream. Sometimes the register status can include interrupts that were masked. We can, for example, get the RF-Kill bit set in the interrupt status register although this interrupt was masked. Then if we get the ALIVE interrupt (for example) that was not masked, we need to *not* service the RF-Kill interrupt. Fix this in the MSI-X interrupt handler. Cc: stable@vger.kernel.org Signed-off-by: Emmanuel Grumbach Signed-off-by: Luca Coelho Signed-off-by: Greg Kroah-Hartman --- drivers/net/wireless/intel/iwlwifi/pcie/rx.c | 27 +++++++++++++++++++++------ 1 file changed, 21 insertions(+), 6 deletions(-) --- a/drivers/net/wireless/intel/iwlwifi/pcie/rx.c +++ b/drivers/net/wireless/intel/iwlwifi/pcie/rx.c @@ -2060,10 +2060,18 @@ irqreturn_t iwl_pcie_irq_msix_handler(in return IRQ_NONE; } - if (iwl_have_debug_level(IWL_DL_ISR)) - IWL_DEBUG_ISR(trans, "ISR inta_fh 0x%08x, enabled 0x%08x\n", - inta_fh, + if (iwl_have_debug_level(IWL_DL_ISR)) { + IWL_DEBUG_ISR(trans, + "ISR inta_fh 0x%08x, enabled (sw) 0x%08x (hw) 0x%08x\n", + inta_fh, trans_pcie->fh_mask, iwl_read32(trans, CSR_MSIX_FH_INT_MASK_AD)); + if (inta_fh & ~trans_pcie->fh_mask) + IWL_DEBUG_ISR(trans, + "We got a masked interrupt (0x%08x)\n", + inta_fh & ~trans_pcie->fh_mask); + } + + inta_fh &= trans_pcie->fh_mask; if ((trans_pcie->shared_vec_mask & IWL_SHARED_IRQ_NON_RX) && inta_fh & MSIX_FH_INT_CAUSES_Q0) { @@ -2103,11 +2111,18 @@ irqreturn_t iwl_pcie_irq_msix_handler(in } /* After checking FH register check HW register */ - if (iwl_have_debug_level(IWL_DL_ISR)) + if (iwl_have_debug_level(IWL_DL_ISR)) { IWL_DEBUG_ISR(trans, - "ISR inta_hw 0x%08x, enabled 0x%08x\n", - inta_hw, + "ISR inta_hw 0x%08x, enabled (sw) 0x%08x (hw) 0x%08x\n", + inta_hw, trans_pcie->hw_mask, iwl_read32(trans, CSR_MSIX_HW_INT_MASK_AD)); + if (inta_hw & ~trans_pcie->hw_mask) + IWL_DEBUG_ISR(trans, + "We got a masked interrupt 0x%08x\n", + inta_hw & ~trans_pcie->hw_mask); + } + + inta_hw &= trans_pcie->hw_mask; /* Alive notification via Rx interrupt will do the real work */ if (inta_hw & MSIX_HW_INT_CAUSES_REG_ALIVE) {