diff mbox series

sched/deadline: remove unused dl_bandwidth

Message ID 20230524102514.407486-1-linmiaohe@huawei.com
State New
Headers
Series sched/deadline: remove unused dl_bandwidth |

Commit Message

Miaohe Lin May 24, 2023, 10:25 a.m. UTC 
  The default deadline bandwidth control structure has been removed since
commit eb77cf1c151c ("sched/deadline: Remove unused def_dl_bandwidth")
leading to unused init_dl_bandwidth() and struct dl_bandwidth. Remove
them to clean up the code.

Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>
---
 kernel/sched/deadline.c | 7 -------
 kernel/sched/sched.h    | 7 -------
 2 files changed, 14 deletions(-)

Comments

Juri Lelli May 24, 2023, 6:47 a.m. UTC | #1 
On 24/05/23 18:25, Miaohe Lin wrote:
> The default deadline bandwidth control structure has been removed since
> commit eb77cf1c151c ("sched/deadline: Remove unused def_dl_bandwidth")
> leading to unused init_dl_bandwidth() and struct dl_bandwidth. Remove
> them to clean up the code.
> 
> Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>

Ah, indeed.

Acked-by: Juri Lelli <juri.lelli@redhat.com>

Thanks!
Juri
Daniel Bristot de Oliveira May 25, 2023, 9:53 a.m. UTC | #2 
On 5/24/23 12:25, Miaohe Lin wrote:
> The default deadline bandwidth control structure has been removed since
> commit eb77cf1c151c ("sched/deadline: Remove unused def_dl_bandwidth")
> leading to unused init_dl_bandwidth() and struct dl_bandwidth. Remove
> them to clean up the code.

I wondered if it desire a Fixes tags... Anyways, it is just dead code.

> Signed-off-by: Miaohe Lin <linmiaohe@huawei.com>

Reviewed-by: Daniel Bristot de Oliveira <bristot@kernel.org>

Thanks!
-- Daniel
diff mbox series

Patch

diff --git a/kernel/sched/deadline.c b/kernel/sched/deadline.c
index fe983ed7bb12..7c98ca1f7300 100644
--- a/kernel/sched/deadline.c
+++ b/kernel/sched/deadline.c
@@ -482,13 +482,6 @@  static inline int is_leftmost(struct task_struct *p, struct dl_rq *dl_rq)
 
 static void init_dl_rq_bw_ratio(struct dl_rq *dl_rq);
 
-void init_dl_bandwidth(struct dl_bandwidth *dl_b, u64 period, u64 runtime)
-{
-	raw_spin_lock_init(&dl_b->dl_runtime_lock);
-	dl_b->dl_period = period;
-	dl_b->dl_runtime = runtime;
-}
-
 void init_dl_bw(struct dl_bw *dl_b)
 {
 	raw_spin_lock_init(&dl_b->lock);
diff --git a/kernel/sched/sched.h b/kernel/sched/sched.h
index 1704763897d0..d79f4fd53600 100644
--- a/kernel/sched/sched.h
+++ b/kernel/sched/sched.h
@@ -286,12 +286,6 @@  struct rt_bandwidth {
 
 void __dl_clear_params(struct task_struct *p);
 
-struct dl_bandwidth {
-	raw_spinlock_t		dl_runtime_lock;
-	u64			dl_runtime;
-	u64			dl_period;
-};
-
 static inline int dl_bandwidth_enabled(void)
 {
 	return sysctl_sched_rt_runtime >= 0;
@@ -2394,7 +2388,6 @@  extern struct rt_bandwidth def_rt_bandwidth;
 extern void init_rt_bandwidth(struct rt_bandwidth *rt_b, u64 period, u64 runtime);
 extern bool sched_rt_bandwidth_account(struct rt_rq *rt_rq);
 
-extern void init_dl_bandwidth(struct dl_bandwidth *dl_b, u64 period, u64 runtime);
 extern void init_dl_task_timer(struct sched_dl_entity *dl_se);
 extern void init_dl_inactive_task_timer(struct sched_dl_entity *dl_se);