Re: [PATCH 1/9] sched: test below 0 on unsignedsysctl_sched_rt_period
From: Peter Zijlstra
Date: Tue Jul 22 2008 - 05:09:20 EST
On Mon, 2008-07-21 at 20:28 -0400, roel kluin wrote:
> Peter, you added the test in global_rt_runtime(), in commit
> could you take a look at it and tell whether this test should
> be replaced by something else?
> The patch was only checkpatch tested
> sysctl_sched_rt_period is unsigned so the test doesn't work.
> state_filter is an unsigned long so the test didn't work
> Signed-off-by: Roel Kluin <roel.kluin@xxxxxxxxx>
> diff --git a/kernel/sched.c b/kernel/sched.c
> index 99e6d85..2030340 100644
> --- a/kernel/sched.c
> +++ b/kernel/sched.c
> @@ -832,9 +832,6 @@ static inline u64 global_rt_period(void)
> static inline u64 global_rt_runtime(void)
> - if (sysctl_sched_rt_period < 0)
> - return RUNTIME_INF;
Gah, I'm a moron. That should read:
if (sysctl_sched_rt_runtime < 0)
Care to send a patch to fix it?
Thanks for spotting this!
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/