From: Michal Hocko <mhocko@kernel.org>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Dmitry Vyukov <dvyukov@google.com>,
Byungchul Park <byungchul.park@lge.com>,
syzbot
<bot+e7353c7141ff7cbb718e4c888a14fa92de41ebaa@syzkaller.appspotmail.com>,
Andrew Morton <akpm@linux-foundation.org>,
Dan Williams <dan.j.williams@intel.com>,
Johannes Weiner <hannes@cmpxchg.org>, Jan Kara <jack@suse.cz>,
jglisse@redhat.com, LKML <linux-kernel@vger.kernel.org>,
linux-mm@kvack.org, shli@fb.com, syzkaller-bugs@googlegroups.com,
Thomas Gleixner <tglx@linutronix.de>,
Vlastimil Babka <vbabka@suse.cz>,
ying.huang@intel.com, kernel-team@lge.com
Subject: Re: possible deadlock in lru_add_drain_all
Date: Tue, 31 Oct 2017 15:58:04 +0100 [thread overview]
Message-ID: <20171031145804.ulrpk245ih6t7q7h@dhcp22.suse.cz> (raw)
In-Reply-To: <20171031145247.5kjbanjqged34lbp@hirez.programming.kicks-ass.net>
On Tue 31-10-17 15:52:47, Peter Zijlstra wrote:
[...]
> If we want to save those stacks; we have to save a stacktrace on _every_
> lock acquire, simply because we never know ahead of time if there will
> be a new link. Doing this is _expensive_.
>
> Furthermore, the space into which we store stacktraces is limited;
> since memory allocators use locks we can't very well use dynamic memory
> for lockdep -- that would give recursive and robustness issues.
Wouldn't stackdepot help here? Sure the first stack unwind will be
costly but then you amortize that over time. It is quite likely that
locks are held from same addresses.
--
Michal Hocko
SUSE Labs
next prev parent reply other threads:[~2017-10-31 14:58 UTC|newest]
Thread overview: 27+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <089e0825eec8955c1f055c83d476@google.com>
2017-10-27 9:34 ` possible deadlock in lru_add_drain_all Michal Hocko
2017-10-27 9:44 ` Dmitry Vyukov
2017-10-27 9:47 ` Dmitry Vyukov
2017-10-27 13:42 ` Michal Hocko
2017-10-30 8:22 ` Michal Hocko
2017-10-30 10:09 ` Byungchul Park
2017-10-30 15:10 ` Peter Zijlstra
2017-10-30 15:22 ` Peter Zijlstra
2017-10-31 13:13 ` Michal Hocko
2017-10-31 13:51 ` Peter Zijlstra
2017-10-31 13:55 ` Dmitry Vyukov
2017-10-31 14:52 ` Peter Zijlstra
2017-10-31 14:58 ` Michal Hocko [this message]
2017-10-31 15:10 ` Peter Zijlstra
2017-11-01 8:59 ` Byungchul Park
2017-11-01 12:01 ` Peter Zijlstra
2017-11-01 23:54 ` Byungchul Park
2018-02-14 14:01 ` Dmitry Vyukov
2018-02-14 15:44 ` Michal Hocko
2018-02-14 15:57 ` Dmitry Vyukov
2017-10-31 15:25 ` Peter Zijlstra
2017-10-31 15:45 ` Michal Hocko
2017-10-31 16:30 ` Peter Zijlstra
2017-11-01 8:31 ` Byungchul Park
2017-10-30 10:26 ` Byungchul Park
2017-10-30 11:48 ` Michal Hocko
2017-10-27 11:27 ` Vlastimil Babka
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=20171031145804.ulrpk245ih6t7q7h@dhcp22.suse.cz \
--to=mhocko@kernel.org \
--cc=akpm@linux-foundation.org \
--cc=bot+e7353c7141ff7cbb718e4c888a14fa92de41ebaa@syzkaller.appspotmail.com \
--cc=byungchul.park@lge.com \
--cc=dan.j.williams@intel.com \
--cc=dvyukov@google.com \
--cc=hannes@cmpxchg.org \
--cc=jack@suse.cz \
--cc=jglisse@redhat.com \
--cc=kernel-team@lge.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=peterz@infradead.org \
--cc=shli@fb.com \
--cc=syzkaller-bugs@googlegroups.com \
--cc=tglx@linutronix.de \
--cc=vbabka@suse.cz \
--cc=ying.huang@intel.com \
/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;
as well as URLs for NNTP newsgroup(s).