Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751529AbdIMP3t convert rfc822-to-8bit (ORCPT ); Wed, 13 Sep 2017 11:29:49 -0400 Received: from mga09.intel.com ([134.134.136.24]:58127 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751062AbdIMP3s (ORCPT ); Wed, 13 Sep 2017 11:29:48 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.42,388,1500966000"; d="scan'208";a="151421670" From: "Liang, Kan" To: Arnaldo Carvalho de Melo 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 Thread-Topic: [PATCH RFC V2 00/10] perf top optimization Thread-Index: AQHTKqUGKqa3XFGJ30eMjpDMlJ7smaKybY4AgACGmHA= Date: Wed, 13 Sep 2017 15:29:44 +0000 Message-ID: <37D7C6CF3E00A74B8858931C1DB2F077537AF31A@SHSMSX103.ccr.corp.intel.com> References: <1505096603-215017-1-git-send-email-kan.liang@intel.com> <20170913152506.GK5866@kernel.org> In-Reply-To: <20170913152506.GK5866@kernel.org> Accept-Language: zh-CN, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiZDNmNDA4NjgtY2IxMi00MDM1LWE5Y2QtNmU1NzkxNGE2MTM4IiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX0lDIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE2LjUuOS4zIiwiVHJ1c3RlZExhYmVsSGFzaCI6InB0MXcyOUZSMFhJc3VWYVI2d0hNMmJwSGlEOFI5bEVYb2Q3ejh1UmRqNFk9In0= x-ctpclassification: CTP_IC dlp-product: dlpe-windows dlp-version: 11.0.0.116 dlp-reaction: no-action 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: 2381 Lines: 60 > > 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? Thanks, Kan > Thanks, > > - Arnaldo > > > Changes since V1: > > - Patch 1: machine threads and hashtable related renaming (Arnaldo) > > - Patch 6: use a smaller locked section for comm_str__put > > add a locked wrapper for comm_str__findnew (Arnaldo) > > > > Kan Liang (10): > > perf tools: hashtable for machine threads > > perf tools: using scandir to replace readdir > > petf tools: using comm_str to replace comm in hist_entry > > petf tools: introduce a new function to set namespaces id > > perf tools: lock to protect thread list > > perf tools: lock to protect comm_str rb tree > > perf tools: change machine comm_exec type to atomic > > perf top: implement multithreading for perf_event__synthesize_threads > > perf top: add option to set the number of thread for event synthesize > > perf top: switch back to overwrite mode > > > > tools/perf/builtin-kvm.c | 3 +- > > tools/perf/builtin-record.c | 2 +- > > tools/perf/builtin-top.c | 9 +- > > tools/perf/builtin-trace.c | 21 +++-- > > tools/perf/tests/mmap-thread-lookup.c | 2 +- > > tools/perf/ui/browsers/hists.c | 2 +- > > tools/perf/util/comm.c | 18 +++- > > tools/perf/util/event.c | 149 +++++++++++++++++++++++++------- > > tools/perf/util/event.h | 14 ++- > > tools/perf/util/evlist.c | 5 +- > > tools/perf/util/hist.c | 11 +-- > > tools/perf/util/machine.c | 158 +++++++++++++++++++++------------- > > tools/perf/util/machine.h | 34 ++++++-- > > tools/perf/util/rb_resort.h | 5 +- > > tools/perf/util/sort.c | 8 +- > > tools/perf/util/sort.h | 2 +- > > tools/perf/util/thread.c | 68 ++++++++++++--- > > tools/perf/util/thread.h | 6 +- > > tools/perf/util/top.h | 1 + > > 19 files changed, 376 insertions(+), 142 deletions(-) > > > > -- > > 2.5.5