[ARM CPU] Fix AVGPool tests (openvinotoolkit#17272)

* fix interpolate bug * fix ceil part in avgpool * Update acl_interpolate.cpp * Update acl_interpolate.cpp * Update acl_pooling.cpp * Update acl_pooling.cpp
syaifulnizamiphone7 · Apr 28, 2023 · f4d78ff · f4d78ff
1 parent c6ec6d4
commit f4d78ff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/src/plugins/intel_cpu/src/nodes/executors/acl/acl_pooling.cpp b/src/plugins/intel_cpu/src/nodes/executors/acl/acl_pooling.cpp
@@ -42,6 +42,11 @@ bool AclPoolingExecutor::isSupported(const TensorInfo& srcTensorInfo,
         DEBUG_LOG("Unknown pooling algorithm: ", static_cast<int>(poolingAttrs.algorithm));
         return false;
     }
+
+    // The combination of parameters: NCHW + CEIL gives an accuracy problem in AvgPool. 
+    // One workaround is to disable the ACL executor for these parameters. 
+    // Then OneDNN will run this case in ACL backend as reorder -> NHWC -> reorder
+    if (dataLayout == arm_compute::DataLayout::NCHW && poolingAttrs.rounding == op::RoundingType::CEIL) return false;
     DimensionRoundingType round = (poolingAttrs.rounding == op::RoundingType::CEIL) ?
                                    DimensionRoundingType::CEIL : DimensionRoundingType::FLOOR;
 

diff --git a/src/plugins/intel_cpu/tests/functional/shared_tests_instances/skip_tests_config.cpp b/src/plugins/intel_cpu/tests/functional/shared_tests_instances/skip_tests_config.cpp
@@ -191,7 +191,6 @@ std::vector<std::string> disabledTestPatterns() {
         retVector.emplace_back(R"(smoke_CPU_OVClassCompileModelAndCheckSecondaryPropertiesTest.*)");
         retVector.emplace_back(R"(smoke_CPU_OVClassCompileModelAndCheckWithSecondaryPropertiesDoubleTest.*)");
     }
-    retVector.emplace_back(R"(smoke_AvgPool_ExplicitPad_CeilRounding/PoolingLayerTest.CompareWithRefs.*)");
     // invalid test: checks u8 precision for runtime graph, while it should be f32
     retVector.emplace_back(R"(smoke_NegativeQuantizedMatMulMultiplyFusion.*)");
     // int8 specific