Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1470985 > unrolled thread

Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU callbacks

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU  callbacks Vince Weaver <vince@deater.net> - 2016-08-26 22:50 +0200
    Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU callbacks Alexander Shishkin <alexander.shishkin@linux.intel.com> - 2016-08-27 08:50 +0200

#1470985 — Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU callbacks

On Wed, 24 Aug 2016, Alexander Shishkin wrote:

> Alexander Shishkin <alexander.shishkin@linux.intel.com> writes:
> 
> > Alexander Shishkin <alexander.shishkin@linux.intel.com> writes:
> >
> >> Signed-off-by: Alexander Shishkin <alexander.shishkin@linux.intel.com>
> >
> > Ok, this one is broken, please disregard.
> 
> Vince, can you try the following (with the other two in this series)?
> 

Sorry for the delay.  Not sure if testing was still needed (or if it was 
superceded by Will Deacon's patch) but I ran the fuzzer for a few hours 
with this patch and the other two applied and didn't see any of the NMI 
errors that I saw with the original patch.

Vince

[toc] | [next] | [standalone]

#1471110 — Re: [PATCH 1/3] perf/x86/intel/bts: Fix confused ordering of PMU callbacks

Yes, Will's patch notwithstanding, these patches fix the
warnings/oopses that you observed.

On 26 August 2016 at 23:49, Vince Weaver <vince@deater.net> wrote:
> On Wed, 24 Aug 2016, Alexander Shishkin wrote:
>
>> Alexander Shishkin <alexander.shishkin@linux.intel.com> writes:
>>
>> > Alexander Shishkin <alexander.shishkin@linux.intel.com> writes:
>> >
>> >> Signed-off-by: Alexander Shishkin <alexander.shishkin@linux.intel.com>
>> >
>> > Ok, this one is broken, please disregard.
>>
>> Vince, can you try the following (with the other two in this series)?
>>
>
> Sorry for the delay.  Not sure if testing was still needed (or if it was
> superceded by Will Deacon's patch) but I ran the fuzzer for a few hours
> with this patch and the other two applied and didn't see any of the NMI
> errors that I saw with the original patch.
>
> Vince

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web