Re: [Qemu-devel] [PATCH v7 0/9] qemu-log, -dfilter and other logging tweaks

[Alex Bennée]

Alex Bennée <address@hidden> writes:

> As soft-freeze is approaching I'm trying to get these off my queue. v6
> missed out one of the review comments - now I believe all review
> comments since v5 have been addressed. The two patches that have seen
> the most change are:
>
>      qemu-log: new option -dfilter to limit output
>      qemu-log: support simple pid substitution for logs
>
> The later having been moved up the patch queue as both now include
> unit tests to defend their parsing behaviour.
>
> As usual the details are in the individual commits. I think these
> patches are now ready to be picked up by the TCG tree?

Ping?

Any chance of a final sign-off and merge before soft-freeze?

>
> Alex Bennée (7):
>   tcg: pass down TranslationBlock to tcg_code_gen
>   qemu-log: correct help text for -d cpu
>   qemu-log: new option -dfilter to limit output
>   qemu-log: dfilter-ise exec, out_asm, op and opt_op
>   target-arm: dfilter support for in_asm
>   qemu-log: support simple pid substitution for logs
>   cputlb: modernise the debug support
>
> Peter Maydell (2):
>   qemu-log: Avoid function call for disabled qemu_log_mask logging
>   qemu-log: Improve the "exec" TB execution logging
>
>  cpu-exec.c                 |  21 ++++----
>  cputlb.c                   |  54 +++++++++++++-------
>  include/exec/exec-all.h    |   5 ++
>  include/qemu/log.h         |  28 ++++++++++-
>  qemu-options.hx            |  18 +++++++
>  target-arm/translate-a64.c |   3 +-
>  target-arm/translate.c     |   3 +-
>  tcg/tcg.c                  |  12 +++--
>  tcg/tcg.h                  |   2 +-
>  tests/Makefile             |   4 ++
>  tests/test-logging.c       | 114 ++++++++++++++++++++++++++++++++++++++++++
>  translate-all.c            |   5 +-
>  util/log.c                 | 120 
> +++++++++++++++++++++++++++++++++++++++------
>  vl.c                       |   3 ++
>  14 files changed, 338 insertions(+), 54 deletions(-)
>  create mode 100644 tests/test-logging.c


--
Alex Bennée