Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=7c92e54f6f9601cfa9d8894ee248abcf62ed9a1c
Commit:     7c92e54f6f9601cfa9d8894ee248abcf62ed9a1c
Parent:     cf2ab4696ee42f895eed88c2b6e432fe03dda0db
Author:     Peter Zijlstra <[EMAIL PROTECTED]>
AuthorDate: Wed Sep 5 14:32:49 2007 +0200
Committer:  Ingo Molnar <[EMAIL PROTECTED]>
CommitDate: Wed Sep 5 14:32:49 2007 +0200

    sched: simplify __check_preempt_curr_fair()
    
    Preparatory patch for fix-ideal-runtime:
    
    simplify __check_preempt_curr_fair(): get rid of the integer return.
    
       text    data     bss     dec     hex filename
      13404     228    1204   14836    39f4 sched.o.before
      13393     228    1204   14825    39e9 sched.o.after
    
    functionality is unchanged.
    
    Signed-off-by: Peter Zijlstra <[EMAIL PROTECTED]>
    Signed-off-by: Ingo Molnar <[EMAIL PROTECTED]>
---
 kernel/sched_fair.c |    8 +++-----
 1 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/kernel/sched_fair.c b/kernel/sched_fair.c
index bac2aff..f0dd4be 100644
--- a/kernel/sched_fair.c
+++ b/kernel/sched_fair.c
@@ -673,7 +673,7 @@ dequeue_entity(struct cfs_rq *cfs_rq, struct sched_entity 
*se, int sleep)
 /*
  * Preempt the current task with a newly woken task if needed:
  */
-static int
+static void
 __check_preempt_curr_fair(struct cfs_rq *cfs_rq, struct sched_entity *se,
                          struct sched_entity *curr, unsigned long granularity)
 {
@@ -686,9 +686,8 @@ __check_preempt_curr_fair(struct cfs_rq *cfs_rq, struct 
sched_entity *se,
         */
        if (__delta > niced_granularity(curr, granularity)) {
                resched_task(rq_of(cfs_rq)->curr);
-               return 1;
+               curr->prev_sum_exec_runtime = curr->sum_exec_runtime;
        }
-       return 0;
 }
 
 static inline void
@@ -764,8 +763,7 @@ static void entity_tick(struct cfs_rq *cfs_rq, struct 
sched_entity *curr)
        if (delta_exec > ideal_runtime)
                gran = 0;
 
-       if (__check_preempt_curr_fair(cfs_rq, next, curr, gran))
-               curr->prev_sum_exec_runtime = curr->sum_exec_runtime;
+       __check_preempt_curr_fair(cfs_rq, next, curr, gran);
 }
 
 /**************************************************
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to