All of lore.kernel.org
 help / color / mirror / Atom feed
From: Stuart MENEFY <stuart.menefy@st.com>
To: linux-sh@vger.kernel.org
Subject: Re: [BUG] Error in copy_process, when enable CONFIG_PROVE_LOCKING.
Date: Mon, 18 May 2009 17:18:02 +0000	[thread overview]
Message-ID: <4A11984A.3000809@st.com> (raw)
In-Reply-To: <29ab51dc0905112140v3187b462qcc8a9e5385de8ef6@mail.gmail.com>

Paul Mundt wrote:
> On Tue, May 12, 2009 at 01:15:22PM +0100, Stuart MENEFY wrote:
>> Nobuhiro Iwamatsu wrote:
>>> Error in copy_process, when enable CONFIG_PROVE_LOCKING.
>>> This error has p->hardirqs_enabled in kernel.fork.c
>>>
>>>  987 #ifdef CONFIG_PROVE_LOCKING
>>>  988     DEBUG_LOCKS_WARN_ON(!p->hardirqs_enabled);
>>>  989     DEBUG_LOCKS_WARN_ON(!p->softirqs_enabled);
>>>  990 #endif
>>>  991     retval = -EAGAIN;
>>>
>>> The error message is as follows.
>>> I am debuging this. Please teach if you know the revision method of
>>> this problem.
>> As it happens I was looking at this recently. I think there are some
>> missing calls to trace_hardirqs_on/off in the assembly
>> code. Unfortunately my kernel is quite different from the mainline in
>> this area, but you could try this patch which is an *untested* forward
>> port of the code I'm currently running successfully.
>>
> This deadlocks for me with PROVE_LOCKING enabled, though it is certainly
> an improvement over what we had before. I'll try and debug it a bit more
> over the next few days when I get some spare cycles. If you have your
> kernel source available somewhere it would certainly make comparing the
> differences easier.

There's a git tree at:
  git://git.stlinux.com/stm/linux-sh4-2.6.23.y.git
now.

Stuart

  parent reply	other threads:[~2009-05-18 17:18 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-12  4:40 [BUG] Error in copy_process, when enable CONFIG_PROVE_LOCKING Nobuhiro Iwamatsu
2009-05-12 12:15 ` Stuart MENEFY
2009-05-18  4:11 ` Paul Mundt
2009-05-18 17:18 ` Stuart MENEFY [this message]
2009-07-29 14:11 ` Paul Mundt
2009-08-19  6:35 ` Nobuhiro Iwamatsu
2009-09-04  8:15 ` Nobuhiro Iwamatsu

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=4A11984A.3000809@st.com \
    --to=stuart.menefy@st.com \
    --cc=linux-sh@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.