Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752887AbcLFOqb (ORCPT ); Tue, 6 Dec 2016 09:46:31 -0500 Received: from mail.kernel.org ([198.145.29.136]:53522 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751779AbcLFOq3 (ORCPT ); Tue, 6 Dec 2016 09:46:29 -0500 Date: Tue, 6 Dec 2016 11:46:24 -0300 From: Arnaldo Carvalho de Melo To: Alexis Berlemont Cc: linux-kernel@vger.kernel.org, peterz@infradead.org, mingo@redhat.com, alexander.shishkin@linux.intel.com Subject: Re: [PATCH v3 0/2] perf: measure page fault duration in perf trace Message-ID: <20161206144624.GC2399@kernel.org> References: <20161027083714.GF3568@worktop.programming.kicks-ass.net> <20161118002138.29150-1-alexis.berlemont@gmail.com> <20161205234006.GA22075@therese> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161205234006.GA22075@therese> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.7.1 (2016-10-04) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1193 Lines: 40 Em Tue, Dec 06, 2016 at 12:40:06AM +0100, Alexis Berlemont escreveu: > Alexis Berlemont wrote: > > Hi, > > > > Here is another attempt to make the perf-trace subcommand print the > > page fault durations. > > > > This solution is based on static tracepoints and it was necessary to > > add 2 more tracepoints. I understood that adding tracepoints is a > > questionable solution; I can try something else if someone has another > > idea. > > > > Could you indicate me a way to improve these patches so as to move > forward ? I have to test this, and review PeterZ latest comment about it, to see if all is well with him by now, Peter? - Arnaldo > Regards, > > Alexis. > > > Alexis. > > > > Alexis Berlemont (2): > > perf, x86-mm: declare page-faults tracepoints like irq-vectors ones > > perf: add page fault duration measures in perf trace > > > > arch/x86/include/asm/trace/exceptions.h | 17 ++- > > arch/x86/mm/fault.c | 17 ++- > > tools/perf/Documentation/perf-trace.txt | 4 +- > > tools/perf/builtin-trace.c | 231 ++++++++++++++++++++++++++++---- > > 4 files changed, 238 insertions(+), 31 deletions(-) > > > > -- > > 2.10.2 > >