Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751950AbbG2Kh0 (ORCPT ); Wed, 29 Jul 2015 06:37:26 -0400 Received: from e28smtp03.in.ibm.com ([122.248.162.3]:52255 "EHLO e28smtp03.in.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750864AbbG2KhY (ORCPT ); Wed, 29 Jul 2015 06:37:24 -0400 X-Helo: d28dlp01.in.ibm.com X-MailFrom: hemant@linux.vnet.ibm.com X-RcptTo: linux-kernel@vger.kernel.org Message-ID: <55B8ACD9.8090404@linux.vnet.ibm.com> Date: Wed, 29 Jul 2015 16:07:13 +0530 From: Hemant Kumar User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0 MIME-Version: 1.0 To: Scott Wood CC: maddy@linux.vnet.ibm.com, srikar@linux.vnet.ibm.com, linux-kernel@vger.kernel.org, acme@kernel.org, paulus@samba.org, warrier@linux.vnet.ibm.com, sukadev@linux.vnet.ibm.com, linuxppc-dev@lists.ozlabs.org, mingo@kernel.org Subject: Re: [PATCH v5 1/2] perf,kvm/ppc: Add kvm_perf.h for powerpc References: <1437061722-15438-1-git-send-email-hemant@linux.vnet.ibm.com> <1437077417.2993.108.camel@freescale.com> In-Reply-To: <1437077417.2993.108.camel@freescale.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 15072910-0009-0000-0000-000006CC141A Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2798 Lines: 87 Hi Scott, On 07/17/2015 01:40 AM, Scott Wood wrote: > On Thu, 2015-07-16 at 21:18 +0530, Hemant Kumar wrote: >> To analyze the exit events with perf, we need kvm_perf.h to be added in >> the arch/powerpc directory, where the kvm tracepoints needed to trace >> the KVM exit events are defined. >> >> This patch adds "kvm_perf_book3s.h" to indicate that the tracepoints are >> book3s specific. Generic "kvm_perf.h" then can just include >> "kvm_perf_book3s.h". >> >> Signed-off-by: Hemant Kumar >> --- >> Changes: >> - Not exporting the exit reasons compared to previous patchset (suggested >> by Paul) >> >> arch/powerpc/include/uapi/asm/kvm_perf.h | 6 ++++++ >> arch/powerpc/include/uapi/asm/kvm_perf_book3s.h | 14 ++++++++++++++ >> 2 files changed, 20 insertions(+) >> create mode 100644 arch/powerpc/include/uapi/asm/kvm_perf.h >> create mode 100644 arch/powerpc/include/uapi/asm/kvm_perf_book3s.h >> >> diff --git a/arch/powerpc/include/uapi/asm/kvm_perf.h >> b/arch/powerpc/include/uapi/asm/kvm_perf.h >> new file mode 100644 >> index 0000000..5ed2ff3 >> --- /dev/null >> +++ b/arch/powerpc/include/uapi/asm/kvm_perf.h >> @@ -0,0 +1,6 @@ >> +#ifndef _ASM_POWERPC_KVM_PERF_H >> +#define _ASM_POWERPC_KVM_PERF_H >> + >> +#include >> + >> +#endif >> diff --git a/arch/powerpc/include/uapi/asm/kvm_perf_book3s.h >> b/arch/powerpc/include/uapi/asm/kvm_perf_book3s.h >> new file mode 100644 >> index 0000000..8c8d8c2 >> --- /dev/null >> +++ b/arch/powerpc/include/uapi/asm/kvm_perf_book3s.h >> @@ -0,0 +1,14 @@ >> +#ifndef _ASM_POWERPC_KVM_PERF_BOOK3S_H >> +#define _ASM_POWERPC_KVM_PERF_BOOK3S_H >> + >> +#include >> + >> +#define DECODE_STR_LEN 20 >> + >> +#define VCPU_ID "vcpu_id" >> + >> +#define KVM_ENTRY_TRACE "kvm_hv:kvm_guest_enter" >> +#define KVM_EXIT_TRACE "kvm_hv:kvm_guest_exit" >> +#define KVM_EXIT_REASON "trap" >> + >> +#endif /* _ASM_POWERPC_KVM_PERF_BOOK3S_H */ > Again, why is book3s stuff being presented via uapi as generic > with generic symbol names? > > -Scott Ok. We can change the KVM_ENTRY_TRACE macro to something like KVM_BOOK3S_ENTRY_TRACE and likewise for KVM_EXIT_TRACE and KVM_EXIT_REASON and then, to resolve the issue of generic macro names in the userspace side, we can handle it using __weak modifier. What would you suggest? > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev -- Thanks, Hemant Kumar -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/