lkml.org 
[lkml]   [2006]   [May]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    SubjectRE: Regression seen for patch "sched:dont decrease idle sleep avg"
    Date
    Con Kolivas wrote on Sunday, May 14, 2006 9:03 AM
    > There would be no difference if the priority boost is done lower. The if and
    > else blocks both end up equating to the same amount of priority boost, with
    > the former having a ceiling on it, so yes it is the intent. You'll see that
    > the amount of sleep required to jump from lowest priority to MAX_SLEEP_AVG -
    > DEF_TIMESLICE is INTERACTIVE_SLEEP.

    I don't think the if and the else block is doing the same thing. In the if
    block, the p->sleep_avg is unconditionally boosted to ceiling for all tasks,
    though it will not reduce sleep_avg for tasks that already exceed the ceiling.
    Bumping up sleep_avg will then translate into priority boost of MAX_BONUS-1,
    which potentially can be too high.

    But that's fine if it is the intent. At minimum, the comment in the source
    code should say so instead of fooling people who don't actually read the code.


    [patch] sched: update comments in priority calculation w.r.t. implementation.

    Signed-off-by: Ken Chen <kenneth.w.chen@intel.com>

    --- ./kernel/sched.c.orig 2006-05-15 12:24:02.000000000 -0700
    +++ ./kernel/sched.c 2006-05-15 12:37:16.000000000 -0700
    @@ -746,10 +746,12 @@ static int recalc_task_prio(task_t *p, u
    if (likely(sleep_time > 0)) {
    /*
    * User tasks that sleep a long time are categorised as
    - * idle. They will only have their sleep_avg increased to a
    - * level that makes them just interactive priority to stay
    - * active yet prevent them suddenly becoming cpu hogs and
    - * starving other processes.
    + * idle. If they sleep longer than INTERACTIVE_SLEEP, it
    + * will have its priority boosted to minimum MAX_BONUS-1.
    + * For short sleep, they will only have their sleep_avg
    + * increased to a level that makes them just interactive
    + * priority to stay active yet prevent them suddenly becoming
    + * cpu hogs and starving other processes.
    */
    if (p->mm && sleep_time > INTERACTIVE_SLEEP(p)) {
    unsigned long ceiling;

    -
    To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
    the body of a message to majordomo@vger.kernel.org
    More majordomo info at http://vger.kernel.org/majordomo-info.html
    Please read the FAQ at http://www.tux.org/lkml/

    \
     
     \ /
      Last update: 2006-05-15 21:06    [W:3.908 / U:0.016 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site