[PATCH 0/9] Documentation/rtla: Cover default options

Tomas Glozar posted 9 patches 2 months, 1 week ago
Documentation/tools/rtla/common_options.txt      | 16 +++++++++++++---
.../tools/rtla/common_timerlat_options.txt       |  4 ++--
Documentation/tools/rtla/rtla-timerlat-top.rst   |  2 +-
Documentation/trace/timerlat-tracer.rst          | 12 ++++++------
4 files changed, 22 insertions(+), 12 deletions(-)
[PATCH 0/9] Documentation/rtla: Cover default options
Posted by Tomas Glozar 2 months, 1 week ago
RTLA has many options that have a default value that is used when
the option is not set associated with them. Those are not covered in
the documentation for the options, which creates confusion among users.

Document the default behavior for all relevant options: -H, -P, -C,
--trace-buffer-size. Some of these are covered in general
descriptions, only missing from the option documentation.

Also, fix a few typos and incorrect naming of tracers.

-----------------------

The patchset contains two parts: fixes (first five commits), and new
additions for default options, which were previously undocumented or
poorly documented (the rest).

The patchset assumes [1] is applied, renaming included files to *.txt
from *.rst, to prevent build error on "make htmldocs".

Note: I am aware that there are more issues with the documentation, just
not everything can (and should) be fixed in one patchset.

[1] https://lore.kernel.org/linux-trace-kernel/20251008184522.13201-1-krishnagopi487@gmail.com/

Tomas Glozar (9):
  Documentation/rtla: Fix typo in common_options.txt
  Documentation/rtla: Fix typo in common_timerlat_options.txt
  Documentation/rtla: Fix typo in rtla-timerlat-top.rst
  Documentation/rtla: Fix typo in common_timerlat_options.txt
  Documentation/rtla: Correct tracer name for common options
  Documentation/rtla: Mention default priority
  Documentation/rtla: Mention default cgroup state
  Documentation/trace: Specify exact priority for timerlat
  Documentation/rtla: Include defaults for tracer options

 Documentation/tools/rtla/common_options.txt      | 16 +++++++++++++---
 .../tools/rtla/common_timerlat_options.txt       |  4 ++--
 Documentation/tools/rtla/rtla-timerlat-top.rst   |  2 +-
 Documentation/trace/timerlat-tracer.rst          | 12 ++++++------
 4 files changed, 22 insertions(+), 12 deletions(-)

-- 
2.51.0
Re: [PATCH 0/9] Documentation/rtla: Cover default options
Posted by Jonathan Corbet 1 month, 2 weeks ago
Tomas Glozar <tglozar@redhat.com> writes:

> RTLA has many options that have a default value that is used when
> the option is not set associated with them. Those are not covered in
> the documentation for the options, which creates confusion among users.
>
> Document the default behavior for all relevant options: -H, -P, -C,
> --trace-buffer-size. Some of these are covered in general
> descriptions, only missing from the option documentation.
>
> Also, fix a few typos and incorrect naming of tracers.

So I see that this series is still unapplied...should I take it through
docs?

Thanks,

jon
Re: [PATCH 0/9] Documentation/rtla: Cover default options
Posted by Bagas Sanjaya 1 month, 2 weeks ago
On Mon, Nov 03, 2025 at 04:35:32PM -0700, Jonathan Corbet wrote:
> Tomas Glozar <tglozar@redhat.com> writes:
> 
> > RTLA has many options that have a default value that is used when
> > the option is not set associated with them. Those are not covered in
> > the documentation for the options, which creates confusion among users.
> >
> > Document the default behavior for all relevant options: -H, -P, -C,
> > --trace-buffer-size. Some of these are covered in general
> > descriptions, only missing from the option documentation.
> >
> > Also, fix a few typos and incorrect naming of tracers.
> 
> So I see that this series is still unapplied...should I take it through
> docs?

Of course!

Acked-by: Bagas Sanjaya <bagasdotme@gmail.com>

-- 
An old man doll... just what I always wanted! - Clara
Re: [PATCH 0/9] Documentation/rtla: Cover default options
Posted by Steven Rostedt 1 month, 2 weeks ago
On Mon, 03 Nov 2025 16:35:32 -0700
Jonathan Corbet <corbet@lwn.net> wrote:

> Tomas Glozar <tglozar@redhat.com> writes:
> 
> > RTLA has many options that have a default value that is used when
> > the option is not set associated with them. Those are not covered in
> > the documentation for the options, which creates confusion among users.
> >
> > Document the default behavior for all relevant options: -H, -P, -C,
> > --trace-buffer-size. Some of these are covered in general
> > descriptions, only missing from the option documentation.
> >
> > Also, fix a few typos and incorrect naming of tracers.  
> 
> So I see that this series is still unapplied...should I take it through
> docs?

Yes please!

Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>

Thanks,

-- Steve
Re: [PATCH 0/9] Documentation/rtla: Cover default options
Posted by Jonathan Corbet 1 month, 1 week ago
Steven Rostedt <rostedt@goodmis.org> writes:

> On Mon, 03 Nov 2025 16:35:32 -0700
> Jonathan Corbet <corbet@lwn.net> wrote:
>
>> Tomas Glozar <tglozar@redhat.com> writes:
>> 
>> > RTLA has many options that have a default value that is used when
>> > the option is not set associated with them. Those are not covered in
>> > the documentation for the options, which creates confusion among users.
>> >
>> > Document the default behavior for all relevant options: -H, -P, -C,
>> > --trace-buffer-size. Some of these are covered in general
>> > descriptions, only missing from the option documentation.
>> >
>> > Also, fix a few typos and incorrect naming of tracers.  
>> 
>> So I see that this series is still unapplied...should I take it through
>> docs?
>
> Yes please!
>
> Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>

OK, I have done that.

Thanks,

jon