Re: [PATCH 07/10] trace: remove control-vcpu.h

[Philippe Mathieu-Daudé]

On 20/4/23 17:00, Alex Bennée wrote:
> Now we no longer have vcpu controlled trace events we can excise the
> code that allows us to query its status.
>
> Signed-off-by: Alex Bennée <alex.bennee@linaro.org>
> ---
>   trace/control-vcpu.h          | 47 -----------------------------------
>   trace/qmp.c                   |  2 +-
>   scripts/tracetool/format/h.py |  5 +---
>   3 files changed, 2 insertions(+), 52 deletions(-)
>   delete mode 100644 trace/control-vcpu.h


> diff --git a/scripts/tracetool/format/h.py b/scripts/tracetool/format/h.py
> index 285d7b03a9..ea126b07ea 100644
> --- a/scripts/tracetool/format/h.py
> +++ b/scripts/tracetool/format/h.py
> @@ -16,10 +16,7 @@
>   
>   
>   def generate(events, backend, group):
> -    if group == "root":
> -        header = "trace/control-vcpu.h"
> -    else:
> -        header = "trace/control.h"
> +    header = "trace/control.h"

Just include it in place in the template :) With that:

Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>

>   
>       out('/* This file is autogenerated by tracetool, do not edit. */',
>           '',