Re: [PATCH] selftest/vDSO: Fix cross build for the random tests

From: Jason A. Donenfeld
Date: Fri Aug 30 2024 - 11:33:54 EST


On Fri, Aug 30, 2024 at 03:06:35PM +0100, Mark Brown wrote:
> Unlike the check for the standalone x86 test the check for building the
> vDSO getrandom and chacaha tests looks at the architecture for the host
> rather than the architecture for the target when deciding if they should
> be built. Since the chacha test includes some assembler code this means
> that cross building with x86 as either the target or host is broken. Use
> a check for ARCH instead.
>
> Fixes: 4920a2590e91 ("selftests/vDSO: add tests for vgetrandom")
> Signed-off-by: Mark Brown <broonie@xxxxxxxxxx>

Thanks for the patch. Seems reasonable; I'll queue it up.

> ---
> The x86_64 build is still broken for me because nothing installs
> tools/arch/x86_64/vdso/vgetrandom-chacha.S (I beleive it's supposed to
> be copied from ./arch/x86/entry/vdso/vgetrandom-chacha.S but I don't see
> how?) but this at least fixes all the other architectures.

There should be a symlink installed for that. Are you using this tree?

https://git.kernel.org/pub/scm/linux/kernel/git/crng/random.git

That's where all these fixups are going for 6.12. (And yea, there are a
lot.)