On Mon, Jun 18, 2007 at 10:59:21AM -0700, Christoph Lameter wrote:
>       for_each_domain(this_cpu, sd) {
> -             if (sd->flags & SD_BALANCE_NEWIDLE) {
> +             if (sd->flags & SD_BALANCE_NEWIDLE)
>                       /* If we've pulled tasks over stop searching: */
>                       pulled_task = load_balance_newidle(this_cpu,
>                                                       this_rq, sd);
> -                     if (time_after(next_balance,
> -                               sd->last_balance + sd->balance_interval))
> -                             next_balance = sd->last_balance
> -                                             + sd->balance_interval;
> -                     if (pulled_task)
> -                             break;
> -             }
> +             if (time_after(next_balance,
> +                       sd->last_balance + sd->balance_interval))
> +                     next_balance = sd->last_balance
> +                                     + sd->balance_interval;

don't we have to do, msecs_to_jiffies(sd->balance_interval)?

thanks,
suresh
> +             if (pulled_task)
> +                     break;
>       }
>       if (!pulled_task)
>               /*
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to