On Thu, 2 Feb 2023 at 21:13, Aaron Lindsay <aa...@os.amperecomputing.com> wrote:
>
> Signed-off-by: Aaron Lindsay <aa...@os.amperecomputing.com>
> diff --git a/target/arm/pauth_helper.c b/target/arm/pauth_helper.c
> index d0483bf051..a0c9bea06b 100644
> --- a/target/arm/pauth_helper.c
> +++ b/target/arm/pauth_helper.c
> @@ -282,8 +282,8 @@ static uint64_t pauth_computepac_impdef(uint64_t data,
> uint64_t modifier,
> static uint64_t pauth_computepac(CPUARMState *env, uint64_t data,
> uint64_t modifier, ARMPACKey key)
> {
> - if (cpu_isar_feature(aa64_pauth_arch, env_archcpu(env))) {
> - return pauth_computepac_architected(data, modifier, key);
> + if (cpu_isar_feature(aa64_pauth_arch_qarma5, env_archcpu(env))) {
> + return pauth_computepac_architected(data, modifier, key, false);
> } else {
> return pauth_computepac_impdef(data, modifier, key);
> }
> --
> 2.25.1
Just noticed -- this should I guess be in a later patch, because
it's added an extra argument to the function call without changing
the function definition or declaration, so I think this patch
won't compile as-is.
-- PMM
