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=-8.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS,USER_AGENT_GIT autolearn=ham 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 ECC4DC433DF for ; Wed, 26 Aug 2020 19:40:16 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id BAD322078D for ; Wed, 26 Aug 2020 19:40:16 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598470816; bh=ARCsePM4U5cyGQ9q5wju6jf2KfGwFsdgiSLmv/UNiHs=; h=From:To:Cc:Subject:Date:List-ID:From; b=n1u30MDayBhfYavB4PANZU0Bp6tPBhlo6eYfxz24HCnk2JkangQxvXqEJ3Jf8mRY3 mM3axvlZDAxlk0DzDtMusBXEkqCqazmQpyb3TsQpM4889FVb/ceYvZIqI4DZQdubff N8XilS9g/3ZSay71WSRbTDdug9fudVd0FVr8ZSB8= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727017AbgHZTkP (ORCPT ); Wed, 26 Aug 2020 15:40:15 -0400 Received: from mail.kernel.org ([198.145.29.99]:49436 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726753AbgHZTkL (ORCPT ); Wed, 26 Aug 2020 15:40:11 -0400 Received: from kicinski-fedora-PC1C0HJN.thefacebook.com (unknown [163.114.132.4]) (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 5F10B2076C; Wed, 26 Aug 2020 19:40:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598470810; bh=ARCsePM4U5cyGQ9q5wju6jf2KfGwFsdgiSLmv/UNiHs=; h=From:To:Cc:Subject:Date:From; b=gSdxup9OjHoCBLMpN/XCh0FMkwP1ZXrabGp04v8EaWkpu0yXC1xERAO3jFJcXRBDj VA4V5xduBPzSulmdqC/hEncnT8MwVmvOl8TeXc7Ui6dEPXelES7154wyZxjXU3ydS5 H2cv4ilepDcwakAck7YT3zQ4YopS/rgXgxln6Y8M= From: Jakub Kicinski To: davem@davemloft.net Cc: eric.dumazet@gmail.com, michael.chan@broadcom.com, netdev@vger.kernel.org, kernel-team@fb.com, Jakub Kicinski Subject: [PATCH net 0/2] net: fix netpoll crash with bnxt Date: Wed, 26 Aug 2020 12:40:05 -0700 Message-Id: <20200826194007.1962762-1-kuba@kernel.org> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org Hi! Rob run into crashes when using XDP on bnxt. Upon investigation it turns out that during driver reconfig irq core produces a warning message when IRQs are requested. This triggers netpoll, which in turn accesses uninitialized driver state. Same crash can also be triggered on this platform by changing the number of rings. Looks like we have two missing pieces here, netif_napi_add() has to make sure we start out with netpoll blocked. The driver also has to be more careful about when napi gets enabled. Tested XDP and channel count changes, the warning message no longer causes a crash. Not sure if the memory barriers added in patch 1 are necessary, but it seems we should have them. Jakub Kicinski (2): net: disable netpoll on fresh napis bnxt: don't enable NAPI until rings are ready drivers/net/ethernet/broadcom/bnxt/bnxt.c | 12 ++++-------- net/core/dev.c | 3 ++- net/core/netpoll.c | 2 +- 3 files changed, 7 insertions(+), 10 deletions(-) -- 2.26.2