mbox series

[0/3] Introduce new API to reset ftrace instance

Message ID 20230328150308.34783-1-tz.stoyanov@gmail.com (mailing list archive)
Headers show
Series Introduce new API to reset ftrace instance | expand

Message

Tzvetomir Stoyanov (VMware) March 28, 2023, 3:03 p.m. UTC 
A new API tracefs_instance_reset() for resetting instance to its default
state, based on the logic from "trace-cmd reset" command.

Tzvetomir Stoyanov (VMware) (3):
  libtracefs: New API to reset ftrace instance
  libtracefs: Documentation for tracefs_instance_reset()
  libtracefs: Unit test for tracefs_instance_reset()

 Documentation/libtracefs-instances-manage.txt |   7 +-
 Documentation/libtracefs.txt                  |   1 +
 include/tracefs-local.h                       |   1 +
 include/tracefs.h                             |   1 +
 src/tracefs-instance.c                        | 196 ++++++++++++++++++
 src/tracefs-utils.c                           |  20 ++
 utest/tracefs-utest.c                         | 183 ++++++++++++++++
 7 files changed, 407 insertions(+), 2 deletions(-)

Comments

Steven Rostedt March 28, 2023, 3:13 p.m. UTC | #1 
On Tue, 28 Mar 2023 18:03:05 +0300
"Tzvetomir Stoyanov (VMware)" <tz.stoyanov@gmail.com> wrote:

> A new API tracefs_instance_reset() for resetting instance to its default
> state, based on the logic from "trace-cmd reset" command.

HAH!!!

I was just thinking that I needed to implement this earlier today.

I'm so happy when someone else does my work for me :-D

Thanks Tzvetomir, I'll hopefully get a chance to review this sometime this
week.

-- Steve

> 
> Tzvetomir Stoyanov (VMware) (3):
>   libtracefs: New API to reset ftrace instance
>   libtracefs: Documentation for tracefs_instance_reset()
>   libtracefs: Unit test for tracefs_instance_reset()
> 
>  Documentation/libtracefs-instances-manage.txt |   7 +-
>  Documentation/libtracefs.txt                  |   1 +
>  include/tracefs-local.h                       |   1 +
>  include/tracefs.h                             |   1 +
>  src/tracefs-instance.c                        | 196 ++++++++++++++++++
>  src/tracefs-utils.c                           |  20 ++
>  utest/tracefs-utest.c                         | 183 ++++++++++++++++
>  7 files changed, 407 insertions(+), 2 deletions(-)
>
Tzvetomir Stoyanov (VMware) April 13, 2023, 12:17 p.m. UTC | #2 
On Tue, Mar 28, 2023 at 6:13 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
> On Tue, 28 Mar 2023 18:03:05 +0300
> "Tzvetomir Stoyanov (VMware)" <tz.stoyanov@gmail.com> wrote:
>
> > A new API tracefs_instance_reset() for resetting instance to its default
> > state, based on the logic from "trace-cmd reset" command.
>
> HAH!!!
>
> I was just thinking that I needed to implement this earlier today.
>
> I'm so happy when someone else does my work for me :-D
>
> Thanks Tzvetomir, I'll hopefully get a chance to review this sometime this
> week.

ping
Did you have time to take a look at these patches? I'm planing to use
the library in a new project and need this new functionality.

>
> -- Steve
>
> >
> > Tzvetomir Stoyanov (VMware) (3):
> >   libtracefs: New API to reset ftrace instance
> >   libtracefs: Documentation for tracefs_instance_reset()
> >   libtracefs: Unit test for tracefs_instance_reset()
> >
> >  Documentation/libtracefs-instances-manage.txt |   7 +-
> >  Documentation/libtracefs.txt                  |   1 +
> >  include/tracefs-local.h                       |   1 +
> >  include/tracefs.h                             |   1 +
> >  src/tracefs-instance.c                        | 196 ++++++++++++++++++
> >  src/tracefs-utils.c                           |  20 ++
> >  utest/tracefs-utest.c                         | 183 ++++++++++++++++
> >  7 files changed, 407 insertions(+), 2 deletions(-)
> >
>
Steven Rostedt April 13, 2023, 12:20 p.m. UTC | #3 
On Thu, 13 Apr 2023 15:17:11 +0300
Tzvetomir Stoyanov <tz.stoyanov@gmail.com> wrote:

> ping
> Did you have time to take a look at these patches? I'm planing to use
> the library in a new project and need this new functionality.

No, I haven't had a chance. I've been focusing on kernel scheduler changes.
But I'll try to carve out time for this as well, as I will need this
functionality too.

-- Steve