From: Dave Jones <davej@redhat.com>
To: Tejun Heo <tj@kernel.org>
Cc: Linus Torvalds <torvalds@linux-foundation.org>,
Chris Metcalf <cmetcalf@tilera.com>,
Andrew Morton <akpm@linux-foundation.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
linux-mm <linux-mm@kvack.org>,
Lai Jiangshan <laijs@cn.fujitsu.com>
Subject: Re: deadlock in lru_add_drain ? (3.14rc5)
Date: Mon, 10 Mar 2014 16:15:08 -0400 [thread overview]
Message-ID: <20140310201508.GA16200@redhat.com> (raw)
In-Reply-To: <20140310200957.GF25290@htj.dyndns.org>
On Mon, Mar 10, 2014 at 04:09:57PM -0400, Tejun Heo wrote:
> Hmmm... this is puzzling. At least according to the slightly
> truncated (pids < 13) sysrq-t output, there's no kworker running
> lru_add_drain_per_cpu() and nothing blocked on lru_add_drain_all::lock
> can introduce any complex dependency. Also, at least from glancing
> over, I don't see anything behind lru_add_rain_per_cpu() which can get
> involved in a complex dependency chain.
>
> Assuming that the handful lost traces didn't reveal serious ah-has, it
> almost looks like workqueue either failed to initiate execution of a
> queued work item or flush_work() somehow got confused on a work item
> which already finished, both of which are quite unlikely given that we
> haven't had any simliar report on any other work items.
>
> I think it'd be wise to extend sysrq-t output to include the states of
> workqueue if for nothing else to easily rule out doubts about basic wq
> functions. Dave, is this as much information we're gonna get from the
> trinity instance? I assume trying to reproduce the case isn't likely
> to work?
I tried enabling the function tracer, and ended up locking up the box entirely,
so had to reboot. Rerunning it now on rc6, will let you know if it reproduces
(though it took like a day or so last time).
Dave
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>
prev parent reply other threads:[~2014-03-10 20:15 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-03-08 22:00 deadlock in lru_add_drain ? (3.14rc5) Dave Jones
2014-03-09 1:18 ` Linus Torvalds
2014-03-10 15:01 ` Tejun Heo
2014-03-10 15:50 ` Dave Jones
2014-03-10 20:09 ` Tejun Heo
2014-03-10 20:15 ` Dave Jones [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=20140310201508.GA16200@redhat.com \
--to=davej@redhat.com \
--cc=akpm@linux-foundation.org \
--cc=cmetcalf@tilera.com \
--cc=laijs@cn.fujitsu.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=tj@kernel.org \
--cc=torvalds@linux-foundation.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;
as well as URLs for NNTP newsgroup(s).