From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id F393726E175 for ; Mon, 16 Feb 2026 09:58:47 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771235928; cv=none; b=teiL1TfW1xLmL5wnmbgGiSVPIO7svrMTN0WuDYjCZ33QAXIRV6Me3TPascaqpi5WOKvo1+shYLZI4Id2MjkBqti0o+76o7nFBUw6KIehwu4Jw+xLoUmQ1TVaRXoTMBNyB2TnDmhg/A08SIdpTUtxMzmNvuMSPlmBdNf7M0FlRGw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1771235928; c=relaxed/simple; bh=xLnxBPoxz7jY7mQDTiHj1tRiku2s/hDA7dXnusnpYro=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=Fk28yCkEWkrcWl32U9aGcpc5N58xbqWPsrBFgXceBCZnnA+gzN9A9xQ3ubk+pYakl946QGUoin891kHJamgsDtqtHK3CXsfCCmT0VcdA7vhWg5vzNGaWOr6Zy7B0fbaFvC5qkl/QoHU+vrGmFrGHbboZM6RfQ3H++Eq9Iey0hNg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=hjIhysGO; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="hjIhysGO" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 26BBFC116C6; Mon, 16 Feb 2026 09:58:46 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1771235927; bh=xLnxBPoxz7jY7mQDTiHj1tRiku2s/hDA7dXnusnpYro=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=hjIhysGOyKetYaC2yHCgtw+7W0pXqLm2SYiYuh0LmJ3H6JUa3UGwJM0I7YpOI0PlG bWP7qIbEvEyJbEYo+wZ2M52QRbOBoHrZFj65bvagan2KGwVonl1ToxfQ5hdBImXAAr 6dGLeW7GCX8RD8Q31TyXr3L8TaWoXfMu5P41NpcA= Date: Mon, 16 Feb 2026 10:58:44 +0100 From: Greg KH To: Joshua Washington Cc: stable@vger.kernel.org, Ankit Garg , Jordan Rhee , Harshitha Ramamurthy , Paolo Abeni Subject: Re: [PATCH 6.6.y] gve: defer interrupt enabling until NAPI registration Message-ID: <2026021654-catsup-occupier-6753@gregkh> References: <20260213211702.447894-1-joshwash@google.com> <20260213211702.447894-4-joshwash@google.com> Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20260213211702.447894-4-joshwash@google.com> On Fri, Feb 13, 2026 at 01:17:02PM -0800, Joshua Washington wrote: > From: Ankit Garg > > [ Upstream commit 3d970eda003441f66551a91fda16478ac0711617 ] > > Currently, interrupts are automatically enabled immediately upon > request. This allows interrupt to fire before the associated NAPI > context is fully initialized and cause failures like below: > > [ 0.946369] Call Trace: > [ 0.946369] > [ 0.946369] __napi_poll+0x2a/0x1e0 > [ 0.946369] net_rx_action+0x2f9/0x3f0 > [ 0.946369] handle_softirqs+0xd6/0x2c0 > [ 0.946369] ? handle_edge_irq+0xc1/0x1b0 > [ 0.946369] __irq_exit_rcu+0xc3/0xe0 > [ 0.946369] common_interrupt+0x81/0xa0 > [ 0.946369] > [ 0.946369] > [ 0.946369] asm_common_interrupt+0x22/0x40 > [ 0.946369] RIP: 0010:pv_native_safe_halt+0xb/0x10 > > Use the `IRQF_NO_AUTOEN` flag when requesting interrupts to prevent auto > enablement and explicitly enable the interrupt in NAPI initialization > path (and disable it during NAPI teardown). > > This ensures that interrupt lifecycle is strictly coupled with > readiness of NAPI context. > > Cc: stable@vger.kernel.org > Fixes: 893ce44df565 ("gve: Add basic driver framework for Compute Engine Virtual NIC") Why did you change the Fixes line here? Did the original commit lie about it? If so, that's fine, but this is really going to cause tools a mess to keep track of... > Signed-off-by: Ankit Garg > Reviewed-by: Jordan Rhee > Signed-off-by: Harshitha Ramamurthy > Link: https://patch.msgid.link/20251219102945.2193617-1-hramamurthy@google.com > Signed-off-by: Paolo Abeni > Signed-off-by: Joshua Washington > --- > > Note: This patch has been modified form the original to re-introduce the > irq member to struct gve_notify_block, which was introuduced in commit > 9a5e0776d11f ("gve: Avoid rescheduling napi if on wrong cpu"). Can you put this in a "comment" above your signed off like: Signed-off-by: Paolo Abeni [ modified to re-introduce the irq member to struct gve_notify_block, which was introuduced in commit 9a5e0776d11f ("gve: Avoid rescheduling napi if on wrong cpu"). ] Signed-off-by: Joshua Washington Also, it's "from", not "form" :) Same for all of the other backports here, can you fix them all up please and send a v2? thanks, greg k-h