[PATCH] sched: Fix pointless comparison between unsigned int with zero

From: Liao Chang
Date: Wed Apr 06 2022 - 09:04:28 EST


It is pointless to do a comparison bewteen unsigned variable and zero,
since validation routine requires the MSB of sysctl_sched_rt_period_ms
is 0, so casting variable to signed value before comparing with zero.

Signed-off-by: Liao Chang <liaochang1@xxxxxxxxxx>
---
kernel/sched/rt.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/sched/rt.c b/kernel/sched/rt.c
index 7b4f4fbbb404..58e105180e67 100644
--- a/kernel/sched/rt.c
+++ b/kernel/sched/rt.c
@@ -2906,7 +2906,7 @@ static int sched_rt_global_constraints(void)

static int sched_rt_global_validate(void)
{
- if (sysctl_sched_rt_period <= 0)
+ if ((int)sysctl_sched_rt_period <= 0)
return -EINVAL;

if ((sysctl_sched_rt_runtime != RUNTIME_INF) &&
--
2.17.1