KVM: selftests: Actually emit forced emulation prefix for kvm_asm_safe_fep()

sean-jc · sean-jc · commit 89ea56a4043a · 2025-02-12T09:00:39.000-08:00
Use KVM_ASM_SAFE_FEP, not simply KVM_ASM_SAFE, for kvm_asm_safe_fep(), as the non-FEP version doesn't force emulation (stating the obvious). Note, there are currently no users of kvm_asm_safe_fep(). Fixes: ab3b6a7 ("KVM: selftests: Add a forced emulation variation of KVM_ASM_SAFE()") Link: https://lore.kernel.org/r/20250130163135.270770-1-seanjc@google.com Signed-off-by: Sean Christopherson <seanjc@google.com>
diff --git a/tools/testing/selftests/kvm/include/x86/processor.h b/tools/testing/selftests/kvm/include/x86/processor.h
@@ -1244,7 +1244,7 @@ void vm_install_exception_handler(struct kvm_vm *vm, int vector,
 	uint64_t ign_error_code;					\
 	uint8_t vector;							\
 									\
-	asm volatile(KVM_ASM_SAFE(insn)					\
+	asm volatile(KVM_ASM_SAFE_FEP(insn)				\
 		     : KVM_ASM_SAFE_OUTPUTS(vector, ign_error_code)	\
 		     : inputs						\
 		     : KVM_ASM_SAFE_CLOBBERS);				\

Original file line number	Diff line number	Diff line change
`@@ -1244,7 +1244,7 @@ void vm_install_exception_handler(struct kvm_vm *vm, int vector,`
`1244`	`1244`	`uint64_t ign_error_code; \`
`1245`	`1245`	`uint8_t vector; \`
`1246`	`1246`	`\`
`1247`		`- asm volatile(KVM_ASM_SAFE(insn) \`
	`1247`	`+ asm volatile(KVM_ASM_SAFE_FEP(insn) \`
`1248`	`1248`	`: KVM_ASM_SAFE_OUTPUTS(vector, ign_error_code) \`
`1249`	`1249`	`: inputs \`
`1250`	`1250`	`: KVM_ASM_SAFE_CLOBBERS); \`