From mboxrd@z Thu Jan 1 00:00:00 1970 From: Cornelia Huck Subject: Re: [PATCH] KVM: trace events: update list of exit reasons Date: Thu, 14 Sep 2017 12:06:46 +0200 Message-ID: <20170914120646.49139c2a.cohuck@redhat.com> References: <20170914095007.21331-1-lprosek@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Cc: kvm@vger.kernel.org To: Ladi Prosek Return-path: Received: from mx1.redhat.com ([209.132.183.28]:50190 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751277AbdINKGu (ORCPT ); Thu, 14 Sep 2017 06:06:50 -0400 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id B5E85155E1 for ; Thu, 14 Sep 2017 10:06:50 +0000 (UTC) In-Reply-To: <20170914095007.21331-1-lprosek@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On Thu, 14 Sep 2017 11:50:07 +0200 Ladi Prosek 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 > --- > 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 /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.