Thread (5 messages) 5 messages, 2 authors, 2026-03-27

Re: [PATCH] media: rockchip: Disable VIDEO_ROCKCHIP_VDEC when compile testing for Hexagon

From: Nathan Chancellor <nathan@kernel.org>
Date: 2026-03-19 20:24:37
Also in: linux-media, linux-rockchip, lkml, llvm

On Thu, Mar 19, 2026 at 04:08:29PM -0400, Nicolas Dufresne wrote:
Le lundi 16 février 2026 à 11:17 -0500, Nicolas Dufresne a écrit :
quoted
Le vendredi 13 février 2026 à 15:10 -0500, Nathan Chancellor a écrit :
quoted
Building rkvdec-vdpu383-h264.c can take a few hours to finish building
with Clang 20.1.0 or newer when compile testing for Hexagon. While this
is further investigated and understood on the LLVM side [1], disable
CONFIG_VIDEO_ROCKCHIP_VDEC when compile testing for Hexagon.

Link: https://github.com/llvm/llvm-project/issues/178535 [1]
Signed-off-by: Nathan Chancellor <nathan@kernel.org>
---
 drivers/media/platform/rockchip/rkvdec/Kconfig | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/media/platform/rockchip/rkvdec/Kconfig
b/drivers/media/platform/rockchip/rkvdec/Kconfig
index 5f3bdd848a2c..d03689464206 100644
--- a/drivers/media/platform/rockchip/rkvdec/Kconfig
+++ b/drivers/media/platform/rockchip/rkvdec/Kconfig
@@ -1,7 +1,8 @@
 # SPDX-License-Identifier: GPL-2.0
 config VIDEO_ROCKCHIP_VDEC
 	tristate "Rockchip Video Decoder driver"
-	depends on ARCH_ROCKCHIP || COMPILE_TEST
+	# !HEXAGON: https://github.com/llvm/llvm-project/issues/178535
+	depends on ARCH_ROCKCHIP || (COMPILE_TEST && !HEXAGON)
This is clearly not a pleasing change to make. As this specific data structure
and usage of bitfield has been discussed (along with the numerous issues in
clang/llvm around these). We also agreed to move away from bitfield for this
data structure and use a bitwriter. I would favour delaying this change to
give
devs the time to port instead. Ping again if nothing moves within few weeks.

best regards,
Nicolas
I haven't heard back about the port to plain bitwriter. I guess I have to pick
this patch, but I really don't want to have to maintain too many of these hacks.
Anyone else with an opinion on the topic ? Or a better idea how this can be
workaround differently ?
For what it's worth, I don't anticipate more of these hacks. Hexagon is
rather special in that it has a lot of target specific optimization
passes, which do not always see code as complex as Linux has in places
during development. Ideally, it is resolved at some point during the
LLVM 23 development cycle then we can version check this workaround and
eventually clean it up altogether.

Cheers,
Nathan
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help