From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755714AbcBXJIu (ORCPT ); Wed, 24 Feb 2016 04:08:50 -0500 Received: from e23smtp04.au.ibm.com ([202.81.31.146]:39437 "EHLO e23smtp04.au.ibm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751814AbcBXJIp (ORCPT ); Wed, 24 Feb 2016 04:08:45 -0500 X-IBM-Helo: d23dlp03.au.ibm.com X-IBM-MailFrom: ravi.bangoria@linux.vnet.ibm.com X-IBM-RcptTo: linux-kernel@vger.kernel.org From: Ravi Bangoria To: acme@kernel.org Cc: linux-kernel@vger.kernel.org, hemant@linux.vnet.ibm.com, naveen.n.rao@linux.vnet.ibm.com, Ravi Bangoria Subject: [RFC 0/4] perf kvm: Guest Symbol Resolution for powerpc Date: Wed, 24 Feb 2016 14:37:41 +0530 Message-Id: <1456304865-15131-1-git-send-email-ravi.bangoria@linux.vnet.ibm.com> X-Mailer: git-send-email 2.1.4 X-TM-AS-MML: disable X-Content-Scanned: Fidelis XPS MAILER x-cbid: 16022409-0013-0000-0000-000002CBE03A Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Design of [patch v2] Guest Symbol Resolution is focused on enabling perf kvm {record|report} on powerpc. Here is the link for the same: thread.gmane.org/gmane.linux.kernel/2132409 As per the point raised by acme, this design does not enable cross arch reporting functionality. i.e. record on powerpc and report on !powerpc. This patch aims to enable cross arch reporting functionality along with enabling perf kvm {record|report} on powerpc. Note that basic principle of enabling perf kvm {record|report} on powerpc using tracepoint kvm_hv:kvm_guest_exit has not been changed. Major change between [patch v2] and this [RFC] patch is, I've moved 'perf kvm report' related and ppc specific functionality from tool/perf/arch/powerpc/ to generic tool/perf/ code. This is required because perf binary needs ppc specific code even if it's compiled on !ppc to enable cross arch reporting. I need suggestion specifically on patch 3 (Enable 'report' on powerpc) which contains arch specific code in generic area. Right now I've added code in util/evsel.c. But please let me know if there's any better way to do this. This patch is to get suggestions on approach so I've tagged it as RFC and not following the patch version series. Ravi Bangoria (4): perf kvm: Enable 'record' on powerpc perf kvm: Introduce evsel as argument to perf_event__preprocess_sample perf kvm: Enable 'report' on powerpc perf kvm: Fix output fields instead of 'trace' for perf kvm report on powerpc tools/perf/arch/powerpc/util/Build | 1 + tools/perf/arch/powerpc/util/kvm.c | 18 +++++++++ tools/perf/builtin-annotate.c | 3 +- tools/perf/builtin-diff.c | 3 +- tools/perf/builtin-mem.c | 10 +++-- tools/perf/builtin-report.c | 8 +++- tools/perf/builtin-script.c | 3 +- tools/perf/builtin-timechart.c | 8 ++-- tools/perf/builtin-top.c | 3 +- tools/perf/tests/hists_cumulate.c | 2 +- tools/perf/tests/hists_filter.c | 2 +- tools/perf/tests/hists_link.c | 4 +- tools/perf/tests/hists_output.c | 2 +- tools/perf/util/event.c | 8 ++-- tools/perf/util/event.h | 3 +- tools/perf/util/evlist.c | 9 +++++ tools/perf/util/evlist.h | 1 + tools/perf/util/evsel.c | 77 ++++++++++++++++++++++++++++++++++++++ tools/perf/util/evsel.h | 7 ++++ tools/perf/util/session.c | 7 ++-- tools/perf/util/util.c | 5 +++ tools/perf/util/util.h | 1 + 22 files changed, 161 insertions(+), 24 deletions(-) create mode 100644 tools/perf/arch/powerpc/util/kvm.c -- 2.1.4