cleanup: tidy up code before release2007.2

This gets rid of quite a few FIXMEs and TODOs, as well as some cruft.

1 files changed, 0 insertions, 12 deletions

diff --git a/kernel/sched_global_edf.c b/kernel/sched_global_edf.c
index 6e3928b7df..4b36bc5a48 100644
--- a/kernel/sched_global_edf.c
+++ b/kernel/sched_global_edf.c
@@ -64,7 +64,6 @@ static void adjust_cpu_queue(int exec_rt, jiffie_t deadline)
         entry->executes_realtime = exec_rt;
         entry->cur_deadline      = deadline;
 
-        /* TODO: move instead of del+reinsert */
         list_del(&entry->list);
         /* if we do not execute real-time jobs we just move 
          * to the end of the queue 
@@ -94,10 +93,6 @@ static void adjust_cpu_queue(int exec_rt, jiffie_t deadline)
  * kick the next if necessary, and so on. The caller is responsible for making
  * sure that it is not the last entry or that a reschedule is not necessary.
  *
- * TODO: This function is probably way too trigger happy. It should only send
- *       IPIs if the other CPU is not  going to reschedule anyway. But that is 
- *       hard to detect reliably. Too many schedules will hurt performance 
- *       but do not cause incorrect schedules.
  */
 static int gedf_check_resched(rt_domain_t *edf) 
 {
@@ -336,9 +331,6 @@ static void gedf_wake_up_task(struct task_struct *task)
         }
         else if (task->time_slice) {
                 /* came back in time before deadline
-                 * TODO: clip budget to fit into period, otherwise it could 
-                 *       cause a deadline overrun in the next period, i.e. 
-                 *       over allocation in the next period.
                  */
                 set_rt_flags(task, RT_F_RUNNING);
                 add_ready(&gedf, task);
@@ -356,8 +348,6 @@ static void gedf_task_blocks(struct task_struct *t)
          * it is running, and when it is not running it is not in any 
          * queue anyway.
          *
-         * TODO: Check whether the assumption is correct for SIGKILL and 
-         *       SIGSTOP.
          */
         TRACE("task %d blocks with budget=%d\n", t->pid, t->time_slice);
         BUG_ON(t->rt_list.next != LIST_POISON1);
@@ -433,7 +423,6 @@ static sched_plugin_t s_plugin __cacheline_aligned_in_smp = {
         .prepare_task           = gedf_prepare_task,\
         .sleep_next_period      = edf_sleep_next_period,\
         .tear_down              = gedf_tear_down,\
-        .shutdown_hook          = 0,\
         .schedule               = gedf_schedule,\
         .finish_switch          = gedf_finish_switch,\
         .mode_change            = gedf_mode_change,\
@@ -538,7 +527,6 @@ static int gedf_np_check_resched(rt_domain_t *edf)
         .prepare_task           = gedf_prepare_task,\
         .sleep_next_period      = edf_sleep_next_period,\
         .tear_down              = gedf_tear_down,\
-        .shutdown_hook          = 0,\
         .schedule               = gedf_schedule,\
         .finish_switch          = gedf_finish_switch,\
         .mode_change            = gedf_mode_change,\