Use hr_timer's active checks instead of having own flag.wip-edzl-critique

1 files changed, 2 insertions, 8 deletions

diff --git a/litmus/sched_edzl.c b/litmus/sched_edzl.c
index c0c8f848f00b..e9357c34561b 100644
--- a/litmus/sched_edzl.c
+++ b/litmus/sched_edzl.c
@@ -247,7 +247,6 @@ static enum hrtimer_restart on_zero_laxity(struct hrtimer *timer)
                                         get_deadline(t) - budget_remaining(t),
                                         get_deadline(t) - now);
 
-        tsk_rt(t)->zl_timer_armed = 0;
         set_zerolaxity(t);
         update_queue_position(t);
 
@@ -265,14 +264,13 @@ static inline struct task_struct* __edzl_take_ready(rt_domain_t* rt)
         {
                 if(get_zerolaxity(t) == 0)
                 {
-                        if(tsk_rt(t)->zl_timer_armed)
+            if(hrtimer_active(&tsk_rt(t)->zl_timer))
                         {
                                 int cancel_ret;
 
                                 TRACE_TASK(t, "Canceling zero-laxity timer.\n");
                                 cancel_ret = hrtimer_try_to_cancel(&tsk_rt(t)->zl_timer);
                                 WARN_ON(cancel_ret == 0); /* should never be inactive. */
-                                tsk_rt(t)->zl_timer_armed = 0;
                         }
                 }
                 else
@@ -305,8 +303,6 @@ static inline void __edzl_add_ready(rt_domain_t* rt, struct task_struct *new)
                                                 0,
                                                 HRTIMER_MODE_ABS_PINNED,
                                                 0);
-
-                tsk_rt(new)->zl_timer_armed = 1;
         }
         else
         {
@@ -662,7 +658,6 @@ static void edzl_task_new(struct task_struct * t, int on_rq, int running)
 
         raw_spin_lock_irqsave(&edzl_lock, flags);
 
-    t->rt_param.zl_timer_armed = 0;
         hrtimer_init(&t->rt_param.zl_timer, CLOCK_MONOTONIC, HRTIMER_MODE_ABS);
         t->rt_param.zl_timer.function = on_zero_laxity;
 
@@ -754,12 +749,11 @@ static void edzl_task_exit(struct task_struct * t)
                 tsk_rt(t)->scheduled_on = NO_CPU;
         }
 
-        if(tsk_rt(t)->zl_timer_armed)
+    if(hrtimer_active(&tsk_rt(t)->zl_timer))
         {
                 /* BUG if reached? */
                 TRACE_TASK(t, "Canceled armed timer while exiting.\n");
                 hrtimer_cancel(&tsk_rt(t)->zl_timer);
-                tsk_rt(t)->zl_timer_armed = 0;
         }
 
         raw_spin_unlock_irqrestore(&edzl_lock, flags);