Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-08 Thread Vince Weaver
On Thu, 8 Sep 2016, Alexander Shishkin wrote: > Vince Weaver writes: > > > On the skylake machine with the original 5 patches I got this after > > continuing to fuzz. Sorry about the lack of frame pointer, next x> > compile will have it enabled. > > > > If it matters, prior to this I hit the u

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-08 Thread Alexander Shishkin
Vince Weaver writes: > On the skylake machine with the original 5 patches I got this after > continuing to fuzz. Sorry about the lack of frame pointer, next > compile will have it enabled. > > If it matters, prior to this I hit the unrelated > [25510.278199] WARNING: CPU: 1 PID: 25405 at kernel

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-08 Thread Alexander Shishkin
Ingo Molnar writes: > * Alexander Shishkin wrote: > >> Ingo Molnar writes: >> >> > * Alexander Shishkin wrote: >> > >> >> Hi, >> >> >> >> There were more bugs since the previous version, plus the BTS barriers >> >> got >> >> fixed. With these patches, my testcase keeps running and no spuri

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Ingo Molnar
* Ingo Molnar wrote: > Yeah, please re-send it - and please also Vince's Reported-by tag to all > commits ^--- add > that would explain failures that Vince reported. Thanks, Ingo

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Ingo Molnar
* Alexander Shishkin wrote: > Ingo Molnar writes: > > > * Alexander Shishkin wrote: > > > >> Hi, > >> > >> There were more bugs since the previous version, plus the BTS barriers got > >> fixed. With these patches, my testcase keeps running and no spurious NMI > >> warnings pop up any more.

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Vince Weaver
On Wed, 7 Sep 2016, Alexander Shishkin wrote: > Vince Weaver writes: > > > On Wed, 7 Sep 2016, Alexander Shishkin wrote: > > > >> Sure. And yes, I did catch a warning, which calls for one more patch > >> (below). Also one unrelated thing in PEBS that Peter fixed. > > > > Does that fix this which

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Alexander Shishkin
Vince Weaver writes: > On Wed, 7 Sep 2016, Alexander Shishkin wrote: > >> Sure. And yes, I did catch a warning, which calls for one more patch >> (below). Also one unrelated thing in PEBS that Peter fixed. > > Does that fix this which I just got on my skylake machine (4.8-rc5 with > your other 5

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Peter Zijlstra
On Wed, Sep 07, 2016 at 11:36:48AM -0400, Vince Weaver wrote: > On Wed, 7 Sep 2016, Alexander Shishkin wrote: > > > Also one unrelated thing in PEBS that Peter fixed. --- Subject: perf,x86: Fix PEBSv3 record drain From: Peter Zijlstra Date: Wed Sep 7 14:42:55 CEST 2016 Alexander hit the WARN_O

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Vince Weaver
On Wed, 7 Sep 2016, Alexander Shishkin wrote: > Sure. And yes, I did catch a warning, which calls for one more patch > (below). Also one unrelated thing in PEBS that Peter fixed. Does that fix this which I just got on my skylake machine (4.8-rc5 with your other 5 patches applied) [ 5351.822559]

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-07 Thread Alexander Shishkin
Ingo Molnar writes: > * Alexander Shishkin wrote: > >> Hi, >> >> There were more bugs since the previous version, plus the BTS barriers got >> fixed. With these patches, my testcase keeps running and no spurious NMI >> warnings pop up any more. > > Could you please also run the fuzzer that Vi

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-06 Thread Vince Weaver
On Tue, 6 Sep 2016, Ingo Molnar wrote: > > There were more bugs since the previous version, plus the BTS barriers got > > fixed. With these patches, my testcase keeps running and no spurious NMI > > warnings pop up any more. > > Could you please also run the fuzzer that Vince uses, does it now

Re: [PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-06 Thread Ingo Molnar
* Alexander Shishkin wrote: > Hi, > > There were more bugs since the previous version, plus the BTS barriers got > fixed. With these patches, my testcase keeps running and no spurious NMI > warnings pop up any more. Could you please also run the fuzzer that Vince uses, does it now pass on h

[PATCH v2 0/5] perf, bts: Fallout from the fuzzer for perf/urgent

2016-09-06 Thread Alexander Shishkin
Hi, There were more bugs since the previous version, plus the BTS barriers got fixed. With these patches, my testcase keeps running and no spurious NMI warnings pop up any more. Original story: Recently Vince has reported warnings and panics coming from the general direction of AUX tracing. I fo