public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Al Viro <viro@ZenIV.linux.org.uk>
To: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
Cc: Tetsuo Handa <penguin-kernel@i-love.sakura.ne.jp>,
	linux-kernel@vger.kernel.org, hugh@veritas.com,
	jmorris@namei.org, akpm@linux-foundation.org
Subject: Re: [2.6.30-rc1] RCU detected CPU 1 stall
Date: Fri, 10 Apr 2009 16:03:53 +0100	[thread overview]
Message-ID: <20090410150353.GL26366@ZenIV.linux.org.uk> (raw)
In-Reply-To: <20090410142203.GA6719@linux.vnet.ibm.com>

On Fri, Apr 10, 2009 at 07:22:03AM -0700, Paul E. McKenney wrote:

> Hmmmm...  This indicates that CPU 1 was spinning in the kernel for
> a long time.  At 250 HZ, 32,565 jiffies is 130 seconds, or just over
> two -minutes-.  Ouch!!!
> 
> The interrupt happened on the stalled CPU, so we know that interrupts
> were enabled.  Because we have CONFIG_PREEMPT_NONE=y, there is no
> preemption, so preemption need not be disabled.  This could be due
> to lock contention, or even a simple infinite loop.
> 
> The timer interrupt (apic_timer_interrupt) occurred in either
> __bprm_mm_init(), __get_user_4(), count(), or do_execve().  There
> have been some recent changes around check_unsafe_exec() -- any
> possibility that these introduced excessive lock contention or
> an infinite loop?  Ditto for the recent security fixes?

Oh, joy... the loop in there is this:
        for (t = next_thread(p); t != p; t = next_thread(t)) {
                if (t->fs == p->fs)
                        n_fs++;
        }
I find it hard to believe that it can take two minutes, though.

  reply	other threads:[~2009-04-10 15:04 UTC|newest]

Thread overview: 24+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-08  0:57 [2.6.30-rc1] RCU detected CPU 1 stall Tetsuo Handa
2009-04-10 14:22 ` Paul E. McKenney
2009-04-10 15:03   ` Al Viro [this message]
2009-04-10 15:32     ` Paul E. McKenney
2009-04-10 21:08       ` Tetsuo Handa
2009-04-10 23:12         ` Paul E. McKenney
2009-04-10 23:39           ` Al Viro
2009-04-11  0:48             ` Paul E. McKenney
2009-04-13  0:48             ` Tetsuo Handa
2009-04-13 19:55               ` Paul E. McKenney
2009-04-13 21:31                 ` Tetsuo Handa
2009-04-13 22:47                   ` Paul E. McKenney
2009-04-14 17:11               ` Hugh Dickins
2009-04-15  3:28                 ` Tetsuo Handa
2009-04-15 20:34                   ` Hugh Dickins
2009-04-16  0:46                     ` Tetsuo Handa
2009-04-16  5:15                       ` Hugh Dickins
2009-04-16  5:35                         ` Tetsuo Handa
2009-04-16 20:37                           ` Greg KH
2009-04-16 20:55                           ` Please revert kobject_uevent UMH_NO_WAIT Hugh Dickins
2009-04-16 20:58                             ` [PATCH] mm: pass correct mm when growing stack Hugh Dickins
2009-04-16 21:24                             ` Please revert kobject_uevent UMH_NO_WAIT Greg KH
2009-04-16 21:30                               ` Arjan van de Ven
2009-04-16 21:36                                 ` Will Newton

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=20090410150353.GL26366@ZenIV.linux.org.uk \
    --to=viro@zeniv.linux.org.uk \
    --cc=akpm@linux-foundation.org \
    --cc=hugh@veritas.com \
    --cc=jmorris@namei.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=paulmck@linux.vnet.ibm.com \
    --cc=penguin-kernel@i-love.sakura.ne.jp \
    /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