Re: [Qemu-devel] [PATCH v2 38/40] trace: split out trace events for target-ppc/ directory

[Thomas Huth <thuth@redhat.com>]

On 16.06.2016 10:40, Daniel P. Berrange wrote:
> Move all trace-events for files in the target-ppc/ directory to
> their own file.
> 
> Signed-off-by: Daniel P. Berrange <berrange@redhat.com>
> ---
>  Makefile.objs           | 1 +
>  target-ppc/trace-events | 5 +++++
>  trace-events            | 4 ----
>  3 files changed, 6 insertions(+), 4 deletions(-)
>  create mode 100644 target-ppc/trace-events
> 
> diff --git a/Makefile.objs b/Makefile.objs
> index f1e8646..ad7419b 100644
> --- a/Makefile.objs
> +++ b/Makefile.objs
> @@ -155,3 +155,4 @@ trace-events-y += audio/trace-events
>  trace-events-y += net/trace-events
>  trace-events-y += target-sparc/trace-events
>  trace-events-y += target-s390x/trace-events
> +trace-events-y += target-ppc/trace-events
> diff --git a/target-ppc/trace-events b/target-ppc/trace-events
> new file mode 100644
> index 0000000..d37da2b
> --- /dev/null
> +++ b/target-ppc/trace-events
> @@ -0,0 +1,5 @@
> +# See docs/trace-events.txt for syntax documentation.
> +
> +# target-ppc/kvm.c
> +kvm_failed_spr_set(int str, const char *msg) "Warning: Unable to set SPR %d to KVM: %s"
> +kvm_failed_spr_get(int str, const char *msg) "Warning: Unable to retrieve SPR %d from KVM: %s"
> diff --git a/trace-events b/trace-events
> index 63233e3..343e65a 100644
> --- a/trace-events
> +++ b/trace-events
> @@ -119,10 +119,6 @@ kvm_device_ioctl(int fd, int type, void *arg) "dev fd %d, type 0x%x, arg %p"
>  kvm_failed_reg_get(uint64_t id, const char *msg) "Warning: Unable to retrieve ONEREG %" PRIu64 " from KVM: %s"
>  kvm_failed_reg_set(uint64_t id, const char *msg) "Warning: Unable to set ONEREG %" PRIu64 " to KVM: %s"
>  
> -# target-ppc/kvm.c
> -kvm_failed_spr_set(int str, const char *msg) "Warning: Unable to set SPR %d to KVM: %s"
> -kvm_failed_spr_get(int str, const char *msg) "Warning: Unable to retrieve SPR %d from KVM: %s"
> -
>  # TCG related tracing (mostly disabled by default)
>  # cpu-exec.c
>  disable exec_tb(void *tb, uintptr_t pc) "tb:%p pc=0x%"PRIxPTR
> 

Reviewed-by: Thomas Huth <thuth@redhat.com>