From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from e28smtp07.in.ibm.com (e28smtp07.in.ibm.com [122.248.162.7]) (using TLSv1 with cipher CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 8837D1A0008 for ; Wed, 13 May 2015 02:06:57 +1000 (AEST) Received: from /spool/local by e28smtp07.in.ibm.com with IBM ESMTP SMTP Gateway: Authorized Use Only! Violators will be prosecuted for from ; Tue, 12 May 2015 21:36:53 +0530 Received: from d28relay03.in.ibm.com (d28relay03.in.ibm.com [9.184.220.60]) by d28dlp03.in.ibm.com (Postfix) with ESMTP id 8889F125804B for ; Tue, 12 May 2015 21:39:01 +0530 (IST) Received: from d28av01.in.ibm.com (d28av01.in.ibm.com [9.184.220.63]) by d28relay03.in.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id t4CG6Pk765863740 for ; Tue, 12 May 2015 21:36:49 +0530 Received: from d28av01.in.ibm.com (localhost [127.0.0.1]) by d28av01.in.ibm.com (8.14.4/8.14.4/NCO v10.0 AVout) with ESMTP id t4CG4saA032360 for ; Tue, 12 May 2015 21:34:55 +0530 Message-ID: <555224A4.5050502@linux.vnet.ibm.com> Date: Tue, 12 May 2015 21:34:52 +0530 From: Hemant Kumar MIME-Version: 1.0 To: Scott Wood Subject: Re: [PATCH v3 1/2] perf/kvm: Port perf kvm to powerpc References: <1431047266-6206-1-git-send-email-hemant@linux.vnet.ibm.com> <1431382120.16357.471.camel@freescale.com> In-Reply-To: <1431382120.16357.471.camel@freescale.com> Content-Type: text/plain; charset=UTF-8; format=flowed Cc: maddy@linux.vnet.ibm.com, srikar@linux.vnet.ibm.com, peterz@infradead.org, linux-kernel@vger.kernel.org, kvm-ppc@vger.kernel.org, agraf@suse.de, mingo@redhat.com, paulus@samba.org, acme@kernel.org, warrier@linux.vnet.ibm.com, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Hi Scott, On 05/12/2015 03:38 AM, Scott Wood wrote: > On Fri, 2015-05-08 at 06:37 +0530, Hemant Kumar wrote: >> From: Srikar Dronamraju >> >> perf kvm can be used to analyze guest exit reasons. This support already >> exists in x86. Hence, porting it to powerpc. >> >> - To trace KVM events : >> perf kvm stat record >> If many guests are running, we can track for a specific guest by using >> --pid as in : perf kvm stat record --pid >> >> - To see the results : >> perf kvm stat report >> >> The result shows the number of exits (from the guest context to >> host/hypervisor context) grouped by their respective exit reasons with >> their frequency. >> >> This patch makes use of the guest exit reasons available in >> "trace_book3s.h". It records on two already available tracepoints : >> "kvm_hv:kvm_guest_exit" and "kvm_hv:kvm_guest_enter". >> >> Note : This patch has a dependency on the patch "kvm/powerpc: Export >> kvm exit reasons" which exports the KVM exit reasons through the uapi. >> >> Here is a sample o/p: >> # pgrep qemu >> 19378 >> 60515 >> >> 2 Guests are running on the host. >> >> # perf kvm stat record -a >> ^C[ perf record: Woken up 1 times to write data ] >> [ perf record: Captured and wrote 4.153 MB perf.data.guest (39624 samples) ] >> >> # perf kvm stat report -p 60515 >> Analyze events for pid(s) 60515, all VCPUs: >> >> VM-EXIT Samples Samples% Time% Min Time Max Time Avg time >> >> H_DATA_STORAGE 5006 35.30% 0.13% 1.94us 49.46us 12.37us ( +- 0.52% ) >> HV_DECREMENTER 4457 31.43% 0.02% 0.72us 16.14us 1.91us ( +- 0.96% ) >> SYSCALL 2690 18.97% 0.10% 2.84us 528.24us 18.29us ( +- 3.75% ) >> RETURN_TO_HOST 1789 12.61% 99.76% 1.58us 672791.91us 27470.23us ( +- 3.00% ) >> EXTERNAL 240 1.69% 0.00% 0.69us 10.67us 1.33us ( +- 5.34% ) >> >> Total Samples:14182, Total events handled time:49264158.30us. >> >> Signed-off-by: Srikar Dronamraju >> Signed-off-by: Hemant Kumar >> --- >> Patch has a dependency on : https://patchwork.ozlabs.org/patch/469839/ >> which exports the exit reasons to perf through uapi. >> >> Changes: >> - Original series split into two patchsets now : perf and powerpc >> side changes. >> >> arch/powerpc/include/uapi/asm/kvm_perf.h | 15 +++++++++++++++ >> tools/perf/arch/powerpc/Makefile | 1 + >> tools/perf/arch/powerpc/util/Build | 1 + >> tools/perf/arch/powerpc/util/kvm-stat.c | 33 ++++++++++++++++++++++++++++++++ >> 4 files changed, 50 insertions(+) >> create mode 100644 arch/powerpc/include/uapi/asm/kvm_perf.h >> create mode 100644 tools/perf/arch/powerpc/util/kvm-stat.c >> >> 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..30fa670 >> --- /dev/null >> +++ b/arch/powerpc/include/uapi/asm/kvm_perf.h >> @@ -0,0 +1,15 @@ >> +#ifndef _ASM_POWERPC_KVM_PERF_H >> +#define _ASM_POWERPC_KVM_PERF_H >> + >> +#include >> +#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_H */ > Please make sure that anything book3s-specific is named that way. Are you suggesting to name it to something like _ASM_POWERPC_BOOK3S_PERF_H ? > And shouldn't this be part of the arch/powerpc-side patchset? It should. Thanks, will move this to arch/powerpc side patchset. >> diff --git a/tools/perf/arch/powerpc/Makefile b/tools/perf/arch/powerpc/Makefile >> index 7fbca17..21322e0 100644 >> --- a/tools/perf/arch/powerpc/Makefile >> +++ b/tools/perf/arch/powerpc/Makefile >> @@ -1,3 +1,4 @@ >> ifndef NO_DWARF >> PERF_HAVE_DWARF_REGS := 1 >> endif >> +HAVE_KVM_STAT_SUPPORT := 1 > Does this stuff fail gracefully if used on a PPC target that doesn't > support this? Yes, it does. > -Scott > > > _______________________________________________ > Linuxppc-dev mailing list > Linuxppc-dev@lists.ozlabs.org > https://lists.ozlabs.org/listinfo/linuxppc-dev -- Thanks, Hemant Kumar