Re: [PATCH] CFQ: Don't store left slice when slice used up or fora idle workload

From: Gui Jianfeng
Date: Tue Jul 13 2010 - 20:42:19 EST


Jeff Moyer wrote:
> Gui Jianfeng <guijianfeng@xxxxxxxxxxxxxx> writes:
>
>> It doesn't make sence to store left time slice for an idle workload
>> or for the cfqq that uses up its slice.
>
> Did you actually observe any problems? As I understand it, if you
> overrun your slice you get a negative offset, so I think we want to keep
> that.

Hi Jeff

If that's the case, do we also need to store the negative offset when slice
used up in cfq_select_queue() and cfq_idle_slice_timer()?

Thanks
Gui

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