Date: Wed, 31 Oct 2012 09:43:23 -0700
From: Andi Kleen <ak@linux.intel.com>
To: Gleb Natapov <gleb@redhat.com>
Cc: Andi Kleen <andi@firstfloor.org>, linux-kernel@vger.kernel.org,
        acme@redhat.com, peterz@infradead.org, jolsa@redhat.com,
        eranian@google.com, mingo@kernel.org, namhyung@kernel.org,
        avi@redhat.com
Subject: Re: [PATCH 05/32] perf, kvm: Support the intx/intx_cp modifiers in
 KVM arch perfmon emulation v3
Message-ID: <20121031164323.GB28297@tassilo.jf.intel.com>
References: <1351643663-23828-1-git-send-email-andi@firstfloor.org>
 <1351643663-23828-6-git-send-email-andi@firstfloor.org>
 <20121031102701.GB25650@redhat.com>
 <20121031103248.GX16230@one.firstfloor.org>
 <20121031103816.GD25650@redhat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20121031103816.GD25650@redhat.com>
User-Agent: Mutt/1.5.21 (2010-09-15)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1157
Lines: 29

On Wed, Oct 31, 2012 at 12:38:16PM +0200, Gleb Natapov wrote:
> On Wed, Oct 31, 2012 at 11:32:48AM +0100, Andi Kleen wrote:
> > On Wed, Oct 31, 2012 at 12:27:01PM +0200, Gleb Natapov wrote:
> > > On Tue, Oct 30, 2012 at 05:33:56PM -0700, Andi Kleen wrote:
> > > > From: Andi Kleen <ak@linux.intel.com>
> > > > 
> > > > This is not arch perfmon, but older CPUs will just ignore it. This makes
> > > > it possible to do at least some TSX measurements from a KVM guest
> > > > 
> > > You are ignoring my reviews.
> > 
> > Huh? I nearly rewrote the patch based on your feedback.
> > It's much more complicated now just for the CPUID check you wanted.
> > I hope all that code is worth it.
> > 
> > If you want more changes please let me know.
> > 
> I did comment on v3 yesterday. May be you missed it. https://lkml.org/lkml/2012/10/30/87

I did. Ok fixed. 
-Andi

-- 
ak@linux.intel.com -- Speaking for myself only
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/