diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2020-06-30 11:38:18 -0400 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2020-07-10 18:02:22 -0400 |
| commit | 1b38750c40281dd0d068f8536b2ea95d7b9bd585 (patch) | |
| tree | 1b42258bbd5256f0e931085af953a9b2f514bf6f /target/i386/sev.c | |
| parent | e4f6278557148151e77260b872b41bcd7ceb4737 (diff) | |
| download | qemu-1b38750c40281dd0d068f8536b2ea95d7b9bd585.zip qemu-1b38750c40281dd0d068f8536b2ea95d7b9bd585.tar.gz qemu-1b38750c40281dd0d068f8536b2ea95d7b9bd585.tar.bz2 | |
target/i386: sev: fail query-sev-capabilities if QEMU cannot use SEV
In some cases, such as if the kvm-amd "sev" module parameter is set
to 0, SEV will be unavailable but query-sev-capabilities will still
return all the information. This tricks libvirt into erroneously
reporting that SEV is available. Check the actual usability of the
feature and return the appropriate error if QEMU cannot use KVM
or KVM cannot use SEV.
Reviewed-by: Eric Blake <eblake@redhat.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'target/i386/sev.c')
| -rw-r--r-- | target/i386/sev.c | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/target/i386/sev.c b/target/i386/sev.c index 7012b1d..c3ecf86 100644 --- a/target/i386/sev.c +++ b/target/i386/sev.c @@ -450,6 +450,15 @@ sev_get_capabilities(Error **errp) uint32_t ebx; int fd; + if (!kvm_enabled()) { + error_setg(errp, "KVM not enabled"); + return NULL; + } + if (kvm_vm_ioctl(kvm_state, KVM_MEMORY_ENCRYPT_OP, NULL) < 0) { + error_setg(errp, "SEV is not enabled in KVM"); + return NULL; + } + fd = open(DEFAULT_SEV_DEVICE, O_RDWR); if (fd < 0) { error_setg_errno(errp, errno, "Failed to open %s", |