[v1] trace: minor documentation fixes for clarity and

[PATCH 0/2] trace: minor documentation fixes for clarity and

Posted by Ryan Chung 1 month ago

Resend: fixed recipients; no changes

Hi all,

This short series cleans up two small wording issues in the tracing docs to
improve readability. There are no functional changes.

Ryan Chung (2):
  trace: rephrase for clearer documentation
  trace: fix grammar error in debugging.rst

 Documentation/trace/boottime-trace.rst | 2 +-
 Documentation/trace/debugging.rst      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

-- 
2.43.0

Re: [PATCH 0/2] trace: minor documentation fixes for clarity and

Posted by Steven Rostedt 1 month ago

On Sun, 31 Aug 2025 19:17:27 +0900
Ryan Chung <seokwoo.chung130@gmail.com> wrote:

> Resend: fixed recipients; no changes
> 
> Hi all,
> 
> This short series cleans up two small wording issues in the tracing docs to
> improve readability. There are no functional changes.
> 
> Ryan Chung (2):
>   trace: rephrase for clearer documentation
>   trace: fix grammar error in debugging.rst
> 
>  Documentation/trace/boottime-trace.rst | 2 +-
>  Documentation/trace/debugging.rst      | 2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 

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

The only comments I would make is the subject tag for the trace files
is actually "tracing:" and not "trace:".

Also, even though the changes are minor, a simple change log would
still be nice.

-- Steve

Re: [PATCH 0/2] trace: minor documentation fixes for clarity and

Posted by Ryan Chung 4 weeks, 1 day ago

On Tue, Sep 02, 2025 at 10:28:31AM -0400, Steven Rostedt wrote:
> On Sun, 31 Aug 2025 19:17:27 +0900
> Ryan Chung <seokwoo.chung130@gmail.com> wrote:
> 
> > Resend: fixed recipients; no changes
> > 
> > Hi all,
> > 
> > This short series cleans up two small wording issues in the tracing docs to
> > improve readability. There are no functional changes.
> > 
> > Ryan Chung (2):
> >   trace: rephrase for clearer documentation
> >   trace: fix grammar error in debugging.rst
> > 
> >  Documentation/trace/boottime-trace.rst | 2 +-
> >  Documentation/trace/debugging.rst      | 2 +-
> >  2 files changed, 2 insertions(+), 2 deletions(-)
> > 
> 
> Acked-by: Steven Rostedt (Google) <rostedt@goodmis.org>
> 
> The only comments I would make is the subject tag for the trace files
> is actually "tracing:" and not "trace:".
> 
> Also, even though the changes are minor, a simple change log would
> still be nice.
> 
> -- Steve

Hi. Thank you for your comment. 
I will make sure to add the change log next time.
Please let me know if I should do it and make a v2. 

Best regards,
Ryan Chung

Re: [PATCH 0/2] trace: minor documentation fixes for clarity and

Posted by Steven Rostedt 4 weeks, 1 day ago

On Thu, 4 Sep 2025 03:25:03 +0900
Ryan Chung <seokwoo.chung130@gmail.com> wrote:

> Hi. Thank you for your comment. 
> I will make sure to add the change log next time.
> Please let me know if I should do it and make a v2. 

I already gave an ack. I'll let Jon decided if there should be a v2 or
not.

-- Steve

Re: [PATCH 0/2] trace: minor documentation fixes for clarity and

Posted by Jonathan Corbet 4 weeks, 1 day ago

Steven Rostedt <rostedt@goodmis.org> writes:

> On Thu, 4 Sep 2025 03:25:03 +0900
> Ryan Chung <seokwoo.chung130@gmail.com> wrote:
>
>> Hi. Thank you for your comment. 
>> I will make sure to add the change log next time.
>> Please let me know if I should do it and make a v2. 
>
> I already gave an ack. I'll let Jon decided if there should be a v2 or
> not.

No need - I applied them and applied the tweak directly.

Thanks,

jon