mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Paul E. McKenney" <paulmck@us.ibm.com>
To: Arjan van de Ven <arjan@infradead.org>
Cc: Pavel Machek <pavel@ucw.cz>,
	linux-kernel@vger.kernel.org, akpm@osdl.org, dipankar@in.ibm.com,
	vatsa@in.ibm.com, rusty@au1.ibm.com, mingo@elte.hu,
	manfred@colorfullife.com
Subject: Re: [PATCH] RCU torture testing
Date: Mon, 3 Oct 2005 08:26:02 -0700	[thread overview]
Message-ID: <20051003152602.GD1300@us.ibm.com> (raw)
In-Reply-To: <1128350188.17024.14.camel@laptopd505.fenrus.org>

On Mon, Oct 03, 2005 at 04:36:28PM +0200, Arjan van de Ven wrote:
> On Mon, 2005-10-03 at 07:30 -0700, Paul E. McKenney wrote:
> > On Sun, Oct 02, 2005 at 11:05:49PM +0200, Pavel Machek wrote:
> > > Can you just run the tests from time to time inside IBM?
> > 
> > In principle, I could, but in practice it is appropriate for non-IBMers to
> > be able to test the RCU infrastructure easily and thoroughly when they
> > work on it.
> 
> how hard would it be to make the few parameters just be module
> options... and then fail module load if the test fails or something?
> (and spew loudly in dmesg :)

Good point -- all I really need for module parameters is the number
of readers.  I should be able to have module load start the test and
module unload stop it (any problems with this approach?).  And doing
a module should remove the intrusions into rcupdate.c and rcupdate.h,
which would be good.

I would rather avoid dmesg.  But perhaps a read-only debugfs for output
(as Greg suggested) combined with module parameters for input could make
this straightforward.

> I'd be all in favor of having such a module in the kernel; in fact it
> would be nice if we roughly could standardize on an way to load/start
> and then find the result, I'd love to have a "make runtests" or
> something that would load such modules one by one

Which would mean that each test needs to give unambiguous machine-readable
indication of failure.  I guess I will nominate the string "!!!".  ;-)

> (and no that's not the task of ltp, ltp should test userspace; things
> that test in kernel code should really be part of the kernel)

I agree that there is definitely a need for both user-level and in-kernel
testing.  User-level testing is needed to make sure that user programs get
what they need, but there is no substitute for in-kernel testing when you
need to apply maximum conceiveable stress on some kernel component.

							Thanx, Paul

  reply	other threads:[~2005-10-03 15:25 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-10-01 18:20 Paul E. McKenney
2005-10-02  0:11 ` Greg KH
2005-10-03 14:28   ` Paul E. McKenney
2005-10-03 14:48     ` Greg KH
2005-10-02 21:05 ` Pavel Machek
2005-10-03  6:30   ` Ingo Molnar
2005-10-03 14:30   ` Paul E. McKenney
2005-10-03 14:36     ` Arjan van de Ven
2005-10-03 15:26       ` Paul E. McKenney [this message]
2005-10-03 15:31         ` Arjan van de Ven
2005-10-03 15:49           ` Paul E. McKenney
2005-10-22 23:09           ` Paul E. McKenney

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=20051003152602.GD1300@us.ibm.com \
    --to=paulmck@us.ibm.com \
    --cc=akpm@osdl.org \
    --cc=arjan@infradead.org \
    --cc=dipankar@in.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=manfred@colorfullife.com \
    --cc=mingo@elte.hu \
    --cc=pavel@ucw.cz \
    --cc=rusty@au1.ibm.com \
    --cc=vatsa@in.ibm.com \
    /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

Powered by JetHome