All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@linux.vnet.ibm.com>
To: Kyle Moffett <kyle@moffetthome.net>
Cc: kernelmail.jms@gmail.com, RKK <kulkarni.ravi4@gmail.com>,
	linux-kernel@vger.kernel.org, cmm@us.ibm.com
Subject: Re: Fw: RCU boot debug patch
Date: Wed, 27 Jul 2011 17:02:54 -0700	[thread overview]
Message-ID: <20110728000254.GB2318@linux.vnet.ibm.com> (raw)
In-Reply-To: <CAGZ=bq+p0kNuj-2dAj40D1Hdru-S64vE_QEhL_mAphYY-0hqyw@mail.gmail.com>

On Wed, Jul 27, 2011 at 07:56:10PM -0400, Kyle Moffett wrote:
> On Wed, Jul 27, 2011 at 17:47,  <kernelmail.jms@gmail.com> wrote:
> >>A debug patch to verify if there is RCU callbacks before the rcu scheduler is active
> > So something gives me the impression we may have an answer somewhere in this dmesg for this... ;-)
> >
> > Here is my dmesg (the whole thing):
> >
> > function free_object_rcu
> > [    0.166707] RCU scheduler is not active yet, Calling _call_rcu() with this function free_object_rcu
> > [....thousands of lines snipped...]
> > [    0.225223] RCU scheduler is not active yet, Calling _call_rcu() with this function free_object_rcu

Thank you, Julie!

> Well this looks an awful lot like a smoking gun here...
> 
> I can only find one function called "free_object_rcu", and it seems to be a
> static function in mm/kmemleak.c, and it certainly does this:
>    call_rcu(&object->rcu, free_object_rcu);
> 
> So the next question for you would be:  Do all the messages go away if you
> turn off the kmemleak config option?

Hello, Kyle,

This is not at all showing a problem in free_object_rcu(), but rather
as a sanity-check for a shoot-from-the-hip fix.  The fix seemed to
work, but my paranoia dictated that I check that my guess was correct.
Which it appears that it was.

So again, these messages do not indicate a problem in free_object_rcu().

							Thanx, Paul

  reply	other threads:[~2011-07-28  0:02 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-07-19 22:41 Fw: RCU boot debug patch Paul E. McKenney
2011-07-20  5:27 ` RKK
2011-07-20 14:59   ` Paul E. McKenney
2011-07-20 15:53     ` RKK
2011-07-20 16:09       ` Paul E. McKenney
2011-07-21  5:39         ` RKK
2011-07-22  2:29           ` Paul E. McKenney
2011-07-27 21:47             ` kernelmail.jms
2011-07-27 23:56               ` Kyle Moffett
2011-07-28  0:02                 ` Paul E. McKenney [this message]
2011-07-28 19:56                   ` Julie Sullivan

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=20110728000254.GB2318@linux.vnet.ibm.com \
    --to=paulmck@linux.vnet.ibm.com \
    --cc=cmm@us.ibm.com \
    --cc=kernelmail.jms@gmail.com \
    --cc=kulkarni.ravi4@gmail.com \
    --cc=kyle@moffetthome.net \
    --cc=linux-kernel@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.