[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20140219190555.GR10134@htj.dyndns.org>
Date: Wed, 19 Feb 2014 14:05:55 -0500
From: Tejun Heo <tj@...nel.org>
To: Peter Zijlstra <peterz@...radead.org>
Cc: Fengguang Wu <fengguang.wu@...el.com>, linux-kernel@...r.kernel.org
Subject: Re: [workqueue] BUG: unable to handle kernel paging request at
ffff880000798f60
On Wed, Feb 19, 2014 at 01:46:14PM +0100, Peter Zijlstra wrote:
> On Wed, Feb 19, 2014 at 08:41:22PM +0800, Fengguang Wu wrote:
> > Greetings,
> >
> > I got the below dmesg and the first bad commit is
> >
> > git://git.kernel.org/pub/scm/linux/kernel/git/tj/wq.git
> >
> > commit 84604340d034a074036e86e85c4055a1c7ea5529
> > Author: Peter Zijlstra <peterz@...radead.org>
> > AuthorDate: Tue Feb 11 12:00:36 2014 +0100
> > Commit: Tejun Heo <tj@...nel.org>
> > CommitDate: Tue Feb 11 10:25:17 2014 -0500
> >
> > workqueue: Fix workqueue lockdep name
> >
> > Tommi noticed a 'funny' lock class name: "%s#5" from a lock acquired
> > in process_one_work(). It turns out that commit b196be89cdc14 forgot
> > to change the lockdep_init_map() when it changed the @lock_name
> > argument from a string to a format.
> >
> > Fixes: b196be89cdc14 ("workqueue: make alloc_workqueue() take printf
> > fmt and args for name")
> >
> > Reported-by: Tommi Rantala <tt.rantala@...il.com>
> > Signed-off-by: Peter Zijlstra <peterz@...radead.org>
> > Signed-off-by: Tejun Heo <tj@...nel.org>
> > Cc: stable@...r.kernel.org
>
> I seem to remember tj already having reverted this patch.
Hmmm... right, this shouldn't be in the tree anymore. Let me check.
Yeap, it's not there anymore.
Thanks.
--
tejun
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists