linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
@ 2013-06-04  5:19 Namhyung Kim
  0 siblings, 0 replies; 7+ messages in thread
From: Namhyung Kim @ 2013-06-04  5:19 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Hello,

This patchset is a resend version of previous work and basis of
upcoming perf ftrace patchset.  Although I saw no issue on this series
when I posted earlier, it'd be great if Steve or others take a look at
it again and give comments.

I also updated 'perf/cleanup' branch on my tree at:

  git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git

Thanks,
Namhyung


Namhyung Kim (16):
  tools lib traceevent: Remove unused install targets
  tools lib traceevent: Get rid of unused gui target
  tools lib traceevent: Add const qualifier to string arguments
  tools lib traceevent: Add trace_seq_reset()
  tools lib traceevent: Add page_size field to pevent
  tools lib traceevent: Port kbuffer parser routines
  perf util: Save page size in a trace file to pevent
  perf util: Save long size of traced system
  perf util: Make file/host_bigendian variable local
  perf util: Skip reading header_event file
  perf util: Parse header_page to get proper long size
  perf util: Get rid of unused header_page_* variables
  perf util: Move latency_format variable to builtin-script.c
  perf util: Rename read_*() functions in trace-event-info.c
  perf util: No need to call read_trace_init() in tracing_data_header()
  perf util: Remove unused enum and macro in trace-event.h

 tools/lib/traceevent/Makefile        |  18 +-
 tools/lib/traceevent/event-parse.c   |   7 +-
 tools/lib/traceevent/event-parse.h   |  15 +-
 tools/lib/traceevent/kbuffer-parse.c | 732 +++++++++++++++++++++++++++++++++++
 tools/lib/traceevent/kbuffer.h       |  67 ++++
 tools/lib/traceevent/trace-seq.c     |  13 +
 tools/perf/builtin-script.c          |   1 +
 tools/perf/util/trace-event-info.c   |  22 +-
 tools/perf/util/trace-event-parse.c  |   6 -
 tools/perf/util/trace-event-read.c   |  52 ++-
 tools/perf/util/trace-event.h        |  15 -
 11 files changed, 875 insertions(+), 73 deletions(-)
 create mode 100644 tools/lib/traceevent/kbuffer-parse.c
 create mode 100644 tools/lib/traceevent/kbuffer.h

-- 
1.7.11.7


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
  2013-06-04  5:20 Namhyung Kim
  2013-06-24  5:38 ` Namhyung Kim
  2013-07-04  8:39 ` Namhyung Kim
@ 2013-07-05 14:57 ` Arnaldo Carvalho de Melo
  2 siblings, 0 replies; 7+ messages in thread
From: Arnaldo Carvalho de Melo @ 2013-07-05 14:57 UTC (permalink / raw)
  To: Namhyung Kim
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Em Tue, Jun 04, 2013 at 02:20:15PM +0900, Namhyung Kim escreveu:
> Hello,
> 
> This patchset is a resend version of previous work and basis of
> upcoming perf ftrace patchset.  Although I saw no issue on this series
> when I posted earlier, it'd be great if Steve or others take a look at
> it again and give comments.

All seems ok, thanks a lot, applied.

- Arnaldo

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
  2013-07-05 13:16   ` Arnaldo Carvalho de Melo
@ 2013-07-05 13:35     ` Namhyung Kim
  0 siblings, 0 replies; 7+ messages in thread
From: Namhyung Kim @ 2013-07-05 13:35 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

On Fri, Jul 5, 2013 at 10:16 PM, Arnaldo Carvalho de Melo
<acme@ghostprotocols.net> wrote:
> Em Thu, Jul 04, 2013 at 05:39:06PM +0900, Namhyung Kim escreveu:
>> Ping!
>
> Pong, should be processing these today,

Great!

Thanks,
Namhyung

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
  2013-07-04  8:39 ` Namhyung Kim
@ 2013-07-05 13:16   ` Arnaldo Carvalho de Melo
  2013-07-05 13:35     ` Namhyung Kim
  0 siblings, 1 reply; 7+ messages in thread
From: Arnaldo Carvalho de Melo @ 2013-07-05 13:16 UTC (permalink / raw)
  To: Namhyung Kim
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Em Thu, Jul 04, 2013 at 05:39:06PM +0900, Namhyung Kim escreveu:
> Ping!

Pong, should be processing these today,

- Arnaldo
 
> On Tue,  4 Jun 2013 14:20:15 +0900, Namhyung Kim wrote:
> > Hello,
> >
> > This patchset is a resend version of previous work and basis of
> > upcoming perf ftrace patchset.  Although I saw no issue on this series
> > when I posted earlier, it'd be great if Steve or others take a look at
> > it again and give comments.
> >
> > I also updated 'perf/cleanup' branch on my tree at:
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git
> >
> > Thanks,
> > Namhyung
> >
> >
> > Namhyung Kim (16):
> >   tools lib traceevent: Remove unused install targets
> >   tools lib traceevent: Get rid of unused gui target
> >   tools lib traceevent: Add const qualifier to string arguments
> >   tools lib traceevent: Add trace_seq_reset()
> >   tools lib traceevent: Add page_size field to pevent
> >   tools lib traceevent: Port kbuffer parser routines
> >   perf util: Save page size in a trace file to pevent
> >   perf util: Save long size of traced system
> >   perf util: Make file/host_bigendian variable local
> >   perf util: Skip reading header_event file
> >   perf util: Parse header_page to get proper long size
> >   perf util: Get rid of unused header_page_* variables
> >   perf util: Move latency_format variable to builtin-script.c
> >   perf util: Rename read_*() functions in trace-event-info.c
> >   perf util: No need to call read_trace_init() in tracing_data_header()
> >   perf util: Remove unused enum and macro in trace-event.h
> >
> >  tools/lib/traceevent/Makefile        |  18 +-
> >  tools/lib/traceevent/event-parse.c   |   7 +-
> >  tools/lib/traceevent/event-parse.h   |  15 +-
> >  tools/lib/traceevent/kbuffer-parse.c | 732 +++++++++++++++++++++++++++++++++++
> >  tools/lib/traceevent/kbuffer.h       |  67 ++++
> >  tools/lib/traceevent/trace-seq.c     |  13 +
> >  tools/perf/builtin-script.c          |   1 +
> >  tools/perf/util/trace-event-info.c   |  22 +-
> >  tools/perf/util/trace-event-parse.c  |   6 -
> >  tools/perf/util/trace-event-read.c   |  52 ++-
> >  tools/perf/util/trace-event.h        |  15 -
> >  11 files changed, 875 insertions(+), 73 deletions(-)
> >  create mode 100644 tools/lib/traceevent/kbuffer-parse.c
> >  create mode 100644 tools/lib/traceevent/kbuffer.h

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
  2013-06-04  5:20 Namhyung Kim
  2013-06-24  5:38 ` Namhyung Kim
@ 2013-07-04  8:39 ` Namhyung Kim
  2013-07-05 13:16   ` Arnaldo Carvalho de Melo
  2013-07-05 14:57 ` Arnaldo Carvalho de Melo
  2 siblings, 1 reply; 7+ messages in thread
From: Namhyung Kim @ 2013-07-04  8:39 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Ping!

On Tue,  4 Jun 2013 14:20:15 +0900, Namhyung Kim wrote:
> Hello,
>
> This patchset is a resend version of previous work and basis of
> upcoming perf ftrace patchset.  Although I saw no issue on this series
> when I posted earlier, it'd be great if Steve or others take a look at
> it again and give comments.
>
> I also updated 'perf/cleanup' branch on my tree at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git
>
> Thanks,
> Namhyung
>
>
> Namhyung Kim (16):
>   tools lib traceevent: Remove unused install targets
>   tools lib traceevent: Get rid of unused gui target
>   tools lib traceevent: Add const qualifier to string arguments
>   tools lib traceevent: Add trace_seq_reset()
>   tools lib traceevent: Add page_size field to pevent
>   tools lib traceevent: Port kbuffer parser routines
>   perf util: Save page size in a trace file to pevent
>   perf util: Save long size of traced system
>   perf util: Make file/host_bigendian variable local
>   perf util: Skip reading header_event file
>   perf util: Parse header_page to get proper long size
>   perf util: Get rid of unused header_page_* variables
>   perf util: Move latency_format variable to builtin-script.c
>   perf util: Rename read_*() functions in trace-event-info.c
>   perf util: No need to call read_trace_init() in tracing_data_header()
>   perf util: Remove unused enum and macro in trace-event.h
>
>  tools/lib/traceevent/Makefile        |  18 +-
>  tools/lib/traceevent/event-parse.c   |   7 +-
>  tools/lib/traceevent/event-parse.h   |  15 +-
>  tools/lib/traceevent/kbuffer-parse.c | 732 +++++++++++++++++++++++++++++++++++
>  tools/lib/traceevent/kbuffer.h       |  67 ++++
>  tools/lib/traceevent/trace-seq.c     |  13 +
>  tools/perf/builtin-script.c          |   1 +
>  tools/perf/util/trace-event-info.c   |  22 +-
>  tools/perf/util/trace-event-parse.c  |   6 -
>  tools/perf/util/trace-event-read.c   |  52 ++-
>  tools/perf/util/trace-event.h        |  15 -
>  11 files changed, 875 insertions(+), 73 deletions(-)
>  create mode 100644 tools/lib/traceevent/kbuffer-parse.c
>  create mode 100644 tools/lib/traceevent/kbuffer.h

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
  2013-06-04  5:20 Namhyung Kim
@ 2013-06-24  5:38 ` Namhyung Kim
  2013-07-04  8:39 ` Namhyung Kim
  2013-07-05 14:57 ` Arnaldo Carvalho de Melo
  2 siblings, 0 replies; 7+ messages in thread
From: Namhyung Kim @ 2013-06-24  5:38 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Namhyung Kim, Peter Zijlstra, Paul Mackerras, Ingo Molnar, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Ping!

FYI, I rebased the perf/cleanup branch on top of current acme/perf/core.

Thanks,
Namhyung


2013-06-04 PM 2:20, Namhyung Kim wrote:
> Hello,
>
> This patchset is a resend version of previous work and basis of
> upcoming perf ftrace patchset.  Although I saw no issue on this series
> when I posted earlier, it'd be great if Steve or others take a look at
> it again and give comments.
>
> I also updated 'perf/cleanup' branch on my tree at:
>
>    git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git
>
> Thanks,
> Namhyung
>
>
> Namhyung Kim (16):
>    tools lib traceevent: Remove unused install targets
>    tools lib traceevent: Get rid of unused gui target
>    tools lib traceevent: Add const qualifier to string arguments
>    tools lib traceevent: Add trace_seq_reset()
>    tools lib traceevent: Add page_size field to pevent
>    tools lib traceevent: Port kbuffer parser routines
>    perf util: Save page size in a trace file to pevent
>    perf util: Save long size of traced system
>    perf util: Make file/host_bigendian variable local
>    perf util: Skip reading header_event file
>    perf util: Parse header_page to get proper long size
>    perf util: Get rid of unused header_page_* variables
>    perf util: Move latency_format variable to builtin-script.c
>    perf util: Rename read_*() functions in trace-event-info.c
>    perf util: No need to call read_trace_init() in tracing_data_header()
>    perf util: Remove unused enum and macro in trace-event.h
>
>   tools/lib/traceevent/Makefile        |  18 +-
>   tools/lib/traceevent/event-parse.c   |   7 +-
>   tools/lib/traceevent/event-parse.h   |  15 +-
>   tools/lib/traceevent/kbuffer-parse.c | 732 +++++++++++++++++++++++++++++++++++
>   tools/lib/traceevent/kbuffer.h       |  67 ++++
>   tools/lib/traceevent/trace-seq.c     |  13 +
>   tools/perf/builtin-script.c          |   1 +
>   tools/perf/util/trace-event-info.c   |  22 +-
>   tools/perf/util/trace-event-parse.c  |   6 -
>   tools/perf/util/trace-event-read.c   |  52 ++-
>   tools/perf/util/trace-event.h        |  15 -
>   11 files changed, 875 insertions(+), 73 deletions(-)
>   create mode 100644 tools/lib/traceevent/kbuffer-parse.c
>   create mode 100644 tools/lib/traceevent/kbuffer.h
>

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling
@ 2013-06-04  5:20 Namhyung Kim
  2013-06-24  5:38 ` Namhyung Kim
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Namhyung Kim @ 2013-06-04  5:20 UTC (permalink / raw)
  To: Arnaldo Carvalho de Melo
  Cc: Peter Zijlstra, Paul Mackerras, Ingo Molnar, Namhyung Kim, LKML,
	Steven Rostedt, Frederic Weisbecker, Jiri Olsa, David Ahern

Hello,

This patchset is a resend version of previous work and basis of
upcoming perf ftrace patchset.  Although I saw no issue on this series
when I posted earlier, it'd be great if Steve or others take a look at
it again and give comments.

I also updated 'perf/cleanup' branch on my tree at:

  git://git.kernel.org/pub/scm/linux/kernel/git/namhyung/linux-perf.git

Thanks,
Namhyung


Namhyung Kim (16):
  tools lib traceevent: Remove unused install targets
  tools lib traceevent: Get rid of unused gui target
  tools lib traceevent: Add const qualifier to string arguments
  tools lib traceevent: Add trace_seq_reset()
  tools lib traceevent: Add page_size field to pevent
  tools lib traceevent: Port kbuffer parser routines
  perf util: Save page size in a trace file to pevent
  perf util: Save long size of traced system
  perf util: Make file/host_bigendian variable local
  perf util: Skip reading header_event file
  perf util: Parse header_page to get proper long size
  perf util: Get rid of unused header_page_* variables
  perf util: Move latency_format variable to builtin-script.c
  perf util: Rename read_*() functions in trace-event-info.c
  perf util: No need to call read_trace_init() in tracing_data_header()
  perf util: Remove unused enum and macro in trace-event.h

 tools/lib/traceevent/Makefile        |  18 +-
 tools/lib/traceevent/event-parse.c   |   7 +-
 tools/lib/traceevent/event-parse.h   |  15 +-
 tools/lib/traceevent/kbuffer-parse.c | 732 +++++++++++++++++++++++++++++++++++
 tools/lib/traceevent/kbuffer.h       |  67 ++++
 tools/lib/traceevent/trace-seq.c     |  13 +
 tools/perf/builtin-script.c          |   1 +
 tools/perf/util/trace-event-info.c   |  22 +-
 tools/perf/util/trace-event-parse.c  |   6 -
 tools/perf/util/trace-event-read.c   |  52 ++-
 tools/perf/util/trace-event.h        |  15 -
 11 files changed, 875 insertions(+), 73 deletions(-)
 create mode 100644 tools/lib/traceevent/kbuffer-parse.c
 create mode 100644 tools/lib/traceevent/kbuffer.h

-- 
1.7.11.7


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2013-07-05 14:57 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-04  5:19 [PATCHSET 00/16] perf tools: Cleanups on tracepoint handling Namhyung Kim
2013-06-04  5:20 Namhyung Kim
2013-06-24  5:38 ` Namhyung Kim
2013-07-04  8:39 ` Namhyung Kim
2013-07-05 13:16   ` Arnaldo Carvalho de Melo
2013-07-05 13:35     ` Namhyung Kim
2013-07-05 14:57 ` Arnaldo Carvalho de Melo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).