From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753842AbcCANHL (ORCPT ); Tue, 1 Mar 2016 08:07:11 -0500 Received: from mail.kernel.org ([198.145.29.136]:53560 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753624AbcCANHJ (ORCPT ); Tue, 1 Mar 2016 08:07:09 -0500 Date: Tue, 1 Mar 2016 10:07:00 -0300 From: Arnaldo Carvalho de Melo To: Andi Kleen Cc: jolsa@kernel.org, linux-kernel@vger.kernel.org Subject: Re: perf, tools: Refactor and support interval and CSV metrics Message-ID: <20160301130700.GC3170@kernel.org> References: <1456785386-19481-1-git-send-email-andi@firstfloor.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1456785386-19481-1-git-send-email-andi@firstfloor.org> X-Url: http://acmel.wordpress.com User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Em Mon, Feb 29, 2016 at 02:36:19PM -0800, Andi Kleen escreveu: > Fixed last feedback. > > [v5: Fix mainly bisect problems. No regressions introduced by one > patch and fixed again later. Some minor fixes in addition] > [v6: Fix running/noise printing patch.] > [v7: Reorder and merge two patches to avoid a bisect hole where unsupported was > printed as 0] > [v8: Minor fixes for review feedback. See changelog in patches.] > [v9: Fix newline bug. Add support for -A for --metric-only] > [v10: Remove extra "noise" printing (Jiri) > Fix fields in documentation (Jiri)] > [v11: Fix manpage again. Avoid extra metric output in CSV mode.] > [v12: Move CSV metrics fields to after running/enabled/variance. > Minor fixes.] > [v13: Address review comments. Now probe for stalled events > in advance to avoid empty columns or lines. Fix -A shadowing. > Various minor changes. Drop merged patches.] > [v14: Fix empty lines with CSV metrics. Avoid one more empty column > in metric-only.] 1/7 was already merged, pushed to Ingo, applied (2,3)/7. - Arnaldo