MIME-Version: 1.0
In-Reply-To: <20160308210707.GG6344@twins.programming.kicks-ass.net>
References: <1457034642-21837-1-git-send-email-eranian@google.com>
	<1457034642-21837-3-git-send-email-eranian@google.com>
	<20160303214312.GI23621@tassilo.jf.intel.com>
	<CABPqkBSOAht0SqOs82r-O1cUf2KzzSbP21hHhZcv6D-3YawOzA@mail.gmail.com>
	<20160307102413.GB6356@twins.programming.kicks-ass.net>
	<20160307121840.GF6375@twins.programming.kicks-ass.net>
	<20160307182731.GA12153@krava.redhat.com>
	<20160307202556.GQ6344@twins.programming.kicks-ass.net>
	<CABPqkBT7ogN2Og20WtpRhKo29Twf0Lt_eZpPFie=dzPps-HWTQ@mail.gmail.com>
	<20160308210707.GG6344@twins.programming.kicks-ass.net>
Date: Tue, 8 Mar 2016 13:13:24 -0800
Message-ID: <CABPqkBQ3T1hx6Ncmhf_GN3ivw7JyOguXbbrCBpGkN7=qsXbf=Q@mail.gmail.com>
Subject: Re: [PATCH 2/3] perf/x86/pebs: add workaround for broken OVFL status
 on HSW
From: Stephane Eranian <eranian@google.com>
To: Peter Zijlstra <peterz@infradead.org>
Cc: Jiri Olsa <jolsa@redhat.com>, Andi Kleen <ak@linux.intel.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Arnaldo Carvalho de Melo <acme@redhat.com>,
        "mingo@elte.hu" <mingo@elte.hu>, "Liang, Kan" <kan.liang@intel.com>,
        Namhyung Kim <namhyung@kernel.org>,
        Adrian Hunter <adrian.hunter@intel.com>
Content-Type: text/plain; charset=UTF-8
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1362
Lines: 38

Hi,

On Tue, Mar 8, 2016 at 1:07 PM, Peter Zijlstra <peterz@infradead.org> wrote:
> On Tue, Mar 08, 2016 at 12:59:23PM -0800, Stephane Eranian wrote:
>> hi,
>>
>> On Mon, Mar 7, 2016 at 12:25 PM, Peter Zijlstra <peterz@infradead.org> wrote:
>> >
>> > On Mon, Mar 07, 2016 at 07:27:31PM +0100, Jiri Olsa wrote:
>> > > On Mon, Mar 07, 2016 at 01:18:40PM +0100, Peter Zijlstra wrote:
>> > > > On Mon, Mar 07, 2016 at 11:24:13AM +0100, Peter Zijlstra wrote:
>> > > >
>> > > > > I suspect Andi is having something along:
>> > > > >
>> > > > >  lkml.kernel.org/r/1445458568-16956-1-git-send-email-andi@firstfloor.org
>> > > > >
>> > > > > applied to his tree.
>> > > >
>> > > > OK, I munged a bunch of patches together, please have a hard look at the
>> > > > end result found in:
>> > > >
>> > > >   git://git.kernel.org/pub/scm/linux/kernel/git/peterz/queue.git perf/core
>> > > >
>>
>> I ran this kernel on Haswell. Even with Andi's fixes the problem I identified is
>> still there, so my patch is still needed.
>
> Right, your patch should be included in that kernel, or did I make a
> royal mess of things?
>
No, it is as expected for the OVF PMI fix.

> I put Andi's late status ack on top of your patch.
>
> Also note, Ingo merged most of those patches today, all except the top
> 3, because Andi wanted to double check something.

Ok. I will try on SKL today.