[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <87pmz422gj.fsf@meer.lwn.net>
Date: Thu, 08 Apr 2021 09:58:04 -0600
From: Jonathan Corbet <corbet@....net>
To: Daniel Bristot de Oliveira <bristot@...hat.com>,
Steven Rostedt <rostedt@...dmis.org>,
linux-kernel@...r.kernel.org
Cc: bristot@...hat.com, kcarcia@...hat.com,
Ingo Molnar <mingo@...hat.com>,
Peter Zijlstra <peterz@...radead.org>,
Thomas Gleixner <tglx@...utronix.de>,
Alexandre Chartre <alexandre.chartre@...cle.com>,
Clark Willaims <williams@...hat.com>,
John Kacur <jkacur@...hat.com>,
Juri Lelli <juri.lelli@...hat.com>, linux-doc@...r.kernel.org
Subject: Re: [RFC PATCH 5/5] tracing: Add the osnoise tracer
Daniel Bristot de Oliveira <bristot@...hat.com> writes:
A quick nit:
> Documentation/trace/osnoise_tracer.rst | 149 ++
> include/linux/ftrace_irq.h | 16 +
> include/trace/events/osnoise.h | 141 ++
> kernel/trace/Kconfig | 34 +
> kernel/trace/Makefile | 1 +
> kernel/trace/trace.h | 9 +-
> kernel/trace/trace_entries.h | 27 +
> kernel/trace/trace_osnoise.c | 1714 ++++++++++++++++++++++++
> kernel/trace/trace_output.c | 72 +-
> 9 files changed, 2159 insertions(+), 4 deletions(-)
> create mode 100644 Documentation/trace/osnoise_tracer.rst
> create mode 100644 include/trace/events/osnoise.h
> create mode 100644 kernel/trace/trace_osnoise.c
When you create a new RST file, you need to add it to an index.rst (or
similar) file so that it gets incorporated into the docs build.
The document itself looks good on a quick read. If you're making
another pass over it, you might consider reducing the ``markup noise`` a
bit; we try to keep that to a minimum in the kernel docs. But otherwise
thanks for writing it!
jon
Powered by blists - more mailing lists