From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754938AbbIHQDf (ORCPT ); Tue, 8 Sep 2015 12:03:35 -0400 Received: from mga02.intel.com ([134.134.136.20]:29404 "EHLO mga02.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753551AbbIHQDe (ORCPT ); Tue, 8 Sep 2015 12:03:34 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,490,1437462000"; d="scan'208";a="800755132" Date: Tue, 8 Sep 2015 09:03:17 -0700 From: Andi Kleen To: Jiri Olsa Cc: Andi Kleen , acme@kernel.org, jolsa@kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 2/6] perf, tools, stat: Abstract stat metrics printing Message-ID: <20150908160317.GF4524@tassilo.jf.intel.com> References: <1441406717-1717-1-git-send-email-andi@firstfloor.org> <1441406717-1717-3-git-send-email-andi@firstfloor.org> <20150907121441.GE5806@krava.brq.redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150907121441.GE5806@krava.brq.redhat.com> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org > - introduce the print_metrics interface for perf_stat__print_shadow_stats > - introduce printout function > - introduce support for printing metrics in csv in interval mode That's already a separate patch. > - and enable i > > > or any other way you find feasible *easy to review*.. I cannot ack it at this state Well it's just printing out values. It's really not rocket science. Nobody is asking you to review a new VM subsystems or something like that. I expect each more "refactoring" to add more problems. It's just really hard to to do, and usually adds more problems. I can try one more time, but if it doesn't work out we'll have to give up on this. -Andi