From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934890AbdAKVzH (ORCPT ); Wed, 11 Jan 2017 16:55:07 -0500 Received: from one.firstfloor.org ([193.170.194.197]:42787 "EHLO one.firstfloor.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1760086AbdAKVzG (ORCPT ); Wed, 11 Jan 2017 16:55:06 -0500 Date: Wed, 11 Jan 2017 13:54:07 -0800 From: Andi Kleen To: Jiri Olsa Cc: Andi Kleen , acme@kernel.org, jolsa@kernel.org, mingo@kernel.org, linux-kernel@vger.kernel.org Subject: Re: Support Intel uncore event lists v4 Message-ID: <20170111215406.GF26852@two.firstfloor.org> References: <20170110013337.4876-1-andi@firstfloor.org> <20170111185344.GA21289@krava> <20170111195258.GE26852@two.firstfloor.org> <20170111214732.GA24030@krava> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170111214732.GA24030@krava> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Jan 11, 2017 at 10:47:32PM +0100, Jiri Olsa wrote: > On Wed, Jan 11, 2017 at 11:52:58AM -0800, Andi Kleen wrote: > > > > v1: Initial post > > > > v2: Address review feedback. See changelog in commits. > > > > v3: Repost. Rebase to latest tree. > > > > v4: Rebase. Change DividedBy to generic simple expression parser. > > > > Fix refactoring problem that broke git bisect. > > > > Address review feedback. > > > > > > > > > > what's the difference to the post from Jan 03? > > > > Jan 03 was v3 I think? > > > > Also I addressed all your review comments (listed in the individual > > patches) > > 6012 Jan 03 Andi Kleen (1.5K) Support Intel uncore event lists v4 > 6421 r Jan 09 Andi Kleen (1.5K) Support Intel uncore event lists v4 > > the messages seem identical Hmm I messed up the version number then. This should be version number 5 then. It was supposed to fix your review feedback for the earlier series and add Acked-by. I listed the (minor) changes in the individual patches. perf, tools: Add support for parsing uncore json files v3: Remove extra __maybe_unused perf, tools: Factor out scale conversion code v3: Use ENOMEM, not -1. Really only trivial changes at this point, so you only need to read the patches you didn't read and ack before. Thanks, -Andi