Re: [Xen-devel] [PATCH v4 4/5] xentrace: enable per-VCPU extratime flag for RTDS

[Dario Faggioli <raistlin@xxxxxxxx>]

On Wed, 2017-10-11 at 14:02 -0400, Meng Xu wrote:
> Change repl_budget event output for xentrace formats and xenalyze
> 
> Signed-off-by: Meng Xu <mengxu@xxxxxxxxxxxxx>
> 
I'd say:

Reviewed-by: Dario Faggioli <raistlin@xxxxxxxx>

However...

> diff --git a/tools/xentrace/xenalyze.c b/tools/xentrace/xenalyze.c
> index 79bdba7..19e050f 100644
> --- a/tools/xentrace/xenalyze.c
> +++ b/tools/xentrace/xenalyze.c
> @@ -7935,23 +7935,29 @@ void sched_process(struct pcpu_info *p)
>                      unsigned int vcpuid:16, domid:16;
>                      uint64_t cur_bg;
>                      int delta;
> +                    unsigned priority_level;
> +                    unsigned has_extratime;
>
...this last field is 'bool' in Xen.

I appreciate that xenalyze does not build if you just make this bool as
well. But it does build for me, if you do that, and also include
stdbool.h, which I think is a fine thing to do.

Anyway, I'll leave this to George and tools' maintainers.

Regards,
Dario
-- 
<<This happens because I choose it to happen!>> (Raistlin Majere)
-----------------------------------------------------------------
Dario Faggioli, Ph.D, http://about.me/dario.faggioli

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
https://lists.xen.org/xen-devel