All of lore.kernel.org
 help / color / mirror / Atom feed
From: Philippe Gerum <rpm@xenomai.org>
To: Jan Kiszka <jan.kiszka@domain.hid>
Cc: xenomai-core <xenomai@xenomai.org>
Subject: [Xenomai-core] Re: XENO_OPT_DEBUG impact (was: exception 768)
Date: Fri, 17 Nov 2006 20:05:15 +0100	[thread overview]
Message-ID: <1163790315.4980.73.camel@domain.hid> (raw)
In-Reply-To: <455E025B.5030906@domain.hid>

On Fri, 2006-11-17 at 19:41 +0100, Jan Kiszka wrote:
> Philippe Gerum wrote:
> > ...
> > Disable CONFIG_XENO_OPT_DEBUG from your kernel configuration, this is
> > the source of the PTE misses you are seeing now on ppc. This is not to
> > say that those errors are normal, and this issue still remains to be
> > fixed, but unless you want to debug the Xenomai nucleus, you don't need
> > this option on (additionally, it adds a large overhead which translates
> > in significantly augmented jitter).
> 
> CONFIG_XENO_OPT_DEBUG should not add large jitters - that's what e.g.

CONFIG_SMP + CONFIG_XENO_OPT_DEBUG instruments the nucleus lock macros
with consistency and latency tracking code. This does not come for free,
even if an order of magnitude less than checking the queues.

> CONFIG_XENO_OPT_DEBUG_QUEUES is now for.
> 
> I'm currently seeing two potential "misuses" of the common switch:
> 
>  - the posix skin (Gilles, how heavy-weighted are those checks?)
>    => CONFIG_XENO_OPT_DEBUG_POSIX
> 
>  - CONFIG_XENO_SPINLOCK_DEBUG => CONFIG_XENO_OPT_DEBUG_SPINLOCK
> 
> Both should be explicitly controllable in Kconfig.
> 

Nack for CONFIG_XENO_OPT_DEBUG_SPINLOCK. Most of the issue we tracked
with Gilles regarding the domain migration code had side-effects on the
nucleus lock. So having CONFIG_XENO_OPT_DEBUG enabled for identifying
internal state weirdnesses - like those triggered by migration bugs -
implies enabling the spinlock watchdogs too.

> Gilles, is CONFIG_XENO_OPT_DEBUG_BHEAP used in any way? Doesn't seem so.
> 
> Jan
> 
-- 
Philippe.




  reply	other threads:[~2006-11-17 19:05 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-11-13 12:13 [Xenomai-help] exception 768 Daniel Schnell
2006-11-17 17:32 ` Philippe Gerum
2006-11-17 18:02   ` Daniel Schnell
2006-11-17 18:41   ` [Xenomai-core] XENO_OPT_DEBUG impact (was: exception 768) Jan Kiszka
2006-11-17 19:05     ` Philippe Gerum [this message]
2006-11-17 19:10       ` [Xenomai-core] Re: XENO_OPT_DEBUG impact Jan Kiszka
2006-11-17 21:58         ` Philippe Gerum
2006-11-20  9:20           ` Jan Kiszka
2006-11-20  9:38             ` Philippe Gerum
2006-11-20 10:01               ` Jan Kiszka
2006-11-20 10:46                 ` Philippe Gerum
2006-11-20 11:39                   ` Jan Kiszka
2006-11-20 13:22                     ` Philippe Gerum
2006-11-20  9:07     ` Gilles Chanteperdrix
2006-11-20  9:14       ` Jan Kiszka

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=1163790315.4980.73.camel@domain.hid \
    --to=rpm@xenomai.org \
    --cc=jan.kiszka@domain.hid \
    --cc=xenomai@xenomai.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.