Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752469AbcKXXpg (ORCPT ); Thu, 24 Nov 2016 18:45:36 -0500 Received: from mx1.redhat.com ([209.132.183.28]:38814 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750804AbcKXXp2 (ORCPT ); Thu, 24 Nov 2016 18:45:28 -0500 Date: Fri, 25 Nov 2016 00:45:22 +0100 From: Jiri Olsa To: Namhyung Kim Cc: "Liang, Kan" , "peterz@infradead.org" , "mingo@redhat.com" , "acme@kernel.org" , "linux-kernel@vger.kernel.org" , "alexander.shishkin@linux.intel.com" , "tglx@linutronix.de" , "jolsa@kernel.org" , "Hunter, Adrian" , "wangnan0@huawei.com" , "mark.rutland@arm.com" , "andi@firstfloor.org" Subject: Re: [PATCH 06/14] perf tools: show NMI overhead Message-ID: <20161124234522.GA20128@krava> References: <1479894292-16277-1-git-send-email-kan.liang@intel.com> <1479894292-16277-7-git-send-email-kan.liang@intel.com> <20161123225151.GC15978@krava> <37D7C6CF3E00A74B8858931C1DB2F07750CA2BBA@SHSMSX103.ccr.corp.intel.com> <20161124152721.GA5250@krava> <20161124232013.GA28557@sejong> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161124232013.GA28557@sejong> User-Agent: Mutt/1.7.1 (2016-10-04) X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.28]); Thu, 24 Nov 2016 23:45:27 +0000 (UTC) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1691 Lines: 50 On Fri, Nov 25, 2016 at 08:20:13AM +0900, Namhyung Kim wrote: > Hi, > > On Thu, Nov 24, 2016 at 04:27:21PM +0100, Jiri Olsa wrote: > > On Thu, Nov 24, 2016 at 01:37:04PM +0000, Liang, Kan wrote: > > > > > > > > > > > > > > On Wed, Nov 23, 2016 at 04:44:44AM -0500, kan.liang@intel.com wrote: > > > > > From: Kan Liang > > > > > > > > > > Caculate the total NMI overhead on each CPU, and display them in perf > > > > > report > > > > > > > > so the output looks like this: > > > > > > > > --- > > > > # Elapsed time: 1720167944 ns > > > > # Overhead: > > > > # CPU 6 > > > > # NMI#: 27 time: 111379 ns > > > > # Multiplexing#: 0 time: 0 ns > > > > # SB#: 57 time: 90045 ns > > > > # > > > > # Samples: 26 of event 'cycles:u' > > > > # Event count (approx.): 1677531 > > > > # > > > > # Overhead Command Shared Object Symbol > > > > # ........ ....... ................ ....................... > > > > # > > > > 24.20% ls ls [.] _init > > > > 17.18% ls libc-2.24.so [.] __strcoll_l > > > > 11.85% ls ld-2.24.so [.] _dl_relocate_object > > > > --- > > > > how about we display the overhead information same way the main perf output: > > > > CPU NMI NMI time MTX MTX time SB SB time > > ... ..... ........ ..... ........ ...... ........ > > 6 27 111379 0 0 57 90045 > > > > > > would be just matter of adding new sort objects > > How would you connect those to hist entries then? It'd be possible if > the sort key had 'cpu' only, no? right, I should have said fields then.. jirka