Summary: | media-libs/opencv-4.8.0-r1: /modules/dnn/src/layers/cpu_kernels/convolution.cpp:143:19: error: ‘struct cv::dnn::FastConv’ has no member named ‘weightsBuf_FP16’; did you mean ‘weightsBufPtr’? | ||
---|---|---|---|
Product: | Gentoo Linux | Reporter: | Sam James <sam> |
Component: | Current packages | Assignee: | No maintainer - Look at https://wiki.gentoo.org/wiki/Project:Proxy_Maintainers if you want to take care of it <maintainer-needed> |
Status: | RESOLVED FIXED | ||
Severity: | normal | Keywords: | PATCH |
Priority: | Normal | ||
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux | ||
See Also: |
https://github.com/opencv/opencv/pull/24203 https://bugs.gentoo.org/show_bug.cgi?id=913131 |
||
Whiteboard: | |||
Package list: | Runtime testing required: | --- | |
Attachments: |
build.log
hack.patch |
Description
Sam James
2023-08-26 05:10:19 UTC
(I've hit this in a normal ~arm64 chroot too, not just w/ gcc 14) arm only path: https://github.com/opencv/opencv/blob/588ddf1b181aa7243144b27d65fc7690fb89e344/modules/dnn/src/layers/cpu_kernels/convolution.hpp#L66 (In reply to Sam James from comment #2) > arm only path: > https://github.com/opencv/opencv/blob/ > 588ddf1b181aa7243144b27d65fc7690fb89e344/modules/dnn/src/layers/cpu_kernels/ > convolution.hpp#L66 The guard at https://github.com/opencv/opencv/blob/588ddf1b181aa7243144b27d65fc7690fb89e344/modules/dnn/src/layers/cpu_kernels/convolution.cpp#L140 is mismatched with the place it's defined at? Created attachment 868789 [details, diff]
hack.patch
Attached hack makes things build.
I think this broke with https://github.com/opencv/opencv/commit/5229312ad2a4fbf9a6abfd6490b265c39587e6f7. With above hack applied, I get: -- CPU/HW features: -- Baseline: NEON FP16 so I guess on arm64 atm it really wants fp16, but it could easily be just because it's detecting the hw really supports it then it finds the mismatch The bug has been closed via the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=4e6bb1138e0e8009b6e66b479e417d4d7c066fda commit 4e6bb1138e0e8009b6e66b479e417d4d7c066fda Author: Sam James <sam@gentoo.org> AuthorDate: 2023-08-28 03:44:11 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-08-28 04:03:47 +0000 media-libs/opencv: fix build on (some) arm64 See patch for details. Closes: https://bugs.gentoo.org/913031 Signed-off-by: Sam James <sam@gentoo.org> .../opencv/files/opencv-4.8.0-arm64-fp16.patch | 272 +++++++++++++++++++++ media-libs/opencv/opencv-4.8.0-r1.ebuild | 1 + 2 files changed, 273 insertions(+) The bug has been referenced in the following commit(s): https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f86f0e650d2bc739837a28cd32bc4c255cca258a commit f86f0e650d2bc739837a28cd32bc4c255cca258a Author: Sam James <sam@gentoo.org> AuthorDate: 2023-08-28 06:43:37 +0000 Commit: Sam James <sam@gentoo.org> CommitDate: 2023-08-28 06:43:37 +0000 media-libs/opencv: fix amd64 build (oops) Bug: https://bugs.gentoo.org/913031 Closes: https://bugs.gentoo.org/913131 Fixes: 4e6bb1138e0e8009b6e66b479e417d4d7c066fda Signed-off-by: Sam James <sam@gentoo.org> .../opencv/files/opencv-4.8.0-arm64-fp16.patch | 56 +++++++++++----------- 1 file changed, 28 insertions(+), 28 deletions(-) |