From patchwork Sat May 19 15:53:22 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Salil Mehta X-Patchwork-Id: 136400 Delivered-To: patch@linaro.org Received: by 2002:a2e:9706:0:0:0:0:0 with SMTP id r6-v6csp2546407lji; Sat, 19 May 2018 08:55:39 -0700 (PDT) X-Google-Smtp-Source: AB8JxZqdUGjXBj4AQ6mfWkqfAH0Jep9eRQCB+YJol+NLvXSUgcXElQ70B211DsdZB3dAIsTx0o3r X-Received: by 2002:a17:902:7b83:: with SMTP id w3-v6mr14245194pll.12.1526745339806; Sat, 19 May 2018 08:55:39 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1526745339; cv=none; d=google.com; s=arc-20160816; b=zgDTe/Rqw47MOKEpw3vk1j/Fntv3Bp5nQK2wVweVuhH+bKodJlXtM45CIgNTS+xvUr dKo6XSjR2pkjTM5zk7zo3NR1/vfATprjHOLil8RwR/6XQ90xx0hMzcdVsz7Qv7ZD0Z4E H32dj5G5rRlnqKuZj5J2l1+690QlPEN/Xgv+PQNcfA2FEWi3icD3NzNWTyp8EOpVPWfc GOBlva/xrKEH04MLKWy7R6yc2CIFqU3MbKxpVojjjJ1X86Wfe+8YE13RGz2GK/4UK9+6 7TuVDysCjLjoECuzU60qEVbcW2KnLGMgVtirxMcKAG57N+lHBSmWHD+fVkKOUtnAR+x2 b+Lg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:arc-authentication-results; bh=sKQrlraOg0D2qQPS38SXuAK3xMcYo64dWkLFbgEKUHo=; b=HV4aih6aqYBZWQwbL8QxmcG9hxRaMGvSIidvScZ7lhD9BH/cRGD/TS7xrWZNruC5vP ekPL4p+FZ6W8V/fhpj3JdEf9CKbQbGZp3HXLpFr66BdPRTQkacWMb6LTW0lsEIkWqOm+ HcY5R5zMB27d6I+GUdAf+ZEjfCqAOUR1qLDhrw0QtQBLOsUMvQt1nNh0kXH3SSolD4Rv SibLpFwYaDpMZY6Cfac0lpSFsOWgSgPcWgS1LcdN6iXS0q0YWQUZP+fUAy0Y5oat29Cw gTICcTKUqUy+nzDCKU91S8dt5NpkAlALuTiW4rq1myibvdGN7hSdzJ9ATlJkF+LFyWd/ 5EkQ== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of netdev-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=netdev-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 m12-v6si8000415pgr.506.2018.05.19.08.55.39; Sat, 19 May 2018 08:55:39 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of netdev-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of netdev-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=netdev-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752658AbeESPzi (ORCPT + 9 others); Sat, 19 May 2018 11:55:38 -0400 Received: from szxga04-in.huawei.com ([45.249.212.190]:8125 "EHLO huawei.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1752612AbeESPy6 (ORCPT ); Sat, 19 May 2018 11:54:58 -0400 Received: from DGGEMS403-HUB.china.huawei.com (unknown [172.30.72.58]) by Forcepoint Email with ESMTP id 5B8249DD689BD; Sat, 19 May 2018 23:54:43 +0800 (CST) Received: from S00293818-DELL1.china.huawei.com (10.202.227.234) by DGGEMS403-HUB.china.huawei.com (10.3.19.203) with Microsoft SMTP Server id 14.3.382.0; Sat, 19 May 2018 23:54:37 +0800 From: Salil Mehta To: CC: , , , , , , , Xi Wang Subject: [PATCH net-next 8/9] net: hns3: Fixes kernel panic issue during rmmod hns3 driver Date: Sat, 19 May 2018 16:53:22 +0100 Message-ID: <20180519155323.68960-9-salil.mehta@huawei.com> X-Mailer: git-send-email 2.8.3 In-Reply-To: <20180519155323.68960-1-salil.mehta@huawei.com> References: <20180519155323.68960-1-salil.mehta@huawei.com> MIME-Version: 1.0 X-Originating-IP: [10.202.227.234] X-CFilter-Loop: Reflected Sender: netdev-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org From: Xi Wang If CONFIG_ARM_SMMU_V3 is enabled, arm64's dma_ops will replace arm64_swiotlb_dma_ops with iommu_dma_ops. When releasing contiguous dma memory, the new ops will call the vunmap function which cannot be run in interrupt context. Currently, spin_lock_bh is called before vunmap is executed. This disables BH and causes the interrupt context to be detected to generate a kernel panic like below: [ 2831.573400] kernel BUG at mm/vmalloc.c:1621! [ 2831.577659] Internal error: Oops - BUG: 0 [#1] PREEMPT SMP ... [ 2831.699907] Process rmmod (pid: 1893, stack limit = 0x0000000055103ee2) [ 2831.706507] Call trace: [ 2831.708941] vunmap+0x48/0x50 [ 2831.711897] dma_common_free_remap+0x78/0x88 [ 2831.716155] __iommu_free_attrs+0xa8/0x1c0 [ 2831.720255] hclge_free_cmd_desc+0xc8/0x118 [hclge] [ 2831.725128] hclge_destroy_cmd_queue+0x34/0x68 [hclge] [ 2831.730261] hclge_uninit_ae_dev+0x90/0x100 [hclge] [ 2831.735127] hnae3_unregister_ae_dev+0xb0/0x868 [hnae3] [ 2831.740345] hns3_remove+0x3c/0x90 [hns3] [ 2831.744344] pci_device_remove+0x48/0x108 [ 2831.748342] device_release_driver_internal+0x164/0x200 [ 2831.753553] driver_detach+0x4c/0x88 [ 2831.757116] bus_remove_driver+0x60/0xc0 [ 2831.761026] driver_unregister+0x34/0x60 [ 2831.764935] pci_unregister_driver+0x30/0xb0 [ 2831.769197] hns3_exit_module+0x10/0x978 [hns3] [ 2831.773715] SyS_delete_module+0x1f8/0x248 [ 2831.777799] el0_svc_naked+0x30/0x34 This patch fixes it by using spin_lock instead of spin_lock_bh. Fixes: 68c0a5c70614 ("net: hns3: Add HNS3 IMP(Integrated Mgmt Proc) Cmd Interface Support") Signed-off-by: Xi Wang Signed-off-by: Peng Li Signed-off-by: Salil Mehta --- drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) -- 2.7.4 diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c index 59fb0eb..c36d647 100644 --- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c +++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_cmd.c @@ -385,9 +385,9 @@ int hclge_cmd_init(struct hclge_dev *hdev) static void hclge_destroy_queue(struct hclge_cmq_ring *ring) { - spin_lock_bh(&ring->lock); + spin_lock(&ring->lock); hclge_free_cmd_desc(ring); - spin_unlock_bh(&ring->lock); + spin_unlock(&ring->lock); } void hclge_destroy_cmd_queue(struct hclge_hw *hw)