lkml.org 
[lkml]   [2017]   [Jul]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    SubjectRe: [RFC PATCH v1 00/11] Create fast idle path for short idle periods
    From
    Date
    On 2017/7/18 15:19, Thomas Gleixner wrote:
    > On Mon, 17 Jul 2017, Andi Kleen wrote:
    >> On Tue, Jul 18, 2017 at 08:43:53AM +0200, Thomas Gleixner wrote:
    >>> On Mon, 17 Jul 2017, Andi Kleen wrote:
    >>>
    >>>>> We need a tradeoff here IMHO. I'll check Daniel's work to understand how/if
    >>>>> it's better than menu governor.
    >>>>
    >>>> I still would like to see how the fast path without the C1 heuristic works.
    >>>>
    >>>> Fast pathing is a different concept from a better predictor. IMHO we need
    >>>> both, but the first is likely lower hanging fruit.
    >>>
    >>> Hacking something on the side is always the lower hanging fruit as it
    >>> avoids solving the hard problems. As Peter said already, that's not going
    >>> to happen unless there is a real technical reason why the general path
    >>> cannot be fixed. So far there is no proof for that.
    >>
    >> You didn't look at Aubrey's data?
    >>
    >> There are some unavoidable slow operations in the current path -- e.g.
    >> reprograming the timer for NOHZ. But we don't need that for really
    >> short idle periods, because as you pointed out they never get woken
    >> up by the tick.
    >>
    >> Similar for other things like RCU.
    >>
    >> I don't see how you can avoid that other than without a fast path mechanism.
    >
    > You can very well avoid it by taking the irq timings or whatever other
    > information into account for the NOHZ decision.
    >
    If I read the source correctly, irq timing statistics computation happens at
    idle time. Sadly, this is what Andi Kleen worried about. People keep putting
    more and more slow stuff in idle path, not realizing it could be a critical path.

    /*
    * The computation happens at idle time. When the CPU is not idle, the
    * interrupts' timestamps are stored in the circular buffer, when the
    * CPU goes idle and this routine is called, all the buffer's values
    * are injected in the statistical model continuing to extend the
    * statistics from the previous busy-idle cycle.
    */

    Thanks,
    -Aubrey

    \
     
     \ /
      Last update: 2017-07-19 08:13    [W:4.053 / U:0.088 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site