public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Ingo Molnar <mingo@elte.hu>
To: Peter Teoh <htmldeveloper@gmail.com>
Cc: Peter Zijlstra <peterz@infradead.org>,
	LKML <linux-kernel@vger.kernel.org>
Subject: Re: Locking API testsuite: Failures
Date: Wed, 27 Aug 2008 08:45:51 +0200	[thread overview]
Message-ID: <20080827064551.GE6255@elte.hu> (raw)
In-Reply-To: <804dabb00808262332l4fb0a4d6i8adee9c809c65901@mail.gmail.com>


* Peter Teoh <htmldeveloper@gmail.com> wrote:

> Several questions:

[...]
> 3.  My current .config (which gave rise to the failure errors) [...]

You should have read this bit of the self-test printout:

[    0.004000] 133 out of 218 testcases failed, as expected. |

there was no unexpected failure. A real test-case failure is displayed 
as: 'FAILED', and you'll get a verbose printout about the unexpected 
testcase failures.

> 1.   I noticed there is a Kconfig.debug with this entry:
> 
> config DEBUG_LOCKING_API_SELFTESTS
>         bool "Locking API boot-time self-tests"
>         depends on DEBUG_KERNEL
>         help
>           Say Y here if you want the kernel to run a short self-test during
>           bootup. The self-test checks whether common types of locking bugs
>           are detected by debugging mechanisms or not. (if you disable
>           lock debugging then those bugs wont be detected of course.)
>           The following locking APIs are covered: spinlocks, rwlocks,
>           mutexes and rwsems.
> 
> How do I use this Kconfig.debug, just overwriting the standard
> lib/Kconfig? [...]

no, that messes up the kernel. Kconfig.debug is a separate, unique 
kconfig file with debugging related options.

> 2.   So does it make sense to  compile with
> CONFIG_DEBUG_LOCKING_API_SELFTESTS without CONFIG_LOCKDEP?

yes. It shows the kinds of bugs we dont find when PROVE_LOCKING is 
disabled. It also shows that the test-cases are working as expected.

	Ingo

      reply	other threads:[~2008-08-27  6:46 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-27  0:27 Locking API testsuite: Failures Peter Teoh
2008-08-27  1:58 ` Peter Zijlstra
2008-08-27  6:32   ` Peter Teoh
2008-08-27  6:45     ` Ingo Molnar [this message]

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=20080827064551.GE6255@elte.hu \
    --to=mingo@elte.hu \
    --cc=htmldeveloper@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=peterz@infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox