Re: [PATCH] target/i386: Remove pointless env_archcpu() in helper_rdmsr()

[Daniel Henrique Barboza]

On 2/10/23 09:57, Philippe Mathieu-Daudé wrote:
> We have a X86CPU *cpu pointer available at the start of the function.
>
> Inspired-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>
> Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
> ---

Reviewed-by: Daniel Henrique Barboza <dbarboza@ventanamicro.com>


>   target/i386/tcg/sysemu/misc_helper.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target/i386/tcg/sysemu/misc_helper.c 
> b/target/i386/tcg/sysemu/misc_helper.c
> index e1528b7f80..e47db9376e 100644
> --- a/target/i386/tcg/sysemu/misc_helper.c
> +++ b/target/i386/tcg/sysemu/misc_helper.c
> @@ -326,7 +326,7 @@ void helper_rdmsr(CPUX86State *env)
>           val = env->sysenter_eip;
>           break;
>       case MSR_IA32_APICBASE:
> -        val = cpu_get_apic_base(env_archcpu(env)->apic_state);
> +        val = cpu_get_apic_base(x86_cpu->apic_state);
>           break;
>       case MSR_EFER:
>           val = env->efer;