From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-124.mimecast.com (us-smtp-delivery-124.mimecast.com [216.205.24.124]) by mx.groups.io with SMTP id smtpd.web11.8029.1608288602120312820 for ; Fri, 18 Dec 2020 02:50:02 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=QgwgcoRE; spf=pass (domain: redhat.com, ip: 216.205.24.124, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1608288601; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=Ie2qmjdmxHBC+CxWPlWo3ZH4wa9wuLGOMRUHMbhW5bA=; b=QgwgcoREKzBWjX2+ogjcEjva+UwiD0MxKJsLmr25Jq08b3JZZr8YpR2vu3RmKwkz3RSG8O e1ALl/fgvNac7rpldVqVk3FGBbZzHySKTkgGuRlCEYbN4EQPToU7naGztBOK8Qlcj00LNl 6wIvp0E0+A3/eyNpASebGtb2O09UmYo= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-295-5Ky3JHvAONutjYz3FkCDjA-1; Fri, 18 Dec 2020 05:49:56 -0500 X-MC-Unique: 5Ky3JHvAONutjYz3FkCDjA-1 Received: from smtp.corp.redhat.com (int-mx02.intmail.prod.int.phx2.redhat.com [10.5.11.12]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id 706CC800D62; Fri, 18 Dec 2020 10:49:54 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-114-229.ams2.redhat.com [10.36.114.229]) by smtp.corp.redhat.com (Postfix) with ESMTP id 3CDA560C43; Fri, 18 Dec 2020 10:49:51 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH 0/2] Update SevSecret API to work for TDX To: devel@edk2.groups.io, jejb@linux.ibm.com Cc: dovmurik@linux.vnet.ibm.com, Dov.Murik1@il.ibm.com, ashish.kalra@amd.com, brijesh.singh@amd.com, tobin@ibm.com, david.kaplan@amd.com, jon.grimm@amd.com, thomas.lendacky@amd.com, frankeh@us.ibm.com, "Dr . David Alan Gilbert" , Jordan Justen , Ard Biesheuvel , "Yao, Jiewen" References: <20201216014146.2229-1-jejb@linux.ibm.com> From: "Laszlo Ersek" Message-ID: <67c62cdc-2362-b480-f476-506c66a2438e@redhat.com> Date: Fri, 18 Dec 2020 11:49:50 +0100 MIME-Version: 1.0 In-Reply-To: <20201216014146.2229-1-jejb@linux.ibm.com> X-Scanned-By: MIMEDefang 2.79 on 10.5.11.12 Authentication-Results: relay.mimecast.com; auth=pass smtp.auth=CUSA124A263 smtp.mailfrom=lersek@redhat.com X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 12/16/20 02:41, James Bottomley wrote: > This patch series changes the EFI configuration table information > which is queried by the bootloader to make it more compatible with > Intel TDX. The first patch changes the ABI to make the table contain > two 64 bit integers instead of two 32 bit ones. The second patch is a > cosmetic one to change the names of the GUIDs and tables to have a > confidential computing prefix instead of a SEV Launch one. > > The first patch *must* be applied before the next stable tag to avoid > ABI breakage. The second is purely cosmetic and doesn't change the > code output. > > Ultimately there will still need to be a TDX collector for the secret, > which would feed the value into the SecretDxe, but these changes > should ensure that no further changes would be required by the secret > consumers. > > James > > --- > > James Bottomley (2): > OvmfPkg: Change SEV Launch Secret API to be UINT64 for base and size > OvmfPkg/AmdSev/SecretDxe: make secret location naming generic > > OvmfPkg/OvmfPkg.dec | 2 +- > OvmfPkg/AmdSev/SecretDxe/SecretDxe.inf | 2 +- > ...aunchSecret.h => ConfidentialComputingSecret.h} | 14 +++++++------- > OvmfPkg/AmdSev/SecretDxe/SecretDxe.c | 6 +++--- > 4 files changed, 12 insertions(+), 12 deletions(-) > rename OvmfPkg/Include/Guid/{SevLaunchSecret.h => ConfidentialComputingSecret.h} (65%) > Merged as commit range c487970ac89d..96201ae7bf97, via . Thanks Laszlo