Script 'mail_helper' called by obssrc Hello community, here is the log from the commit of package onednn for openSUSE:Factory checked in at 2022-09-22 14:49:46 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Comparing /work/SRC/openSUSE:Factory/onednn (Old) and /work/SRC/openSUSE:Factory/.onednn.new.2275 (New) ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Package is "onednn" Thu Sep 22 14:49:46 2022 rev:8 rq:1005197 version:2.6.2 Changes: -------- --- /work/SRC/openSUSE:Factory/onednn/onednn.changes 2022-09-14 13:44:53.305880100 +0200 +++ /work/SRC/openSUSE:Factory/.onednn.new.2275/onednn.changes 2022-09-22 14:50:07.814467486 +0200 @@ -1,0 +2,7 @@ +Tue Sep 20 08:26:43 UTC 2022 - Guillaume GARDET <guillaume.gar...@opensuse.org> + +- Add patch to fix build with latest Arm Compute Library: + * 1428.patch + * fa93750.patch (dep for 1428.patch) + +------------------------------------------------------------------- New: ---- 1428.patch fa93750.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ onednn.spec ++++++ --- /var/tmp/diff_new_pack.aFHR5g/_old 2022-09-22 14:50:08.462469131 +0200 +++ /var/tmp/diff_new_pack.aFHR5g/_new 2022-09-22 14:50:08.466469141 +0200 @@ -37,6 +37,10 @@ License: Apache-2.0 URL: https://01.org/onednn Source0: https://github.com/oneapi-src/oneDNN/archive/v%{version}/oneDNN-%{version}.tar.gz +# PATCH-FIX-UPSTREAM - deps for Patch2 +Patch1: fa93750.patch +# PATCH-FIX-UPSTREAM - Fix build with latest ACL - https://github.com/oneapi-src/oneDNN/pull/1428 +Patch2: 1428.patch BuildRequires: chrpath BuildRequires: cmake BuildRequires: doxygen ++++++ 1428.patch ++++++ >From 24d4c1379d3c4cbf423f979fa25283b1914159c7 Mon Sep 17 00:00:00 2001 From: Diana Bite <diana.b...@arm.com> Date: Thu, 21 Jul 2022 12:27:17 +0100 Subject: [PATCH] cpu: aarch64: acl: disallow large padding in ACL wino to be consist with oneDNN --- src/cpu/aarch64/acl_convolution_utils.cpp | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/cpu/aarch64/acl_convolution_utils.cpp b/src/cpu/aarch64/acl_convolution_utils.cpp index db3bde6a48..542033df0e 100644 --- a/src/cpu/aarch64/acl_convolution_utils.cpp +++ b/src/cpu/aarch64/acl_convolution_utils.cpp @@ -307,10 +307,17 @@ status_t init_conf_wino(acl_conv_conf_t &acp, memory_desc_t &src_md, // General Compute Library checks, memory tags are also set there CHECK(acl_init_conf(acp, src_md, weights_md, dst_md, bias_md, cd, attr)); - const bool wino_shape_ok // unit strides only, no dilations + const bool shape_ok + // only unit strides allowed = (acp.padstride_info.stride() == std::pair<uint, uint> {1, 1}) + // Note: Compute Library supports arbitrary padding for wino kernels + // but we only allow small padding to be consistent with oneDNN + && (acp.padstride_info.pad().first <= 1) // padding left/right + && (acp.padstride_info.pad().second <= 1) // padding top/bottom + // only non-dilated convolutions allowed && (acp.dilation_info == arm_compute::Size2D(1, 1)); - if (!wino_shape_ok) return status::unimplemented; + + ACL_CHECK_SUPPORT(!shape_ok, "shape not supported by winograd kernels"); // clang-format off // Validate convolution manually to check for return status ++++++ fa93750.patch ++++++ ++++ 1223 lines (skipped)
