[Qemu-commits] [qemu/qemu] 26896c: trace: add pid field to simpletrace record

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 26896cbf353e3017f76da8193074839b6e875250
      
https://github.com/qemu/qemu/commit/26896cbf353e3017f76da8193074839b6e875250
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M trace/simple.c

  Log Message:
  -----------
  trace: add pid field to simpletrace record

It is useful to know the QEMU process ID when working with traces from
multiple VMs.  Although the trace filename may contain the pid, tools
that aggregate traces or even trace globally need somewhere to record
the pid.

There is a reserved field in the trace event header struct that we can
use.

It is not necessary to bump the simpletrace file format version number
because it has already been incremented for the QEMU 2.1 release cycle
in commit "trace: [simple] Bump up log version number".

Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 80ff35cd3ff451e8f200413ddf27816058630c1f
      
https://github.com/qemu/qemu/commit/80ff35cd3ff451e8f200413ddf27816058630c1f
  Author: Stefan Hajnoczi <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M scripts/simpletrace.py

  Log Message:
  -----------
  simpletrace: add support for trace record pid field

Extract the pid field from the trace record and print it.

Change the trace record tuple from:
  (event_num, timestamp, arg1, ..., arg6)
to:
  (event_num, timestamp, pid, arg1, ..., arg6)

Trace event methods now support 3 prototypes:
1. <event-name>(arg1, arg2, arg3)
2. <event-name>(timestamp, arg1, arg2, arg3)
3. <event-name>(timestamp, pid, arg1, arg2, arg3)

Existing script continue to work without changes, they only know about
prototypes 1 and 2.

Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 82432638ebeedda8a2e18838b6fbef4b14a94f31
      
https://github.com/qemu/qemu/commit/82432638ebeedda8a2e18838b6fbef4b14a94f31
  Author: Alexey Kardashevskiy <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M trace/control.c

  Log Message:
  -----------
  trace: Replace error with warning if event is not defined

At the moment QEMU exits if trace point is not defined which makes
a developer life harder if he has to switch between branches with
different traces implemented.

This replaces error+exit wit WARNING if the tracepoint does not exist or
not traceable.

Signed-off-by: Alexey Kardashevskiy <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 5b808275f3bbe8cc95bb9301f4d5a41331d0e0e6
      
https://github.com/qemu/qemu/commit/5b808275f3bbe8cc95bb9301f4d5a41331d0e0e6
  Author: Lluís Vilanova <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M .travis.yml
    M Makefile
    M Makefile.target
    M configure
    M docs/tracing.txt
    M qemu-io.c
    M scripts/tracetool.py
    M scripts/tracetool/__init__.py
    M scripts/tracetool/backend/__init__.py
    M trace/Makefile.objs
    M trace/control-internal.h
    M trace/control.c
    M trace/control.h
    R trace/default.c
    M trace/ftrace.c
    M trace/ftrace.h
    M trace/simple.c
    M trace/simple.h
    R trace/stderr.c
    M vl.c

  Log Message:
  -----------
  trace: Multi-backend tracing

Adds support to compile QEMU with multiple tracing backends at the same time.

For example, you can compile QEMU with:

  $ ./configure --enable-trace-backends=ftrace,dtrace

Where 'ftrace' can be handy for having an in-flight record of events, and 
'dtrace' can be later used to extract more information from the system.

This patch allows having both available without recompiling QEMU.

Signed-off-by: Lluís Vilanova <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: a35d9be622a85d9ad6be5448e78c8a3f95ee5f00
      
https://github.com/qemu/qemu/commit/a35d9be622a85d9ad6be5448e78c8a3f95ee5f00
  Author: Alexey Kardashevskiy <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M trace/control.c

  Log Message:
  -----------
  trace: Replace fprintf with error_report and print location

This replaces fprintf(stderr) with error_report.

This moves local variables to the beginning of the function to comply
with QEMU's coding style.

Suggested-by: Lluís Vilanova <address@hidden>
Signed-off-by: Alexey Kardashevskiy <address@hidden>
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 14ac57339288c07b47e7e91fa192735158aa6a1e
      
https://github.com/qemu/qemu/commit/14ac57339288c07b47e7e91fa192735158aa6a1e
  Author: Peter Maydell <address@hidden>
  Date:   2014-06-09 (Mon, 09 Jun 2014)

  Changed paths:
    M .travis.yml
    M Makefile
    M Makefile.target
    M configure
    M docs/tracing.txt
    M qemu-io.c
    M scripts/simpletrace.py
    M scripts/tracetool.py
    M scripts/tracetool/__init__.py
    M scripts/tracetool/backend/__init__.py
    M trace/Makefile.objs
    M trace/control-internal.h
    M trace/control.c
    M trace/control.h
    R trace/default.c
    M trace/ftrace.c
    M trace/ftrace.h
    M trace/simple.c
    M trace/simple.h
    R trace/stderr.c
    M vl.c

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/stefanha/tags/tracing-pull-request' 
into staging

Tracing pull request

# gpg: Signature made Mon 09 Jun 2014 14:44:18 BST using RSA key ID 81AB73C8
# gpg: Good signature from "Stefan Hajnoczi <address@hidden>"
# gpg:                 aka "Stefan Hajnoczi <address@hidden>"

* remotes/stefanha/tags/tracing-pull-request:
  trace: Replace fprintf with error_report and print location
  trace: Multi-backend tracing
  trace: Replace error with warning if event is not defined
  simpletrace: add support for trace record pid field
  trace: add pid field to simpletrace record

Signed-off-by: Peter Maydell <address@hidden>


Compare: https://github.com/qemu/qemu/compare/4a331bb33bdf...14ac57339288