From: John Ogness <john.ogness@linutronix.de>
To: Tomas Glozar <tglozar@redhat.com>
Cc: Daniel Wagner <wagi@monom.org>,
Steven Rostedt <rostedt@goodmis.org>,
Sebastian Andrzej Siewior <bigeasy@linutronix.de>,
Clark Williams <clrkwllms@kernel.org>,
linux-trace-kernel@vger.kernel.org,
linux-rt-devel@lists.linux.dev
Subject: Re: [PATCH v2] rtla/timerlat: Lock the memory of the user-space threads
Date: Fri, 13 Nov 2026 17:23:37 +0106 [thread overview]
Message-ID: <87ld6wwvry.fsf@jogness.linutronix.de> (raw)
In-Reply-To: <CAP4=nvTswKUTSJiFRJK7KXAvc3JMLDCgXW=mSV6BO3mMucyWnA@mail.gmail.com>
On 2026-11-13, Tomas Glozar <tglozar@redhat.com> wrote:
> The only other activities the user thread performs with RT priority on
> before entering the timerlat interface read loop are setting the
> cgroup and opening the timerlat interface file descriptor. I agree
> that ideally, setting the priority should be done just before the
> loop.
If you say it is ideal to do X, then why not do X? Real-time tasks
(especially running at rtprio 95!) need to respect other real-time tasks
on the system. Developers new to RT might see this code and copy it. The
code even has a comment:
"If you want to use as base for your own tool... go ahead."
For me there are two issues here:
1. RT latency measurement tooling needs to be a little intrusive to a
system as practical. Otherwise it may not be possible for developers to
use the tooling on their real environment.
2. Official RT tooling should set the example of how to program for RT
Linux correctly.
John
next prev parent reply other threads:[~2026-11-13 16:17 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-11-13 14:03 [PATCH v2] rtla/timerlat: Lock the memory of the user-space threads Daniel Wagner
2026-11-13 14:56 ` John Ogness
2026-11-13 15:12 ` Tomas Glozar
2026-11-13 16:17 ` John Ogness [this message]
-- strict thread matches above, loose matches on Subject: below --
2026-11-12 21:23 [PATCH] " Daniel Wagner
2026-11-13 9:30 ` Tomas Glozar
2026-11-13 9:44 ` Sebastian Andrzej Siewior
2026-11-13 10:07 ` Tomas Glozar
2026-11-13 10:18 ` Sebastian Andrzej Siewior
2026-11-13 11:02 ` Daniel Wagner
2026-11-13 15:36 ` [PATCH v2] " Daniel Wagner
2026-11-13 12:42 ` [PATCH] " John Ogness
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=87ld6wwvry.fsf@jogness.linutronix.de \
--to=john.ogness@linutronix.de \
--cc=bigeasy@linutronix.de \
--cc=clrkwllms@kernel.org \
--cc=linux-rt-devel@lists.linux.dev \
--cc=linux-trace-kernel@vger.kernel.org \
--cc=rostedt@goodmis.org \
--cc=tglozar@redhat.com \
--cc=wagi@monom.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