public inbox for linux-rt-devel@lists.linux.dev
 help / color / mirror / Atom feed
From: "Ahmed S. Darwish" <darwi@linutronix.de>
To: Steven Rostedt <rostedt@goodmis.org>
Cc: Jonathan Corbet <corbet@lwn.net>,
	Clark Williams <clrkwllms@kernel.org>,
	linux-rt-devel@lists.linux.dev,
	Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
	John Ogness <john.ogness@linutronix.de>,
	Derek Barbosa <debarbos@redhat.com>,
	linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v1 1/1] Documentation: real-time: Add kernel configuration guide
Date: Fri, 6 Mar 2026 12:16:56 +0100	[thread overview]
Message-ID: <aaq3qN_mdpvJ0ixY@lx-t490> (raw)
In-Reply-To: <20260305180741.7bd114f4@gandalf.local.home>

On Thu, 05 Mar 2026, Steven Rostedt wrote:
>
> Very nice document!
>

Oh, thanks :)

>
> > +
> > +``CONFIG_PREEMPT_RT`` enables this option by default.  If this option is
> > +disabled during the kernel build, pass the following boot parameter [1]_::
> > +
> > +  efi=noruntime
>
> The above reads a bit funny. Maybe reword it to:
>
>   ``CONFIG_PREEMPT_RT`` enables this option by default. If this option is
>   manually disabled at build time, the following boot parameter [1]_ may
>   be used to disable EFI runtime at boot up::
>
> Or something like that.
>

Yes, much better; will do.

> > +
> > +There is ongoing `development work`_ to allow EFI variables access for a
> > +real-time Linux system.
>
>   .. to allow access to EFI variables for a real-time Linux system
>
>  ?
>

Yup.

> > +``CONFIG_TRACING`` (and tracing options)
> > +----------------------------------------
> > +
> > +:Expectation: enabled
> > +:Severity: *info*
> > +
> > +Shipping kernels with tracing support enabled (but not actively running)
> > +is highly recommended.  This will allow the users to extract more
> > +information if latency problems arise.
> > +
> > +.. caution::
> > +
> > +  Users should *not* make use of tracers or trace events during production
> > +  real-time kernel operation as they can add considerable overhead and
> > +  degrade the system's latency.
>
> I wonder if a special note should be called out for:
>
>   CONFIG_IRQSOFF_TRACER and CONFIG_PREEMPT_TRACER should be avoided as they
>   do incur measurable overhead even when tracing is not currently active.
>
> Maybe the above should be added in the "Problematic debug options"?
>

Oh, didn't know about that; I'll add them for sure.

>
> > +Kernel Debug Options
> > +====================
> > +
> > +Most kernel debug options add runtime overhead that increases the
> > +worst-case latency.
> > +
> > +.. caution::
> > +
> > +  During development and early testing, users are encouraged to run their
> > +  real-time workloads and peripherals with lockdep and other kernel debug
> > +  options enabled, for a considerable amount of time.  Such workloads
> > +  might trigger kernel code paths that were not triggered during the
> > +  internal Linux real-time kernel development, thus helping to uncover any
> > +  real-time latency issues in the kernel.
>
> Hmm, perhaps there should be some note that connects the use of "lockdep"
> with CONFIG_PROVE_LOCKING below (as that is what enables lockdep). The last
> sentence above makes it sound like lockdep can uncover latency issues, but
> it will most likely cause latency issues. Perhaps a bit more explanation
> should be used here.
>

will do.

> > +
> > +With that in mind, any false real-time kernel configuration could cause a
>
>   s/false/incorrect/ ?
>

Yup.

>
> Nice job!
>

Thanks!  I'll send a v2 on Monday.

All the best,
Ahmed

      reply	other threads:[~2026-03-06 11:17 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-05 20:50 [PATCH v1 0/1] Documentation: Add real-time kernel configuration guide Ahmed S. Darwish
2026-03-05 20:50 ` [PATCH v1 1/1] Documentation: real-time: Add " Ahmed S. Darwish
2026-03-05 21:05   ` Matthew Wilcox
2026-03-05 21:45     ` Ahmed S. Darwish
2026-03-05 23:09     ` Steven Rostedt
2026-03-05 23:07   ` Steven Rostedt
2026-03-06 11:16     ` Ahmed S. Darwish [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=aaq3qN_mdpvJ0ixY@lx-t490 \
    --to=darwi@linutronix.de \
    --cc=bigeasy@linutronix.de \
    --cc=clrkwllms@kernel.org \
    --cc=corbet@lwn.net \
    --cc=debarbos@redhat.com \
    --cc=john.ogness@linutronix.de \
    --cc=linux-doc@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-rt-devel@lists.linux.dev \
    --cc=rostedt@goodmis.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