From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: netconsole warning in 4.19.0-rc7 Date: Fri, 19 Oct 2018 10:45:55 -0700 (PDT) Message-ID: <20181019.104555.830780623382872282.davem@davemloft.net> References: <20181018145922.ytqxvdihp3iqounl@codemonkey.org.uk> <20181019134704.ky2vqeij6m77qrpg@codemonkey.org.uk> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: mroos@linux.ee, xiyou.wangcong@gmail.com, linux-kernel@vger.kernel.org, netdev@vger.kernel.org To: davej@codemonkey.org.uk Return-path: In-Reply-To: <20181019134704.ky2vqeij6m77qrpg@codemonkey.org.uk> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org From: Dave Jones Date: Fri, 19 Oct 2018 09:47:04 -0400 > On Fri, Oct 19, 2018 at 12:51:38PM +0300, Meelis Roos wrote: > > > > > I took another look at that error path. Turns out this is all we need I > > > > > think.. > > > > > > > > With this patch applied on top of 4.19-rc8, I stll get the warning: > > > > > > > > [ 13.722919] WARNING: CPU: 0 PID: 0 at kernel/softirq.c:168 __local_bh_enable_ip+0x2e/0x44 > > > > > > It's going to be a couple days before I get chance to get back to this. > > > Did the previous patch in this thread (without the _bh) fare any better? > > > > Tried it with rcu_read_unlock() instead, still the same. > > Ok, this is going to take more time than I have. DaveM, do you want > to revert 6fe9487892b32cb1c8b8b0d552ed7222a527fe30, or do you want a > patch doing the same ? > > That'll bring back the rcu warning, but fewer people were complaining > about that than this new issue.. I'll revert. Thanks Dave.