From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242])
 by mx.groups.io with SMTP id smtpd.web11.3986.1645074836621624457
 for <devel@edk2.groups.io>;
 Wed, 16 Feb 2022 21:13:58 -0800
Authentication-Results: mx.groups.io;
 dkim=missing; spf=none, err=permanent DNS error (domain: byosoft.com.cn, ip: 58.240.74.242, mailfrom: gaoliming@byosoft.com.cn)
Received: from DESKTOPS6D0PVI ([58.246.60.130])
	(envelope-sender <gaoliming@byosoft.com.cn>)
	by 192.168.6.13 with ESMTP
	for <ardb+tianocore@kernel.org>; Thu, 17 Feb 2022 13:13:45 +0800
X-WM-Sender: gaoliming@byosoft.com.cn
X-Originating-IP: 58.246.60.130
X-WM-AuthFlag: YES
X-WM-AuthUser: gaoliming@byosoft.com.cn
From: "gaoliming" <gaoliming@byosoft.com.cn>
To: <devel@edk2.groups.io>,
	<brijesh.singh@amd.com>,
	"'Gerd Hoffmann'" <kraxel@redhat.com>
Cc: "'Jordan Justen'" <jordan.l.justen@intel.com>,
	"'James Bottomley'" <jejb@linux.ibm.com>,
	"'Pawel Polawski'" <ppolawsk@redhat.com>,
	"'Erdem Aktas'" <erdemaktas@google.com>,
	"'Ard Biesheuvel'" <ardb+tianocore@kernel.org>,
	"'Tom Lendacky'" <thomas.lendacky@amd.com>,
	"'Min Xu'" <min.m.xu@intel.com>,
	"'Jiewen Yao'" <jiewen.yao@intel.com>
References: <20220216070057.1307716-1-kraxel@redhat.com> <eab0054d-7cfd-3a72-b397-48f19801c8e9@amd.com>
In-Reply-To: <eab0054d-7cfd-3a72-b397-48f19801c8e9@amd.com>
Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIDEvMV0gT3ZtZlBrZy9BbWRTZXY6IHJlc2VydmUgc25wIHBhZ2Vz?=
Date: Thu, 17 Feb 2022 13:13:48 +0800
Message-ID: <024401d823bd$255a84f0$700f8ed0$@byosoft.com.cn>
MIME-Version: 1.0
X-Mailer: Microsoft Outlook 16.0
Thread-Index: AQJ6EDkln5nNDzrRud+OiPdZt6ifxgE6wuDTq0mDBdA=
Content-Type: text/plain;
	charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Content-Language: zh-cn

Gerd and Brijesh:
 I see six FDF files include AmdSevDxe. But, only CloudHvX64 and OvmfPkgX64=
 adds reserve snp pages. So, I want to confirm whether others require reser=
ve snp pages.=20

AmdSev\AmdSevX64.fdf
Bhyve\BhyveX64.fdf
CloudHv\CloudHvX64.fdf
OvmfPkgIa32X64.fdf
OvmfPkgX64.fdf
OvmfXen.fdf

Thanks
Liming
> -----=E9=82=AE=E4=BB=B6=E5=8E=9F=E4=BB=B6-----
> =E5=8F=91=E4=BB=B6=E4=BA=BA: devel@edk2.groups.io <devel@edk2.groups.io> =
=E4=BB=A3=E8=A1=A8 Brijesh Singh
> via groups.io
> =E5=8F=91=E9=80=81=E6=97=B6=E9=97=B4: 2022=E5=B9=B42=E6=9C=8817=E6=97=A5 =
0:28
> =E6=94=B6=E4=BB=B6=E4=BA=BA: Gerd Hoffmann <kraxel@redhat.com>; devel@edk=
2.groups.io
> =E6=8A=84=E9=80=81: brijesh.singh@amd.com; Jordan Justen <jordan.l.justen=
@intel.com>;
> James Bottomley <jejb@linux.ibm.com>; Pawel Polawski
> <ppolawsk@redhat.com>; Erdem Aktas <erdemaktas@google.com>; Ard
> Biesheuvel <ardb+tianocore@kernel.org>; Tom Lendacky
> <thomas.lendacky@amd.com>; Min Xu <min.m.xu@intel.com>; Jiewen Yao
> <jiewen.yao@intel.com>
> =E4=B8=BB=E9=A2=98: Re: [edk2-devel] [PATCH 1/1] OvmfPkg/AmdSev: reserve =
snp pages
>=20
>=20
>=20
> On 2/16/22 01:00, Gerd Hoffmann wrote:
> > The SNP patch series updated the OvmfPkgX64 build but forgot the AmdSev
> > variant, resulting in a broken OvmfSevMetadata table.
> >
> > Fixes: cca9cd3dd6bf ("OvmfPkg: reserve CPUID page")
> > Fixes: 707c71a01b9d ("OvmfPkg: reserve SNP secrets page")
> > Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
>=20
> Thanks Gerd
>=20
> Reviewed-by: Brijesh Singh <brijesh.singh@amd.com>
>=20
>=20
>=20
>=20
>=20