commit 066607f1384f502612869196c97b17ed0fc4caf3
author Mohammed Suhail Munshi <MohammedSuhail.Munshi@arm.com> Wed Jan 19 12:22:50 2022 +0000
committer SiCong Li <sicong.li@arm.com> Fri Jan 21 10:56:35 2022 +0000
tree 0a5d37650e551c4f3c49fd7e623498bd02c8ebdd
parent d5c496d87e3b446532dd3dd163e9768de0daff4e

A73 Devices Regression 300% fix

- Currently regresses on A73 devices (tested on android hikey, inceptionv3), this patch solves this
- Changed mws for all cores to use default values
- Existing mws value for A73 tuned for hikey-linux, caused regression on hikey-android

Resolves [COMPMID-5044]

Change-Id: Ifd6faaa34a0b405d0c390015566f2c75436dfb07
Signed-off-by: Mohammed Suhail Munshi <MohammedSuhail.Munshi@arm.com>
Reviewed-on: https://review.mlplatform.org/c/ml/ComputeLibrary/+/6973
Tested-by: Arm Jenkins <bsgcomp@arm.com>
Comments-Addressed: Arm Jenkins <bsgcomp@arm.com>
Reviewed-by: SiCong Li <sicong.li@arm.com>

src/cpu/kernels/CpuAddKernel.cpp

diff --git a/src/cpu/kernels/CpuAddKernel.cpp b/src/cpu/kernels/CpuAddKernel.cpp
index c27ee9f..deb7379 100644
--- a/src/cpu/kernels/CpuAddKernel.cpp
+++ b/src/cpu/kernels/CpuAddKernel.cpp
@@ -256,19 +256,9 @@
 size_t CpuAddKernel::get_mws(const CPUInfo &platform, size_t thread_count) const
 {
     ARM_COMPUTE_UNUSED(thread_count);
-    // Tuning results that gave optimized results in performance investigation
-    if(platform.get_cpu_model() == CPUModel::A73)
-    {
-        return 10240;
-    }
-    else if(platform.get_cpu_model() == CPUModel::A76)
-    {
-        return 9216;
-    }
-    else
-    {
-        return ICPPKernel::default_mws;
-    }
+    ARM_COMPUTE_UNUSED(platform);
+
+    return ICPPKernel::default_mws;
 }
 
 } // namespace kernels