summaryrefslogtreecommitdiffstats
path: root/scripts/module-common.lds
diff options
context:
space:
mode:
authorLinus Torvalds <torvalds@linux-foundation.org>2018-03-20 20:16:59 +0100
committerLinus Torvalds <torvalds@linux-foundation.org>2018-03-20 22:58:34 +0100
commit32d43cd391bacb5f0814c2624399a5dad3501d09 (patch)
tree4968d900518f8f8d4a89c6061b99f25bf64b4212 /scripts/module-common.lds
parentMerge branch 'for-4.16-fixes' of git://git.kernel.org/pub/scm/linux/kernel/gi... (diff)
downloadlinux-32d43cd391bacb5f0814c2624399a5dad3501d09.tar.xz
linux-32d43cd391bacb5f0814c2624399a5dad3501d09.zip
kvm/x86: fix icebp instruction handling
The undocumented 'icebp' instruction (aka 'int1') works pretty much like 'int3' in the absense of in-circuit probing equipment (except, obviously, that it raises #DB instead of raising #BP), and is used by some validation test-suites as such. But Andy Lutomirski noticed that his test suite acted differently in kvm than on bare hardware. The reason is that kvm used an inexact test for the icebp instruction: it just assumed that an all-zero VM exit qualification value meant that the VM exit was due to icebp. That is not unlike the guess that do_debug() does for the actual exception handling case, but it's purely a heuristic, not an absolute rule. do_debug() does it because it wants to ascribe _some_ reasons to the #DB that happened, and an empty %dr6 value means that 'icebp' is the most likely casue and we have no better information. But kvm can just do it right, because unlike the do_debug() case, kvm actually sees the real reason for the #DB in the VM-exit interruption information field. So instead of relying on an inexact heuristic, just use the actual VM exit information that says "it was 'icebp'". Right now the 'icebp' instruction isn't technically documented by Intel, but that will hopefully change. The special "privileged software exception" information _is_ actually mentioned in the Intel SDM, even though the cause of it isn't enumerated. Reported-by: Andy Lutomirski <luto@kernel.org> Tested-by: Paolo Bonzini <pbonzini@redhat.com> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'scripts/module-common.lds')
0 files changed, 0 insertions, 0 deletions