mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Peter Zijlstra <peterz@infradead.org>
To: Anatoly Pugachev <matorola@gmail.com>
Cc: Linux Kernel list <linux-kernel@vger.kernel.org>,
	Sparc kernel list <sparclinux@vger.kernel.org>,
	Ingo Molnar <mingo@kernel.org>
Subject: Re: [sparc64] lockdep: Fix lockdep recursion - call trace
Date: Thu, 22 Oct 2020 13:17:00 +0200	[thread overview]
Message-ID: <20201022111700.GZ2651@hirez.programming.kicks-ass.net> (raw)
In-Reply-To: <20201022110015.GJ2594@hirez.programming.kicks-ass.net>

On Thu, Oct 22, 2020 at 01:00:16PM +0200, Peter Zijlstra wrote:
> On Thu, Oct 22, 2020 at 12:21:55PM +0300, Anatoly Pugachev wrote:
> > Hello!
> > 
> > Bisected the following linux calltrace after v5.9 :
> > 
> > [    8.650198] systemd[1]: Started Journal Service.
> > [    9.028125] ------------[ cut here ]------------
> > [    9.028171] WARNING: CPU: 11 PID: 499 at
> > net/netfilter/nf_tables_api.c:622 nft_chain_parse_hook+0x7c/0x360
> > [nf_tables]
> > [    9.028185] Modules linked in: nf_tables nfnetlink sunrpc ip_tables
> > x_tables ipv6 crc_ccitt autofs4 ext4 crc16 mbcache jbd2 raid10 raid456
> > async_raid6_recov async_mem
> > cpy async_pq async_xor xor async_tx raid6_pq raid1 raid0 multipath
> > linear md_mod crc32c_sparc64
> > [    9.028243] CPU: 11 PID: 499 Comm: nft Tainted: G        W
> > 5.9.0-rc8-00209-gbaffd723e44d #111
> > [    9.028255] Call Trace:
> > [    9.028269] [<00000000004727e8>] __warn+0xa8/0x120
> > [    9.028278] [<0000000000472c20>] warn_slowpath_fmt+0x34/0x74
> > [    9.028291] [<00000000100c19fc>] nft_chain_parse_hook+0x7c/0x360 [nf_tables]
> 
> > commit 4d004099a668c41522242aa146a38cc4eb59cb1e
> 
> What's unexpected.. while I just queued a another fix for that commit:
> 
>   https://lkml.kernel.org/r/20201022103028.GC2611@hirez.programming.kicks-ass.net
> 
> I don't think that explains this WARN. Let me go prod at it.

This looks like lockdep_is_held() encounters an incremented recursion
count and returns 1, which in this case will trigger the WARN.

But that would mean we're leaking a recursion count somewhere... do you
have CONFIG_DEBUG_PREEMPT enabled?

      reply	other threads:[~2020-10-22 11:17 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-10-22  9:21 Anatoly Pugachev
2020-10-22 11:00 ` Peter Zijlstra
2020-10-22 11:17   ` Peter Zijlstra [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20201022111700.GZ2651@hirez.programming.kicks-ass.net \
    --to=peterz@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matorola@gmail.com \
    --cc=mingo@kernel.org \
    --cc=sparclinux@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®