[PATCH] sched: deadline: Remove unnecessary ‘NULL’ values from dl_task_offline_migration

Li zeming posted 1 patch 2 years, 8 months ago
kernel/sched/deadline.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] sched: deadline: Remove unnecessary ‘NULL’ values from dl_task_offline_migration
Posted by Li zeming 2 years, 8 months ago
later_rq is assigned first, so it does not need to initialize the
assignment.

Signed-off-by: Li zeming <zeming@nfschina.com>
---
 kernel/sched/deadline.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
index 5a9a4b81c972..d823e608c7f0 100644
--- a/kernel/sched/deadline.c
+++ b/kernel/sched/deadline.c
@@ -667,7 +667,7 @@ static struct rq *find_lock_later_rq(struct task_struct *task, struct rq *rq);
 
 static struct rq *dl_task_offline_migration(struct rq *rq, struct task_struct *p)
 {
-	struct rq *later_rq = NULL;
+	struct rq *later_rq;
 	struct dl_bw *dl_b;
 
 	later_rq = find_lock_later_rq(p, rq);
-- 
2.18.2
Re: [PATCH] sched: deadline: Remove unnecessary ‘NULL’ values from dl_task_offline_migration
Posted by Mukesh Ojha 2 years, 8 months ago

On 5/24/2023 5:11 AM, Li zeming wrote:
> later_rq is assigned first, so it does not need to initialize the
> assignment.
> 
> Signed-off-by: Li zeming <zeming@nfschina.com>
> ---
>   kernel/sched/deadline.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
> index 5a9a4b81c972..d823e608c7f0 100644
> --- a/kernel/sched/deadline.c
> +++ b/kernel/sched/deadline.c
> @@ -667,7 +667,7 @@ static struct rq *find_lock_later_rq(struct task_struct *task, struct rq *rq);
>   
>   static struct rq *dl_task_offline_migration(struct rq *rq, struct task_struct *p)
>   {
> -	struct rq *later_rq = NULL;
> +	struct rq *later_rq;
>   	struct dl_bw *dl_b;
>   
>   	later_rq = find_lock_later_rq(p, rq);

Acked-by: Mukesh Ojha <quic_mojha@quicinc.com>

-- Mukesh