[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/4] tests/tcg/aarch64: Fix compilation parameters for pauth-
From: |
Peter Maydell |
Subject: |
Re: [PATCH 2/4] tests/tcg/aarch64: Fix compilation parameters for pauth-% |
Date: |
Mon, 20 Jan 2020 13:10:13 +0000 |
On Thu, 16 Jan 2020 at 23:08, Richard Henderson
<address@hidden> wrote:
>
> Hiding the required architecture within asm() is not correct.
> Add it to the cflags of the (cross-) compiler.
This commit message implies that we're just moving the architecture
specification from the asm statements to the cflags, but the
patch also changes the requested value from v8.4-a to v8.3-a.
I plan to tweak the commit message to be a bit clearer:
We were incorrectly requiring ARMv8.4 support for the pauth
tests, but Pointer Authentication is an ARMv8.3 extension.
Further, hiding the required architecture within asm() is
not correct.
Correct the architecture version requested, and specify it
in the cflags of the (cross-) compiler rather than in the asm.
thanks
-- PMM
- [PATCH 0/4] target/arm: Fix ComputePAC (LP 1859713), Richard Henderson, 2020/01/16
- [PATCH 1/4] target/arm: Fix PAuth sbox functions, Richard Henderson, 2020/01/16
- [PATCH 3/4] tests/tcg/aarch64: Add pauth-3, Richard Henderson, 2020/01/16
- [PATCH 4/4] tests/tcg/aarch64: Add pauth-4, Richard Henderson, 2020/01/16
- [PATCH 2/4] tests/tcg/aarch64: Fix compilation parameters for pauth-%, Richard Henderson, 2020/01/16
- Re: [PATCH 0/4] target/arm: Fix ComputePAC (LP 1859713), no-reply, 2020/01/17
- Re: [PATCH 0/4] target/arm: Fix ComputePAC (LP 1859713), Peter Maydell, 2020/01/20