Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753141AbcLFPCZ convert rfc822-to-8bit (ORCPT ); Tue, 6 Dec 2016 10:02:25 -0500 Received: from mga04.intel.com ([192.55.52.120]:37960 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751064AbcLFPCY (ORCPT ); Tue, 6 Dec 2016 10:02:24 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.33,310,1477983600"; d="scan'208";a="39569663" From: "Liang, Kan" To: Peter Zijlstra CC: "mingo@redhat.com" , "acme@kernel.org" , "linux-kernel@vger.kernel.org" , "alexander.shishkin@linux.intel.com" , "tglx@linutronix.de" , "namhyung@kernel.org" , "jolsa@kernel.org" , "Hunter, Adrian" , "wangnan0@huawei.com" , "mark.rutland@arm.com" , "andi@firstfloor.org" Subject: RE: [PATCH V2 03/13] perf/x86: output sampling overhead Thread-Topic: [PATCH V2 03/13] perf/x86: output sampling overhead Thread-Index: AQHSTOHSuMQJFJKXz0+jqt27jlukiaD6RYyAgADDP4A= Date: Tue, 6 Dec 2016 15:02:20 +0000 Message-ID: <37D7C6CF3E00A74B8858931C1DB2F07750CA9EAE@SHSMSX103.ccr.corp.intel.com> References: <1480713561-6617-1-git-send-email-kan.liang@intel.com> <1480713561-6617-4-git-send-email-kan.liang@intel.com> <20161206112013.GJ3124@twins.programming.kicks-ass.net> In-Reply-To: <20161206112013.GJ3124@twins.programming.kicks-ass.net> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiNzc1MzhiY2QtMjFhMi00NmU0LTg1MmUtMWY4MzVmOWZiZDkxIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE1LjkuNi42IiwiVHJ1c3RlZExhYmVsSGFzaCI6IjJmV2d5aVJhOFg5XC9SNkQzMTEzTllIanY0ZzgzdHRtNFplWTJcL284VkZ2az0ifQ== x-ctpclassification: CTP_IC x-originating-ip: [10.239.127.40] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1754 Lines: 58 > On Fri, Dec 02, 2016 at 04:19:11PM -0500, kan.liang@intel.com wrote: > > From: Kan Liang > > > > On x86, NMI handler is the most important part which brings overhead > > for sampling. Adding a pmu specific overhead type > > PERF_PMU_SAMPLE_OVERHEAD for it. > > > > For other architectures which may don't have NMI, the overhead type > > can be reused. > > > > Signed-off-by: Kan Liang > > --- > > arch/x86/events/core.c | 17 ++++++++++++++++- > > arch/x86/events/perf_event.h | 2 ++ > > include/uapi/linux/perf_event.h | 1 + > > 3 files changed, 19 insertions(+), 1 deletion(-) > > > > diff --git a/arch/x86/events/core.c b/arch/x86/events/core.c index > > 9d4bf3a..de40f96 100644 > > --- a/arch/x86/events/core.c > > +++ b/arch/x86/events/core.c > > @@ -1397,6 +1397,9 @@ static void x86_pmu_del(struct perf_event > > *event, int flags) > > > > perf_event_update_userpage(event); > > > > + if ((flags & PERF_EF_LOG) && cpuc->nmi_overhead.nr) > > + perf_log_overhead(event, PERF_PMU_SAMPLE_OVERHEAD, > > +&cpuc->nmi_overhead); > > + > > do_del: > > if (x86_pmu.del) { > > /* > > That's not at all mentioned in the changelog, and it clearly isn't > nmi_overhead. Here it only records the overhead, not calculate. The calculation is in nmi_hanlder as below. I will make it clear in the changelog. @@ -1492,8 +1505,10 @@ perf_event_nmi_handler(unsigned int cmd, struct pt_regs *regs) start_clock = sched_clock(); ret = x86_pmu.handle_irq(regs); finish_clock = sched_clock(); + clock = finish_clock - start_clock; - perf_sample_event_took(finish_clock - start_clock); + perf_calculate_nmi_overhead(clock); + perf_sample_event_took(clock); return ret; } Thanks, Kan