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
prev parent 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