O3int interactivity for 2.5.74-mm2
From: | Con Kolivas <kernel@kolivas.org> | |
To: | linux kernel mailing list <linux-kernel@vger.kernel.org> | |
Subject: | [PATCH] O3int interactivity for 2.5.74-mm2 | |
Date: | Mon, 7 Jul 2003 03:16:55 +1000 | |
Cc: | Andrew Morton <akpm@osdl.org> |
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Attached is an incremental patch against 2.5.74-mm2 with more interactivity work. Audio should be quite resistant to skips with this, and it should not induce further unfairness. Changes: The sleep_avg buffer was not needed with the improved semantics in O2int so it has been removed entirely as it created regressions in O2int. A small change to the idle detection code to only make tasks with enough accumulated sleep_avg become idle. Minor cleanups and clarified code. Other issues: Jerky mouse with heavy page rendering in web browsers remains. This is a different issue to the audio and will need some more thought. The patch is also available for download here: http://kernel.kolivas.org/2.5 Note for those who wish to get smooth X desktop feel now for their own use, the granularity patch on that website will do wonders on top of O3int, but a different approach will be needed for mainstream consumption. Con -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.2.2 (GNU/Linux) iD8DBQE/CFmHF6dfvkL3i1gRAkqTAKCjE3lRwBWomZbn/asWtv+OWiFovQCfZo0P UfqOKVgv88faks6+vPq5BGM= =ZR4Y -----END PGP SIGNATURE----- --- linux-2.5.74/kernel/sched.c 2003-07-07 02:13:57.000000000 +1000 +++ linux-2.5.74-test/kernel/sched.c 2003-07-07 02:58:47.000000000 +1000 @@ -77,6 +77,7 @@ #define MAX_SLEEP_AVG (10*HZ) #define STARVATION_LIMIT (10*HZ) #define NODE_THRESHOLD 125 +#define MAX_BONUS ((MAX_USER_PRIO - MAX_RT_PRIO) * PRIO_BONUS_RATIO / 100) /* * If a task is 'interactive' then we reinsert it in the active @@ -306,7 +307,7 @@ static inline void normalise_sleep(task_ { unsigned long old_avg_time = jiffies - p->avg_start; - if (old_avg_time < MIN_SLEEP_AVG) + if (unlikely(old_avg_time < MIN_SLEEP_AVG)) return; if (p->sleep_avg > MAX_SLEEP_AVG) @@ -406,21 +407,16 @@ static inline void activate_task(task_t */ if (runtime < MAX_SLEEP_AVG) p->sleep_avg += (runtime - p->sleep_avg) * (MAX_SLEEP_AVG - runtime) * - (10 - INTERACTIVE_DELTA) / 10 / MAX_SLEEP_AVG; + (MAX_BONUS - INTERACTIVE_DELTA) / MAX_BONUS / MAX_SLEEP_AVG; - /* - * Keep a buffer of 10% sleep_avg - * to prevent short bursts of cpu activity from making - * interactive tasks lose their bonus - */ - if (p->sleep_avg > MAX_SLEEP_AVG * 11/10) - p->sleep_avg = MAX_SLEEP_AVG * 11/10; + if (p->sleep_avg > MAX_SLEEP_AVG) + p->sleep_avg = MAX_SLEEP_AVG; /* * Tasks that sleep a long time are categorised as idle and * get their static priority only */ - if (sleep_time > MIN_SLEEP_AVG) + if ((sleep_time > MIN_SLEEP_AVG) && (p->sleep_avg > runtime / 2)) p->sleep_avg = runtime / 2; if (unlikely(p->avg_start > jiffies)){