Received: by 2002:ac0:a5a6:0:0:0:0:0 with SMTP id m35-v6csp223178imm; Wed, 29 Aug 2018 19:10:01 -0700 (PDT) X-Google-Smtp-Source: ANB0VdbAu+nzn4cw2a2CQYhwx0S3minkjt+hFeLmLN+nqFqn+lQ2QWR2miSsSm1WaMJuZK9arTUN X-Received: by 2002:a17:902:ba83:: with SMTP id k3-v6mr8003197pls.251.1535595001649; Wed, 29 Aug 2018 19:10:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1535595001; cv=none; d=google.com; s=arc-20160816; b=xjFSEj5GMcwnbKsqnOYUn7yoiMawIBxHE0Rwrvb07zBuzpU51l4SEA/9/HiqVrjl/u 7ZXTgunhIraI++3cT4u217HVUDclf/gZz3DXLEeMyQ4GLAQ3k01CsG+NH0aA3NCBnNR4 q70v8A9aGA5y0O9g6hYeVeiXRf+e+OQfNucTulyGMttsmW7lOAT0nMUbDF+6pXx5QFhN JP9aBR4b7HdAxych+cIhf8Qou1saULKZuPae2Eqcd0WUyoLQ9dX8e6KJPyx55fFhEmh1 kqfDwh9Bo/Gd3/tg41PGyucz00HEHw7Zc56H3ijgOEyA/Lc1G0txe5v45AonOGWnRrJa IbjQ== 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 :references:in-reply-to:from:subject:cc:to:message-id:date :arc-authentication-results; bh=T2dkGcbfsJ0UIv/hh7evambk+u1iL4RDhotptNl24Ng=; b=jf/90aUQKd53pjzLRODiw/MrmZDFoWp4Bxy3caSrstWoP9cFmqVszOh8JOHgUbr9bL w8VysMtidyf9q8xi0WiWZHLPjiTr8D9HnLONW+JdWPGrzLWs18wZsUOXp45kOwz4f3Qe 6RpEHo5fHtTBbhjVrgBskzuFCUG5XLhroDy7RRqC1rPoe68DcNpiTELRghBeUdr3tGeh VyErXTnmnq5RusWfajPj60zeIRCMVb865ZdQdAFlyuL2dsCW2Ebnn1BxN3LA5xX3pq11 YSOITdIJWNJhl/lSVZEuzucZdSoCEmlyxODVI2Y6lD4bBGNsEHxD0DJek7tgVVJSQOl3 JyVA== ARC-Authentication-Results: i=1; mx.google.com; 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 q3-v6si4803574pgv.652.2018.08.29.19.09.46; Wed, 29 Aug 2018 19:10:01 -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; 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 S1727278AbeH3GI3 (ORCPT + 99 others); Thu, 30 Aug 2018 02:08:29 -0400 Received: from shards.monkeyblade.net ([23.128.96.9]:39672 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725844AbeH3GI2 (ORCPT ); Thu, 30 Aug 2018 02:08:28 -0400 Received: from localhost (74-93-104-98-Washington.hfc.comcastbusiness.net [74.93.104.98]) (using TLSv1 with cipher AES256-SHA (256/256 bits)) (Client did not present a certificate) (Authenticated sender: davem-davemloft) by shards.monkeyblade.net (Postfix) with ESMTPSA id 3AF2F108F43BB; Wed, 29 Aug 2018 19:08:43 -0700 (PDT) Date: Wed, 29 Aug 2018 19:08:39 -0700 (PDT) Message-Id: <20180829.190839.1082082323658260994.davem@davemloft.net> To: decui@microsoft.com Cc: kys@microsoft.com, haiyangz@microsoft.com, sthemmin@microsoft.com, netdev@vger.kernel.org, devel@linuxdriverproject.org, linux-kernel@vger.kernel.org, olaf@aepfle.de, apw@canonical.com, jasowang@redhat.com, vkuznets@redhat.com, marcelo.cerri@canonical.com, jopoulso@microsoft.com Subject: Re: [PATCH] hv_netvsc: Fix a deadlock by getting rtnl_lock earlier in netvsc_probe() From: David Miller In-Reply-To: References: <20180829.174841.833303774555397982.davem@davemloft.net> X-Mailer: Mew version 6.7 on Emacs 25.3 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.12 (shards.monkeyblade.net [149.20.54.216]); Wed, 29 Aug 2018 19:08:43 -0700 (PDT) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Dexuan Cui Date: Thu, 30 Aug 2018 00:58:48 +0000 >> From: David Miller >> Sent: Wednesday, August 29, 2018 17:49 >> >> From: Dexuan Cui >> Date: Wed, 22 Aug 2018 21:20:03 +0000 >> >> > --- >> > drivers/net/hyperv/netvsc_drv.c | 11 ++++++++++- >> > 1 file changed, 10 insertions(+), 1 deletion(-) >> > >> > >> > FYI: these are the related 3 paths which show the deadlock: >> >> This incredibly useful information belongs in the commit log >> message, and therefore before the --- and signoffs. > > Hi David, > I was afraid the call-traces are too detailed. :-) > > Can you please move the info to before the --- line? > > Or, should I resend the patch with the commit log updated? Please resend.