Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751611AbdINVTv (ORCPT ); Thu, 14 Sep 2017 17:19:51 -0400 Received: from mail.kernel.org ([198.145.29.99]:47956 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751366AbdINVTu (ORCPT ); Thu, 14 Sep 2017 17:19:50 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org ADA6B21BCE Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=acme@kernel.org Date: Thu, 14 Sep 2017 18:19:46 -0300 From: Arnaldo Carvalho de Melo To: "Liang, Kan" Cc: "peterz@infradead.org" , "mingo@redhat.com" , "linux-kernel@vger.kernel.org" , "jolsa@kernel.org" , "namhyung@kernel.org" , "Hunter, Adrian" , "Odzioba, Lukasz" , "ak@linux.intel.com" Subject: Re: [PATCH RFC V2 00/10] perf top optimization Message-ID: <20170914211946.GB10371@kernel.org> References: <1505096603-215017-1-git-send-email-kan.liang@intel.com> <20170913152506.GK5866@kernel.org> <37D7C6CF3E00A74B8858931C1DB2F077537AF31A@SHSMSX103.ccr.corp.intel.com> <20170913153819.GL5866@kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170913153819.GL5866@kernel.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.8.3 (2017-05-23) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2019 Lines: 41 Em Wed, Sep 13, 2017 at 12:38:19PM -0300, Arnaldo Carvalho de Melo escreveu: > Em Wed, Sep 13, 2017 at 03:29:44PM +0000, Liang, Kan escreveu: > > > > > > Em Sun, Sep 10, 2017 at 07:23:13PM -0700, kan.liang@intel.com escreveu: > > > > > > So I got the first two patches already merged, and made some comments > > > about the other patches, please check those, > > > > > > > Thanks for the review Arnaldo. > > > > I will take a close look for the comments. > > For the next version, I only need to include patch 3-10, correct? > > Right, and go from my perf/core branch. The hashtable patch is still not > there as I am running tests before pushing out, but it should be there > later today. So, its at my repo, branch tmp.perf/threads_hashtable But 'perf trace' is broken, please take a look below: [root@jouet ~]# gdb -c core GNU gdb (GDB) Fedora 8.0-20.fc26 Core was generated by `perf trace -e block:block_bio_queue'. Program terminated with signal SIGSEGV, Segmentation fault. #0 0x000000000051089a in ?? () (gdb) file perf Reading symbols from perf...done. (gdb) bt #0 0x000000000051089a in ____machine__findnew_thread (machine=0x3dfcab0, threads=0x3dfca78, pid=-1, tid=-1, create=false) at util/machine.c:429 #1 0x0000000000510b49 in machine__find_thread (machine=0x3dfcab0, pid=-1, tid=-1) at util/machine.c:498 #2 0x0000000000483dd0 in trace__set_filter_loop_pids (trace=0x7ffc0d23f880) at builtin-trace.c:2247 #3 0x000000000048438b in trace__run (trace=0x7ffc0d23f880, argc=0, argv=0x7ffc0d242a00) at builtin-trace.c:2385 #4 0x0000000000487219 in cmd_trace (argc=0, argv=0x7ffc0d242a00) at builtin-trace.c:3121 #5 0x00000000004c0b4a in run_builtin (p=0xa75d00 , argc=3, argv=0x7ffc0d242a00) at perf.c:296 #6 0x00000000004c0db7 in handle_internal_command (argc=3, argv=0x7ffc0d242a00) at perf.c:348 #7 0x00000000004c0f09 in run_argv (argcp=0x7ffc0d24285c, argv=0x7ffc0d242850) at perf.c:392 #8 0x00000000004c12d7 in main (argc=3, argv=0x7ffc0d242a00) at perf.c:536 (gdb)