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 EAD59E95A99 for ; Mon, 9 Oct 2023 12:52:06 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1346623AbjJIMwG (ORCPT ); Mon, 9 Oct 2023 08:52:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44888 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1346634AbjJIMwF (ORCPT ); Mon, 9 Oct 2023 08:52:05 -0400 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 6939893; Mon, 9 Oct 2023 05:52:03 -0700 (PDT) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 774D0C433C8; Mon, 9 Oct 2023 12:52:02 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1696855923; bh=Gd/aUOkQ3dpFucjV6W8k2UydiSxe2oz68p9fiYKX+K0=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=lYXb87BJOIbZNHpULMLqKQHXJCHAzg69J5v6nPr8DqAzWt92Z0FH1J15o4p07MySj 1dKjpFLOKG8AGkZay3MfkzzdcEHqbQt3tPGIVnr9SxYNhNtBhcsC5+vwkK8KKQbec8 nt/xrOFgtgN6o144blUbzhMfZxnWQTIRQm9eFZjM= Date: Mon, 9 Oct 2023 14:52:00 +0200 From: Greg Kroah-Hartman To: Prashanth K Cc: stable@vger.kernel.org, linux-kernel@vger.kernel.org, Hongyu Xie , Mathias Nyman , stable@kernel.org, Hongyu Xie , Mathias Nyman Subject: Re: [PATCH RESEND] xhci: Keep interrupt disabled in initialization until host is running. Message-ID: <2023100943-underhand-wizard-6901@gregkh> References: <1696847966-27555-1-git-send-email-quic_prashk@quicinc.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1696847966-27555-1-git-send-email-quic_prashk@quicinc.com> Precedence: bulk List-ID: X-Mailing-List: stable@vger.kernel.org On Mon, Oct 09, 2023 at 04:09:26PM +0530, Prashanth K wrote: > From: Hongyu Xie > > [ Upstream commit a808925075fb750804a60ff0710614466c396db4 ] > > irq is disabled in xhci_quiesce(called by xhci_halt, with bit:2 cleared > in USBCMD register), but xhci_run(called by usb_add_hcd) re-enable it. > It's possible that you will receive thousands of interrupt requests > after initialization for 2.0 roothub. And you will get a lot of > warning like, "xHCI dying, ignoring interrupt. Shouldn't IRQs be > disabled?". This amount of interrupt requests will cause the entire > system to freeze. > This problem was first found on a device with ASM2142 host controller > on it. > > [tidy up old code while moving it, reword header -Mathias] > > Cc: stable@kernel.org > Signed-off-by: Hongyu Xie > Signed-off-by: Mathias Nyman > Link: https://lore.kernel.org/r/20220623111945.1557702-2-mathias.nyman@linux.intel.com > Signed-off-by: Greg Kroah-Hartman > Cc: # 5.15 > Signed-off-by: Prashanth K > --- Any specific reason you missed adding the extra blank line in this version of the backport that the original added? That is going to cause problems in the future if other patches are added on top of this that would be expecting it because it is that way in Linus's tree. And why is this only relevant for 5.15.y? thanks, greg k-h