Re: [accounting regression since rc1] scheduler updates

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Christian Borntraeger
Date: Wednesday, August 22, 2007 - 12:50 am

Am Dienstag, 21. August 2007 schrieben Sie:

I looked again at the scheduler code and things are getting better when I run 
the patch below on top of your patch and with our sched_clock prototype. I 
guess there is a reason why you want rq->clock advanced by at least one tick?

We discussed calling scheduler_tick with virtual time as well.
Would it have the same result?
What would be the impact on latency?

After looking at the current s390 timer code, it seems that this kind of 
change is not trivial enough to be rc3+ ready. 
I personally think, that for 2.6.23 we should use the patch against 
fs/proc/array.c and everything else for 2.6.24?

Christian

---
 kernel/sched.c |    6 ------
 1 file changed, 6 deletions(-)

Index: linux-2.6/kernel/sched.c
===================================================================
--- linux-2.6.orig/kernel/sched.c
+++ linux-2.6/kernel/sched.c
@@ -3321,15 +3321,9 @@ void scheduler_tick(void)
 	int cpu = smp_processor_id();
 	struct rq *rq = cpu_rq(cpu);
 	struct task_struct *curr = rq->curr;
-	u64 next_tick = rq->tick_timestamp + TICK_NSEC;
 
 	spin_lock(&rq->lock);
 	__update_rq_clock(rq);
-	/*
-	 * Let rq->clock advance by at least TICK_NSEC:
-	 */
-	if (unlikely(rq->clock < next_tick))
-		rq->clock = next_tick;
 	rq->tick_timestamp = rq->clock;
 	update_cpu_load(rq);
 	if (curr != rq->idle) /* FIXME: needed? */


-
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[git pull request] scheduler updates, Ingo Molnar, (Sun Aug 12, 9:32 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 14, 1:37 am)
[PATCH][RFC] Re: accounting regression since rc1, Christian Borntraeger, (Thu Aug 16, 1:17 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Mon Aug 20, 10:03 am)
Re: [accounting regression since rc1] scheduler updates, Ingo Molnar, (Mon Aug 20, 11:08 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Mon Aug 20, 11:33 am)
Re: [accounting regression since rc1] scheduler updates, Balbir Singh, (Mon Aug 20, 12:00 pm)
Re: [accounting regression since rc1] scheduler updates, Ingo Molnar, (Mon Aug 20, 12:05 pm)
Re: [accounting regression since rc1] scheduler updates, Ingo Molnar, (Mon Aug 20, 12:12 pm)
Re: [accounting regression since rc1] scheduler updates, Paul Mackerras, (Mon Aug 20, 4:07 pm)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 12:00 am)
Re: [accounting regression since rc1] scheduler updates, Ingo Molnar, (Tue Aug 21, 12:09 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 12:20 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 1:17 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Tue Aug 21, 2:11 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Tue Aug 21, 2:18 am)
Re: [accounting regression since rc1] scheduler updates, Paul Mackerras, (Tue Aug 21, 2:48 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Tue Aug 21, 3:38 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 3:39 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 3:43 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 4:24 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Tue Aug 21, 4:58 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Tue Aug 21, 4:58 am)
Re: [accounting regression since rc1] scheduler updates, Martin Schwidefsky, (Tue Aug 21, 5:57 am)
Re: [accounting regression since rc1] scheduler updates, Christian Borntraeger, (Wed Aug 22, 12:50 am)
Re: [accounting regression since rc1] scheduler updates, Ingo Molnar, (Wed Aug 22, 12:59 am)