Re: [PATCH] KVM: trace events: update list of exit reasons

[Cornelia Huck <cohuck@redhat.com>]

On Thu, 14 Sep 2017 11:50:07 +0200
Ladi Prosek <lprosek@redhat.com> wrote:

> Adding entries for exit reasons 23 - 27:
> 
>   KVM_EXIT_EPR
>   KVM_EXIT_SYSTEM_EVENT
>   KVM_EXIT_S390_STSI
>   KVM_EXIT_IOAPIC_EOI
>   KVM_EXIT_HYPERV
> 
> Signed-off-by: Ladi Prosek <lprosek@redhat.com>
> ---
>  include/trace/events/kvm.h | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/include/trace/events/kvm.h b/include/trace/events/kvm.h
> index 8ade3eb..dcffedf 100644
> --- a/include/trace/events/kvm.h
> +++ b/include/trace/events/kvm.h
> @@ -14,7 +14,9 @@
>  	ERSN(SHUTDOWN), ERSN(FAIL_ENTRY), ERSN(INTR), ERSN(SET_TPR),	\
>  	ERSN(TPR_ACCESS), ERSN(S390_SIEIC), ERSN(S390_RESET), ERSN(DCR),\
>  	ERSN(NMI), ERSN(INTERNAL_ERROR), ERSN(OSI), ERSN(PAPR_HCALL),	\
> -	ERSN(S390_UCONTROL), ERSN(WATCHDOG), ERSN(S390_TSCH)
> +	ERSN(S390_UCONTROL), ERSN(WATCHDOG), ERSN(S390_TSCH), ERSN(EPR),\
> +	ERSN(SYSTEM_EVENT), ERSN(S390_STSI), ERSN(IOAPIC_EOI),          \
> +	ERSN(HYPERV)
>  
>  TRACE_EVENT(kvm_userspace_exit,
>  	    TP_PROTO(__u32 reason, int errno),

Reviewed-by: Cornelia Huck <cohuck@redhat.com>

/me wonders whether there is a way to avoid them running out of sync...
but otoh, not so many exit reasons are being added anyway.