Re: [PATCH v3 12/16] kunit: qemu_configs: loongarch: Enable LSX/LSAX

[David Gow <davidgow@google.com>]

[-- Attachment #1: Type: text/plain, Size: 1374 bytes --]

On Wed, 11 Jun 2025 at 15:38, Thomas Weißschuh
<thomas.weissschuh@linutronix.de> wrote:
>
> The upcoming kunit UAPI framework will run userspace executables as part of
> kunit. These may use the LSX or LASX instructions.
>
> Make sure the kunit kernel can handle these instructions.
>
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---

Looks fine to me. I confess to not knowing much about the details of
Loongarch CPUs, though, so while it hasn't broken anything here, I
make no further guarantees.

Reviewed-by: David Gow <davidgow@google.com>

Cheers,
-- David


>  tools/testing/kunit/qemu_configs/loongarch.py | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/tools/testing/kunit/qemu_configs/loongarch.py b/tools/testing/kunit/qemu_configs/loongarch.py
> index a92422967d1da9f1658ef1e80d0d7365ddbae307..1dba755284f11ffc94d8946105b0cfa49cb6f604 100644
> --- a/tools/testing/kunit/qemu_configs/loongarch.py
> +++ b/tools/testing/kunit/qemu_configs/loongarch.py
> @@ -11,6 +11,8 @@ CONFIG_PVPANIC_PCI=y
>  CONFIG_SERIAL_8250=y
>  CONFIG_SERIAL_8250_CONSOLE=y
>  CONFIG_SERIAL_OF_PLATFORM=y
> +CONFIG_CPU_HAS_LSX=y
> +CONFIG_CPU_HAS_LASX=y
>  ''',
>                            qemu_arch='loongarch64',
>                            kernel_path='arch/loongarch/boot/vmlinux.elf',
>
> --
> 2.49.0
>

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 5281 bytes --]