From b9f52b6b869b52d52893ed2604f49689bda9a378 Mon Sep 17 00:00:00 2001 From: Felix Fietkau Date: Mon, 7 Mar 2016 21:00:46 +0000 Subject: [PATCH] kernel: add MIPS kernel fix for an uninitialized CPU map Signed-off-by: Felix Fietkau SVN-Revision: 48956 --- ...c-Fix-uninitialised-temp_foreign_map.patch | 31 +++++++++++++++++++ 1 file changed, 31 insertions(+) create mode 100644 target/linux/generic/patches-4.4/103-MIPS-smp.c-Fix-uninitialised-temp_foreign_map.patch diff --git a/target/linux/generic/patches-4.4/103-MIPS-smp.c-Fix-uninitialised-temp_foreign_map.patch b/target/linux/generic/patches-4.4/103-MIPS-smp.c-Fix-uninitialised-temp_foreign_map.patch new file mode 100644 index 0000000000..6fe9d96ccc --- /dev/null +++ b/target/linux/generic/patches-4.4/103-MIPS-smp.c-Fix-uninitialised-temp_foreign_map.patch @@ -0,0 +1,31 @@ +From: James Hogan +Date: Fri, 4 Mar 2016 10:10:51 +0000 +Subject: [PATCH] MIPS: smp.c: Fix uninitialised temp_foreign_map + +When calculate_cpu_foreign_map() recalculates the cpu_foreign_map +cpumask it uses the local variable temp_foreign_map without initialising +it to zero. Since the calculation only ever sets bits in this cpumask +any existing bits at that memory location will remain set and find their +way into cpu_foreign_map too. This could potentially lead to cache +operations suboptimally doing smp calls to multiple VPEs in the same +core, even though the VPEs share primary caches. + +Therefore initialise temp_foreign_map using cpumask_clear() before use. + +Fixes: cccf34e9411c ("MIPS: c-r4k: Fix cache flushing for MT cores") +Signed-off-by: James Hogan +Cc: Ralf Baechle +Cc: Paul Burton +Cc: linux-mips@linux-mips.org +--- + +--- a/arch/mips/kernel/smp.c ++++ b/arch/mips/kernel/smp.c +@@ -121,6 +121,7 @@ static inline void calculate_cpu_foreign + cpumask_t temp_foreign_map; + + /* Re-calculate the mask */ ++ cpumask_clear(&temp_foreign_map); + for_each_online_cpu(i) { + core_present = 0; + for_each_cpu(k, &temp_foreign_map)