From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46009) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bk61l-0008F2-A7 for qemu-devel@nongnu.org; Wed, 14 Sep 2016 04:58:46 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bk61f-000521-98 for qemu-devel@nongnu.org; Wed, 14 Sep 2016 04:58:44 -0400 Received: from mail-wm0-f67.google.com ([74.125.82.67]:36628) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bk61f-00051o-04 for qemu-devel@nongnu.org; Wed, 14 Sep 2016 04:58:39 -0400 Received: by mail-wm0-f67.google.com with SMTP id b184so1495643wma.3 for ; Wed, 14 Sep 2016 01:58:38 -0700 (PDT) Sender: Paolo Bonzini References: <147377800565.11859.4411044563640180545.stgit@brijesh-build-machine> <147377810767.11859.4668503556528840901.stgit@brijesh-build-machine> <20160914052034-mutt-send-email-mst@kernel.org> From: Paolo Bonzini Message-ID: <4f4370ee-bc29-3427-7e6e-a18d50c27ffc@redhat.com> Date: Wed, 14 Sep 2016 10:57:34 +0200 MIME-Version: 1.0 In-Reply-To: <20160914052034-mutt-send-email-mst@kernel.org> Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [RFC PATCH v1 10/22] sev: add SEV debug decrypt command List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" , Brijesh Singh Cc: ehabkost@redhat.com, crosthwaite.peter@gmail.com, p.fedin@samsung.com, qemu-devel@nongnu.org, armbru@redhat.com, lcapitulino@redhat.com, rth@twiddle.net On 14/09/2016 04:28, Michael S. Tsirkin wrote: > On Tue, Sep 13, 2016 at 10:48:27AM -0400, Brijesh Singh wrote: >> The SEV DEBUG_DECRYPT command is used for decrypting a guest memory >> for the debugging purposes. Note that debugging is permitting only >> when guest policy allows it. > > When wouldn't you want to allow it? > I don't see value in a "break debugging" feature. One man's "allow debugging" feature is another man's "break encryption" feature. :) Paolo > >> For more information see [1], section 7.1 >> >> [1] http://support.amd.com/TechDocs/55766_SEV-KM%20API_Spec.pdf > > Please add comments documenting APIs. Spec links to figure out > implementation is one thing, but you really can't require people > to read specs just to figure out how to use an API. > >> The following KVM RFC patches defines and implements this command >> >> http://marc.info/?l=kvm&m=147190852423972&w=2 >> http://marc.info/?l=kvm&m=147191068524579&w=2 >> >> Signed-off-by: Brijesh Singh >> --- >> include/sysemu/sev.h | 10 ++++++++++ >> sev.c | 23 +++++++++++++++++++++++ >> 2 files changed, 33 insertions(+) >> >> diff --git a/include/sysemu/sev.h b/include/sysemu/sev.h >> index ab03c5d..5872c3e 100644 >> --- a/include/sysemu/sev.h >> +++ b/include/sysemu/sev.h >> @@ -55,4 +55,14 @@ int kvm_sev_guest_finish(void); >> */ >> int kvm_sev_guest_measurement(uint8_t *measurement); >> >> +/** >> + * kvm_sev_dbg_decrypt - decrypt the guest memory for debugging purposes >> + * @src - guest memory address >> + * @dest - host memory address where the decrypted data should be copied >> + * @length - length of memory region >> + * >> + * Returns: 0 on success and dest will contains the decrypted data >> + */ >> +int kvm_sev_dbg_decrypt(uint8_t *dest, const uint8_t *src, uint32_t len); >> + >> #endif >> diff --git a/sev.c b/sev.c >> index 055ed83..c7031d3 100644 >> --- a/sev.c >> +++ b/sev.c >> @@ -432,3 +432,26 @@ int kvm_sev_guest_measurement(uint8_t *out) >> >> return 0; >> } >> + >> +int kvm_sev_dbg_decrypt(uint8_t *dst, const uint8_t *src, uint32_t len) >> +{ >> + int ret; >> + struct kvm_sev_dbg_decrypt decrypt; >> + struct kvm_sev_issue_cmd input; >> + >> + decrypt.src_addr = (unsigned long)src; >> + decrypt.dst_addr = (unsigned long)dst; >> + decrypt.length = len; >> + >> + input.cmd = KVM_SEV_DBG_DECRYPT; >> + input.opaque = (unsigned long)&decrypt; >> + ret = kvm_vm_ioctl(kvm_state, KVM_SEV_ISSUE_CMD, &input); >> + if (ret) { >> + fprintf(stderr, "SEV: dbg_decrypt failed ret=%d(%#010x)\n", >> + ret, input.ret_code); >> + return 1; >> + } >> + >> + DPRINTF("SEV: DBG_DECRYPT dst %p src %p sz %d\n", dst, src, len); >> + return 0; >> +} > >