]> git.baikalelectronics.ru Git - kernel.git/commit
x86,static_call: Fix __static_call_return0 for i386
authorPeter Zijlstra <peterz@infradead.org>
Fri, 18 Mar 2022 20:24:38 +0000 (21:24 +0100)
committerPeter Zijlstra <peterz@infradead.org>
Tue, 5 Apr 2022 07:59:37 +0000 (09:59 +0200)
commit888c437bc97e01904ff64126930c49839fe832f0
tree67088bbdb10a57988b8cc323e65abb0cb20c865e
parentdbb008736a8c113ef1e12b4e4b4351fac2a15ede
x86,static_call: Fix __static_call_return0 for i386

Paolo reported that the instruction sequence that is used to replace:

    call __static_call_return0

namely:

    66 66 48 31 c0 data16 data16 xor %rax,%rax

decodes to something else on i386, namely:

    66 66 48 data16 dec %ax
    31 c0 xor    %eax,%eax

Which is a nonsensical sequence that happens to have the same outcome.
*However* an important distinction is that it consists of 2
instructions which is a problem when the thing needs to be overwriten
with a regular call instruction again.

As such, replace the instruction with something that decodes the same
on both i386 and x86_64.

Fixes: 870d65d12716 ("static_call/x86: Add __static_call_return0()")
Reported-by: Paolo Bonzini <pbonzini@redhat.com>
Signed-off-by: Peter Zijlstra (Intel) <peterz@infradead.org>
Link: https://lkml.kernel.org/r/20220318204419.GT8939@worktop.programming.kicks-ass.net
arch/x86/kernel/static_call.c