From patchwork Thu May 14 06:41:02 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Gibson X-Patchwork-Id: 11548041 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 4CF6E913 for ; Thu, 14 May 2020 06:41:34 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 388D1206D4 for ; Thu, 14 May 2020 06:41:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (1024-bit key) header.d=gibson.dropbear.id.au header.i=@gibson.dropbear.id.au header.b="Qi1X9mS3" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726159AbgENGld (ORCPT ); Thu, 14 May 2020 02:41:33 -0400 Received: from ozlabs.org ([203.11.71.1]:60495 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725931AbgENGlc (ORCPT ); Thu, 14 May 2020 02:41:32 -0400 Received: by ozlabs.org (Postfix, from userid 1007) id 49N24j0crHz9sSd; Thu, 14 May 2020 16:41:24 +1000 (AEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gibson.dropbear.id.au; s=201602; t=1589438485; bh=vvYudY/uRKvNzTDTxC0FfBGKSAwOXcVJGXD3YDR90r8=; h=From:To:Cc:Subject:Date:From; b=Qi1X9mS3JVsGJg+QbwlDGmsYTuiA4YSccRAY/Py97qcQMVRwNMOPvlEs7c7Pn91lu GWudnf/ACmQ0Y39KeD7PtetxzEiQhHc/FV+7ArvAPj3DM3cDsg2Q1DWrkBWnz+TisQ E9EReUmtMIIyFJvFy6eQpPzjN9qoLK8mWZnJum7c= From: David Gibson To: dgilbert@redhat.com, frankja@linux.ibm.com, pair@us.redhat.com, qemu-devel@nongnu.org, brijesh.singh@amd.com Cc: kvm@vger.kernel.org, qemu-ppc@nongnu.org, David Gibson , Richard Henderson , cohuck@redhat.com, Paolo Bonzini , Marcel Apfelbaum , "Michael S. Tsirkin" , Eduardo Habkost , qemu-devel@nongnu.-rg, mdroth@linux.vnet.ibm.com Subject: [RFC 00/18] Refactor configuration of guest memory protection Date: Thu, 14 May 2020 16:41:02 +1000 Message-Id: <20200514064120.449050-1-david@gibson.dropbear.id.au> X-Mailer: git-send-email 2.26.2 MIME-Version: 1.0 Sender: kvm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: kvm@vger.kernel.org A number of hardware platforms are implementing mechanisms whereby the hypervisor does not have unfettered access to guest memory, in order to mitigate the security impact of a compromised hypervisor. AMD's SEV implements this with in-cpu memory encryption, and Intel has its own memory encryption mechanism. POWER has an upcoming mechanism to accomplish this in a different way, using a new memory protection level plus a small trusted ultravisor. s390 also has a protected execution environment. The current code (committed or draft) for these features has each platform's version configured entirely differently. That doesn't seem ideal for users, or particularly for management layers. AMD SEV introduces a notionally generic machine option "machine-encryption", but it doesn't actually cover any cases other than SEV. This series is a proposal to at least partially unify configuration for these mechanisms, by renaming and generalizing AMD's "memory-encryption" property. It is replaced by a "guest-memory-protection" property pointing to a platform specific object which configures and manages the specific details. For now this series covers just AMD SEV and POWER PEF. I'm hoping it can be extended to cover the Intel and s390 mechanisms as well, though. Note: I'm using the term "guest memory protection" throughout to refer to mechanisms like this. I don't particular like the term, it's both long and not really precise. If someone can think of a succinct way of saying "a means of protecting guest memory from a possibly compromised hypervisor", I'd be grateful for the suggestion. David Gibson (18): target/i386: sev: Remove unused QSevGuestInfoClass target/i386: sev: Move local structure definitions into .c file target/i386: sev: Rename QSevGuestInfo target/i386: sev: Embed SEVState in SevGuestState target/i386: sev: Partial cleanup to sev_state global target/i386: sev: Remove redundant cbitpos and reduced_phys_bits fields target/i386: sev: Remove redundant policy field target/i386: sev: Remove redundant handle field target/i386: sev: Unify SEVState and SevGuestState guest memory protection: Add guest memory protection interface guest memory protection: Handle memory encrption via interface guest memory protection: Perform KVM init via interface guest memory protection: Move side effect out of machine_set_memory_encryption() guest memory protection: Rework the "memory-encryption" property guest memory protection: Decouple kvm_memcrypt_*() helpers from KVM use errp for gmpo kvm_init spapr: Added PEF based guest memory protection guest memory protection: Alter virtio default properties for protected guests accel/kvm/kvm-all.c | 40 +-- accel/kvm/sev-stub.c | 5 - accel/stubs/kvm-stub.c | 10 - backends/Makefile.objs | 2 + backends/guest-memory-protection.c | 29 ++ hw/core/machine.c | 61 ++++- hw/i386/pc_sysfw.c | 6 +- include/exec/guest-memory-protection.h | 77 ++++++ include/hw/boards.h | 4 +- include/sysemu/kvm.h | 17 -- include/sysemu/sev.h | 6 +- target/i386/sev.c | 358 +++++++++++++------------ target/i386/sev_i386.h | 49 ---- target/ppc/Makefile.objs | 2 +- target/ppc/pef.c | 81 ++++++ 15 files changed, 445 insertions(+), 302 deletions(-) create mode 100644 backends/guest-memory-protection.c create mode 100644 include/exec/guest-memory-protection.h create mode 100644 target/ppc/pef.c