Re: [regression] cross core scheduling frequency drop bisected to 0c313cb20732

[Rik van Riel]

On Mon, 2016-04-11 at 15:21 +0200, Rafael J. Wysocki wrote:
> On Mon, Apr 11, 2016 at 2:38 PM, Rik van Riel <riel@xxxxxxxxxx>
> wrote:
> > 
> > On Mon, 2016-04-11 at 05:04 +0200, Mike Galbraith wrote:
> > > 
> > > On Sun, 2016-04-10 at 16:24 -0400, Rik van Riel wrote:
> > > > 
> > > > 
> > > > On Sun, 2016-04-10 at 17:39 +0200, Mike Galbraith wrote:
> > > > 
> > > > > 
> > > > > 
> > > > > Should the default idle state not then be governor
> > > > > dependent?ÂÂWhen I
> > > > > set gov=performance, I'm expecting box to go just as fast as
> > > > > it
> > > > > can
> > > > > go
> > > > > without melting.ÂÂDoes polling risk CPU -> lava conversion?
> > > > Current CPUs can only have some cores run at full speed
> > > > (turbo mode) if other cores are idling and/or running at
> > > > lower speeds.
> > > The real world is very unlikely to miss the prettier numbers I'm
> > > grieving over one tiny bit.ÂÂKnowing that doesn't make giving
> > > them up
> > > any easier though.. byebye cycles (sniff) ;-)
> > I suspect your pipe benchmark could be very relevant to
> > network performance numbers, too.
> > 
> > I would like to go into polling a little bit more aggressively
> > in a future kernel,
> Agreed, but ->
> 
> > 
> > and I think we can get away with it if we
> > teach the polling loop to exit after we have spent enough time
> > there that the menu governor will pick HLT after a few timed
> > out poll loops.
> -> my concern about this approach is that it would add an artificial
> point to the menu governor statistics at whatever the timeout is
> chosen to be.

I would set the threshold to at least the HLT target residency +
exit latency, so 3 poll timeouts in 8 wakeups would cause
us to fall back to HLT.

On the other hand, if the system is legitimately very busy,
and we break out of the HLT loop due to activities happening
before the timeout most of the time, we should automatically
pick polling.

Does that make sense?

Am I overlooking something?

-- 
All Rights Reversed.

Attachment: signature.asc
Description: This is a digitally signed message part