linux-kernel - Re: [PATCH] x86,kvm/xen: Remove superfluous .fixup usage

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <ca66ba8d-63dd-7087-7d5b-b42ccf8debd7@redhat.com>
Date:   Mon, 24 Jan 2022 14:53:13 +0100
From:   Paolo Bonzini <pbonzini@...hat.com>
To:     Peter Zijlstra <peterz@...radead.org>, x86@...nel.org
Cc:     dwmw@...zon.co.uk, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] x86,kvm/xen: Remove superfluous .fixup usage

On 1/23/22 13:42, Peter Zijlstra wrote:
> 
> Commit 14243b387137 ("KVM: x86/xen: Add KVM_IRQ_ROUTING_XEN_EVTCHN and
> event channel delivery") adds superfluous .fixup usage after the whole
> .fixup section was removed in commit e5eefda5aa51 ("x86: Remove .fixup
> section").
> 
> Fixes: 14243b387137 ("KVM: x86/xen: Add KVM_IRQ_ROUTING_XEN_EVTCHN and event channel delivery")
> Reported-by: Borislav Petkov <bp@...en8.de>
> Signed-off-by: Peter Zijlstra (Intel) <peterz@...radead.org>
> ---
>   arch/x86/kvm/xen.c | 10 ++--------
>   1 file changed, 2 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/x86/kvm/xen.c b/arch/x86/kvm/xen.c
> index 0e3f7d6e9fd7..bad57535fad0 100644
> --- a/arch/x86/kvm/xen.c
> +++ b/arch/x86/kvm/xen.c
> @@ -316,10 +316,7 @@ int __kvm_xen_has_interrupt(struct kvm_vcpu *v)
>   				     "\tnotq %0\n"
>   				     "\t" LOCK_PREFIX "andq %0, %2\n"
>   				     "2:\n"
> -				     "\t.section .fixup,\"ax\"\n"
> -				     "3:\tjmp\t2b\n"
> -				     "\t.previous\n"
> -				     _ASM_EXTABLE_UA(1b, 3b)
> +				     _ASM_EXTABLE_UA(1b, 2b)
>   				     : "=r" (evtchn_pending_sel),
>   				       "+m" (vi->evtchn_pending_sel),
>   				       "+m" (v->arch.xen.evtchn_pending_sel)
> @@ -335,10 +332,7 @@ int __kvm_xen_has_interrupt(struct kvm_vcpu *v)
>   				     "\tnotl %0\n"
>   				     "\t" LOCK_PREFIX "andl %0, %2\n"
>   				     "2:\n"
> -				     "\t.section .fixup,\"ax\"\n"
> -				     "3:\tjmp\t2b\n"
> -				     "\t.previous\n"
> -				     _ASM_EXTABLE_UA(1b, 3b)
> +				     _ASM_EXTABLE_UA(1b, 2b)
>   				     : "=r" (evtchn_pending_sel32),
>   				       "+m" (vi->evtchn_pending_sel),
>   				       "+m" (v->arch.xen.evtchn_pending_sel)
> 

Queued, thanks.

Paolo