diff options
| author | Joerg Roedel <jroedel@suse.de> | 2020-10-02 09:44:32 +0200 |
|---|---|---|
| committer | Joerg Roedel <jroedel@suse.de> | 2020-10-02 09:44:32 +0200 |
| commit | ff6f1efb74f69a152916bbb4c60248b1ca0d1a73 (patch) | |
| tree | 25c2ca3f0e0478b5cdbb7019c35bc86f71e29e93 | |
| parent | 5282faf01e085d57658a39494ea760c2b7309f3d (diff) | |
| download | linux-sev-es-tip-updates.tar.gz | |
x86/sev-es: Replace open-coded hlt-loops with sev_es_terminate()sev-es-tip-updates
There are a few places left in the SEV-ES C code where hlt loops and/or
terminate requests are implemented. Replace them all with calls to
sev_es_terminate().
Signed-off-by: Joerg Roedel <jroedel@suse.de>
| -rw-r--r-- | arch/x86/boot/compressed/sev-es.c | 12 | ||||
| -rw-r--r-- | arch/x86/kernel/sev-es-shared.c | 10 |
2 files changed, 6 insertions, 16 deletions
diff --git a/arch/x86/boot/compressed/sev-es.c b/arch/x86/boot/compressed/sev-es.c index 954cb2702e239f..9aecb2e9c743c0 100644 --- a/arch/x86/boot/compressed/sev-es.c +++ b/arch/x86/boot/compressed/sev-es.c @@ -201,14 +201,8 @@ void do_boot_stage2_vc(struct pt_regs *regs, unsigned long exit_code) } finish: - if (result == ES_OK) { + if (result == ES_OK) vc_finish_insn(&ctxt); - } else if (result != ES_RETRY) { - /* - * For now, just halt the machine. That makes debugging easier, - * later we just call sev_es_terminate() here. - */ - while (true) - asm volatile("hlt\n"); - } + else if (result != ES_RETRY) + sev_es_terminate(GHCB_SEV_ES_REASON_GENERAL_REQUEST); } diff --git a/arch/x86/kernel/sev-es-shared.c b/arch/x86/kernel/sev-es-shared.c index 48bb14563dcdb6..b51f63bd5b521d 100644 --- a/arch/x86/kernel/sev-es-shared.c +++ b/arch/x86/kernel/sev-es-shared.c @@ -24,7 +24,7 @@ static bool __init sev_es_check_cpu_features(void) return true; } -static void sev_es_terminate(unsigned int reason) +static void __noreturn sev_es_terminate(unsigned int reason) { u64 val = GHCB_SEV_TERMINATE; @@ -210,12 +210,8 @@ void __init do_vc_no_ghcb(struct pt_regs *regs, unsigned long exit_code) return; fail: - sev_es_wr_ghcb_msr(GHCB_SEV_TERMINATE); - VMGEXIT(); - - /* Shouldn't get here - if we do halt the machine */ - while (true) - asm volatile("hlt\n"); + /* Terminate the guest */ + sev_es_terminate(GHCB_SEV_ES_REASON_GENERAL_REQUEST); } static enum es_result vc_insn_string_read(struct es_em_ctxt *ctxt, |