[Qemu-commits] [qemu/qemu] 43b48c: trace: count number of enabled events

qemu-commits
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-commits] [qemu/qemu] 43b48c: trace: count number of enabled events

From:	GitHub
Subject:	[Qemu-commits] [qemu/qemu] 43b48c: trace: count number of enabled events
Date:	Thu, 04 Feb 2016 03:30:05 -0800
  Branch: refs/heads/master
  Home:   https://github.com/qemu/qemu
  Commit: 43b48cfc3e8ff745a10a6b78a55519d5cf7ec5e8
      
https://github.com/qemu/qemu/commit/43b48cfc3e8ff745a10a6b78a55519d5cf7ec5e8
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M trace/control-internal.h
    M trace/control.c

  Log Message:
  -----------
  trace: count number of enabled events

This lets trace_event_get_state_dynamic quickly return false.  Right
now there is hardly any benefit because there are also many assertions
and indirections, but the next patch will streamline all of this.

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


  Commit: 585ec7273e6fdab902b2128bc6c2a8136aafef04
      
https://github.com/qemu/qemu/commit/585ec7273e6fdab902b2128bc6c2a8136aafef04
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M scripts/tracetool/format/events_c.py
    M trace/control-internal.h
    M trace/control.c
    M trace/control.h
    M trace/event-internal.h

  Log Message:
  -----------
  trace: track enabled events in a separate array

This is more cache friendly on the fast path, where we already have
the event id available.

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


  Commit: 52449a314e100e2db990ab9a156bb0173017f28d
      
https://github.com/qemu/qemu/commit/52449a314e100e2db990ab9a156bb0173017f28d
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M qemu-options.hx

  Log Message:
  -----------
  trace: fix documentation

Mention the ftrace backend too.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 45bd0b41bdbf554e49fb510673309164c513a8c8
      
https://github.com/qemu/qemu/commit/45bd0b41bdbf554e49fb510673309164c513a8c8
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M qemu-io.c
    M trace/control.c
    M trace/control.h
    M vl.c

  Log Message:
  -----------
  trace: split trace_init_events out of trace_init_backends

This is cleaner and has two advantages.  First, it improves error
reporting with -daemonize.  Second, multiple "-trace events" options
now cumulate.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 41fc57e44ed64cd4ab5393d83624afd897dabd4f
      
https://github.com/qemu/qemu/commit/41fc57e44ed64cd4ab5393d83624afd897dabd4f
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M qemu-io.c
    M trace/control.c
    M trace/control.h
    M trace/simple.c
    M trace/simple.h
    M vl.c

  Log Message:
  -----------
  trace: split trace_init_file out of trace_init_backends

This is cleaner, and improves error reporting with -daemonize.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: f246b86672118e4f0d0782f4c240d3666aa36710
      
https://github.com/qemu/qemu/commit/f246b86672118e4f0d0782f4c240d3666aa36710
  Author: Denis V. Lunev <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M vl.c

  Log Message:
  -----------
  trace: no need to call trace_backend_init in different branches now

original idea to split calling locations was to spawn tracing thread
in the final child process according to

    commit 8a745f2a9296ad2cf6bda33534ed298f2625a4ad
    Author: Michael Mueller
    Date:   Mon Sep 23 16:36:54 2013 +0200

os_daemonize is now on top of both locations. Drop unneeded ifs.

Signed-off-by: Denis V. Lunev <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 10578a257d94fb59449d0b0e441990c45a036ccc
      
https://github.com/qemu/qemu/commit/10578a257d94fb59449d0b0e441990c45a036ccc
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M qemu-options.hx
    M trace/control.c
    M trace/control.h
    M vl.c

  Log Message:
  -----------
  trace: add "-trace enable=..."

Allow enabling events without going through a file, for example:

   qemu-system-x86_64 -trace bdrv_aio_writev -trace bdrv_aio_readv

or with globbing too:

   qemu-system-x86_64 -trace 'bdrv_aio_*'

if an appropriate backend is enabled (simple, stderr, ftrace).

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: e9527dd399ed213ec6eb2c0ea54be663a73032b0
      
https://github.com/qemu/qemu/commit/e9527dd399ed213ec6eb2c0ea54be663a73032b0
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M qemu-options.hx
    M trace/control.c
    M trace/control.h

  Log Message:
  -----------
  trace: add "-trace help"

Print a list of trace points

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 508127e243122cf3ed67d2aaa472a1b4f1be055e
      
https://github.com/qemu/qemu/commit/508127e243122cf3ed67d2aaa472a1b4f1be055e
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M bsd-user/main.c
    M cpu-exec.c
    M exec.c
    M hw/acpi/cpu_hotplug.c
    M hw/timer/a9gtimer.c
    A include/exec/log.h
    M include/qemu/log.h
    M linux-user/main.c
    M qom/cpu.c
    M target-alpha/translate.c
    M target-arm/translate-a64.c
    M target-arm/translate.c
    M target-cris/translate.c
    M target-i386/seg_helper.c
    M target-i386/smm_helper.c
    M target-i386/translate.c
    M target-lm32/helper.c
    M target-lm32/translate.c
    M target-m68k/translate.c
    M target-microblaze/helper.c
    M target-microblaze/translate.c
    M target-mips/helper.c
    M target-mips/translate.c
    M target-moxie/translate.c
    M target-openrisc/translate.c
    M target-ppc/mmu-hash32.c
    M target-ppc/mmu-hash64.c
    M target-ppc/mmu_helper.c
    M target-ppc/translate.c
    M target-s390x/translate.c
    M target-sh4/helper.c
    M target-sh4/translate.c
    M target-sparc/int32_helper.c
    M target-sparc/int64_helper.c
    M target-sparc/translate.c
    M target-tilegx/translate.c
    M target-tricore/translate.c
    M target-unicore32/translate.c
    M target-xtensa/translate.c
    M tcg/tcg.c
    M translate-all.c

  Log Message:
  -----------
  log: do not unnecessarily include qom/cpu.h

Split the bits that require it to exec/log.h.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: d890d50d1821a18366a0b9f72e6c93689a47bcd4
      
https://github.com/qemu/qemu/commit/d890d50d1821a18366a0b9f72e6c93689a47bcd4
  Author: Denis V. Lunev <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M Makefile.objs
    R qemu-log.c
    M util/Makefile.objs
    A util/log.c

  Log Message:
  -----------
  log: move qemu-log.c into util/ directory

log will become common facility with tracepoints support in next step.

Signed-off-by: Denis V. Lunev <address@hidden>
Reviewed-by: Paolo Bonzini <address@hidden>
Signed-off-by: Paolo Bonzini <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: ed7f5f1d8db06fc31352a5ef4f54985e630c575a
      
https://github.com/qemu/qemu/commit/ed7f5f1d8db06fc31352a5ef4f54985e630c575a
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M .travis.yml
    M configure
    M include/qemu/log.h
    A scripts/tracetool/backend/log.py
    R scripts/tracetool/backend/stderr.py
    M trace/control.c
    M util/log.c
    M vl.c

  Log Message:
  -----------
  trace: convert stderr backend to log

[Also update .travis.yml --enable-trace-backends=stderr
--Stefan]

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: baf86d6b3ca0191372f8b60299ef500b5484fd8f
      
https://github.com/qemu/qemu/commit/baf86d6b3ca0191372f8b60299ef500b5484fd8f
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M configure

  Log Message:
  -----------
  trace: switch default backend to "log"

This enables integration with other QEMU logging facilities.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: c84ea00dc29b2084a62bd31f498ed0f245d3848e
      
https://github.com/qemu/qemu/commit/c84ea00dc29b2084a62bd31f498ed0f245d3848e
  Author: Paolo Bonzini <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M util/log.c

  Log Message:
  -----------
  log: add "-d trace:PATTERN"

This is a bit easier to use than "-trace" if you are also enabling
other kinds of logging.  It is also more discoverable for experienced
QEMU users, and accessible from user-mode emulators.

Signed-off-by: Paolo Bonzini <address@hidden>
Signed-off-by: Denis V. Lunev <address@hidden>
Acked-by: Christian Borntraeger <address@hidden>
Message-id: address@hidden
Signed-off-by: Stefan Hajnoczi <address@hidden>


  Commit: 382d34ff9fcc534db32d54eb82590de7c04f9b33
      
https://github.com/qemu/qemu/commit/382d34ff9fcc534db32d54eb82590de7c04f9b33
  Author: Peter Maydell <address@hidden>
  Date:   2016-02-03 (Wed, 03 Feb 2016)

  Changed paths:
    M .travis.yml
    M Makefile.objs
    M bsd-user/main.c
    M configure
    M cpu-exec.c
    M exec.c
    M hw/acpi/cpu_hotplug.c
    M hw/timer/a9gtimer.c
    A include/exec/log.h
    M include/qemu/log.h
    M linux-user/main.c
    M qemu-io.c
    R qemu-log.c
    M qemu-options.hx
    M qom/cpu.c
    A scripts/tracetool/backend/log.py
    R scripts/tracetool/backend/stderr.py
    M scripts/tracetool/format/events_c.py
    M target-alpha/translate.c
    M target-arm/translate-a64.c
    M target-arm/translate.c
    M target-cris/translate.c
    M target-i386/seg_helper.c
    M target-i386/smm_helper.c
    M target-i386/translate.c
    M target-lm32/helper.c
    M target-lm32/translate.c
    M target-m68k/translate.c
    M target-microblaze/helper.c
    M target-microblaze/translate.c
    M target-mips/helper.c
    M target-mips/translate.c
    M target-moxie/translate.c
    M target-openrisc/translate.c
    M target-ppc/mmu-hash32.c
    M target-ppc/mmu-hash64.c
    M target-ppc/mmu_helper.c
    M target-ppc/translate.c
    M target-s390x/translate.c
    M target-sh4/helper.c
    M target-sh4/translate.c
    M target-sparc/int32_helper.c
    M target-sparc/int64_helper.c
    M target-sparc/translate.c
    M target-tilegx/translate.c
    M target-tricore/translate.c
    M target-unicore32/translate.c
    M target-xtensa/translate.c
    M tcg/tcg.c
    M trace/control-internal.h
    M trace/control.c
    M trace/control.h
    M trace/event-internal.h
    M trace/simple.c
    M trace/simple.h
    M translate-all.c
    M util/Makefile.objs
    A util/log.c
    M vl.c

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

# gpg: Signature made Wed 03 Feb 2016 15:47:34 GMT 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:
  log: add "-d trace:PATTERN"
  trace: switch default backend to "log"
  trace: convert stderr backend to log
  log: move qemu-log.c into util/ directory
  log: do not unnecessarily include qom/cpu.h
  trace: add "-trace help"
  trace: add "-trace enable=..."
  trace: no need to call trace_backend_init in different branches now
  trace: split trace_init_file out of trace_init_backends
  trace: split trace_init_events out of trace_init_backends
  trace: fix documentation
  trace: track enabled events in a separate array
  trace: count number of enabled events

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


Compare: https://github.com/qemu/qemu/compare/87574621b18f...382d34ff9fcc
[Prev in Thread]
Current Thread
[Next in Thread]
[Qemu-commits] [qemu/qemu] 43b48c: trace: count number of enabled events, GitHub <=
Prev by Date: [Qemu-commits] [qemu/qemu] cb9ab7: zap qemu_egl_has_ext in include/ui/egl-helpers.h
Next by Date: [Qemu-commits] [qemu/qemu] a43e68: virt-acpi-build: add always-on property for timer
Previous by thread: [Qemu-commits] [qemu/qemu] cb9ab7: zap qemu_egl_has_ext in include/ui/egl-helpers.h
Next by thread: [Qemu-commits] [qemu/qemu] a43e68: virt-acpi-build: add always-on property for timer
Index(es):
- Date
- Thread