mbox series

[0/3] trace-cmd: Fix some bugs that include issues with --max-graph-depth

Message ID 20190612181927.608151291@goodmis.org (mailing list archive)
Headers show
Series trace-cmd: Fix some bugs that include issues with --max-graph-depth | expand

Message

Steven Rostedt June 12, 2019, 6:19 p.m. UTC
Some last minute bugs that were found and fixed.

Steven Rostedt (VMware) (3):
      trace-cmd: Fix typo in Makefile bidr to bdir
      trace-cmd: Have --max-graph-depth only be part of instance
      trace-cmd: Check the return of get_file_content() before calling add_reset_file()

----
 tracecmd/Makefile              |  2 +-
 tracecmd/include/trace-local.h |  2 ++
 tracecmd/trace-record.c        | 33 +++++++++++++++++++--------------
 3 files changed, 22 insertions(+), 15 deletions(-)

Comments

Slavomir Kaslev June 13, 2019, 2:51 p.m. UTC | #1
On Wed, Jun 12, 2019 at 9:21 PM Steven Rostedt <rostedt@goodmis.org> wrote:
>
>
> Some last minute bugs that were found and fixed.
>
> Steven Rostedt (VMware) (3):
>       trace-cmd: Fix typo in Makefile bidr to bdir
>       trace-cmd: Have --max-graph-depth only be part of instance
>       trace-cmd: Check the return of get_file_content() before calling add_reset_file()
>
> ----
>  tracecmd/Makefile              |  2 +-
>  tracecmd/include/trace-local.h |  2 ++
>  tracecmd/trace-record.c        | 33 +++++++++++++++++++--------------
>  3 files changed, 22 insertions(+), 15 deletions(-)

Looks good to me.

Reviewed-by: Slavomir Kaslev <kaslevs@vmware.com>

Cheers,

--Slavi
Steven Rostedt June 13, 2019, 2:56 p.m. UTC | #2
On Thu, 13 Jun 2019 14:51:48 +0000
Slavomir Kaslev <kaslevs@vmware.com> wrote:

> On Wed, Jun 12, 2019 at 9:21 PM Steven Rostedt <rostedt@goodmis.org> wrote:
> >
> >
> > Some last minute bugs that were found and fixed.
> >
> > Steven Rostedt (VMware) (3):
> >       trace-cmd: Fix typo in Makefile bidr to bdir
> >       trace-cmd: Have --max-graph-depth only be part of instance
> >       trace-cmd: Check the return of get_file_content() before calling add_reset_file()
> >
> > ----
> >  tracecmd/Makefile              |  2 +-
> >  tracecmd/include/trace-local.h |  2 ++
> >  tracecmd/trace-record.c        | 33 +++++++++++++++++++--------------
> >  3 files changed, 22 insertions(+), 15 deletions(-)  
> 
> Looks good to me.
> 
> Reviewed-by: Slavomir Kaslev <kaslevs@vmware.com>
> 

Thanks Slavomir and Tzvetomir.

I'll add these patches now with both of your reviewed-bys.

-- Steve