Messages in this thread |  | | Date | Wed, 8 Oct 2014 20:59:56 +0100 | From | Matt Fleming <> | Subject | Re: [PATCH 11/11] perf/x86/intel: Perform rotation on Intel CQM RMIDs |
| |
On Wed, 08 Oct, at 09:02:16PM, Thomas Gleixner wrote: > On Wed, 8 Oct 2014, Peter Zijlstra wrote: > > On Wed, Sep 24, 2014 at 03:04:15PM +0100, Matt Fleming wrote: > > > + preempt_disable(); > > < SNIP loooong code > > > > > + preempt_enable(); > > > > Why is all that under preempt_disable()? > > To make life harder for people who care about latencies and > deterministic behaviour perhaps?
Yeah that looks... wrong.
I'll fix that.
-- Matt Fleming, Intel Open Source Technology Center
|  |