diff options
author | Ingo Molnar <mingo@elte.hu> | 2007-07-09 12:51:57 -0400 |
---|---|---|
committer | Ingo Molnar <mingo@elte.hu> | 2007-07-09 12:51:57 -0400 |
commit | 0437e109e1841607f2988891eaa36c531c6aa6ac (patch) | |
tree | e9d8f170786f7e33d4c5829cb008cf38d42a2014 /arch/ia64 | |
parent | 0e6aca43e08a62a48d6770e9a159dbec167bf4c6 (diff) |
sched: zap the migration init / cache-hot balancing code
the SMP load-balancer uses the boot-time migration-cost estimation
code to attempt to improve the quality of balancing. The reason for
this code is that the discrete priority queues do not preserve
the order of scheduling accurately, so the load-balancer skips
tasks that were running on a CPU 'recently'.
this code is fundamental fragile: the boot-time migration cost detector
doesnt really work on systems that had large L3 caches, it caused boot
delays on large systems and the whole cache-hot concept made the
balancing code pretty undeterministic as well.
(and hey, i wrote most of it, so i can say it out loud that it sucks ;-)
under CFS the same purpose of cache affinity can be achieved without
any special cache-hot special-case: tasks are sorted in the 'timeline'
tree and the SMP balancer picks tasks from the left side of the
tree, thus the most cache-cold task is balanced automatically.
Signed-off-by: Ingo Molnar <mingo@elte.hu>
Diffstat (limited to 'arch/ia64')
-rw-r--r-- | arch/ia64/kernel/setup.c | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/arch/ia64/kernel/setup.c b/arch/ia64/kernel/setup.c index eaa6a24bc0b6..188fb73c6845 100644 --- a/arch/ia64/kernel/setup.c +++ b/arch/ia64/kernel/setup.c | |||
@@ -805,7 +805,6 @@ static void __cpuinit | |||
805 | get_max_cacheline_size (void) | 805 | get_max_cacheline_size (void) |
806 | { | 806 | { |
807 | unsigned long line_size, max = 1; | 807 | unsigned long line_size, max = 1; |
808 | unsigned int cache_size = 0; | ||
809 | u64 l, levels, unique_caches; | 808 | u64 l, levels, unique_caches; |
810 | pal_cache_config_info_t cci; | 809 | pal_cache_config_info_t cci; |
811 | s64 status; | 810 | s64 status; |
@@ -835,8 +834,6 @@ get_max_cacheline_size (void) | |||
835 | line_size = 1 << cci.pcci_line_size; | 834 | line_size = 1 << cci.pcci_line_size; |
836 | if (line_size > max) | 835 | if (line_size > max) |
837 | max = line_size; | 836 | max = line_size; |
838 | if (cache_size < cci.pcci_cache_size) | ||
839 | cache_size = cci.pcci_cache_size; | ||
840 | if (!cci.pcci_unified) { | 837 | if (!cci.pcci_unified) { |
841 | status = ia64_pal_cache_config_info(l, | 838 | status = ia64_pal_cache_config_info(l, |
842 | /* cache_type (instruction)= */ 1, | 839 | /* cache_type (instruction)= */ 1, |
@@ -853,9 +850,6 @@ get_max_cacheline_size (void) | |||
853 | ia64_i_cache_stride_shift = cci.pcci_stride; | 850 | ia64_i_cache_stride_shift = cci.pcci_stride; |
854 | } | 851 | } |
855 | out: | 852 | out: |
856 | #ifdef CONFIG_SMP | ||
857 | max_cache_size = max(max_cache_size, cache_size); | ||
858 | #endif | ||
859 | if (max > ia64_max_cacheline_size) | 853 | if (max > ia64_max_cacheline_size) |
860 | ia64_max_cacheline_size = max; | 854 | ia64_max_cacheline_size = max; |
861 | } | 855 | } |