Re: [edk2-devel] [PATCH v13 00/46] SEV-ES guest support

["Lendacky, Thomas" <thomas.lendacky@amd.com>]

On 8/9/20 9:41 PM, Gao, Liming wrote:
> 
> 
> -----Original Message-----
> From: Laszlo Ersek <lersek@redhat.com> 
> Sent: 2020年8月6日 23:39
> To: Tom Lendacky <thomas.lendacky@amd.com>; devel@edk2.groups.io
> Cc: Brijesh Singh <brijesh.singh@amd.com>; Ard Biesheuvel <ard.biesheuvel@arm.com>; Dong, Eric <eric.dong@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>; Gao, Liming <liming.gao@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Ni, Ray <ray.ni@intel.com>; Andrew Fish <afish@apple.com>; Anthony Perard <anthony.perard@citrix.com>; You, Benjamin <benjamin.you@intel.com>; Bi, Dandan <dandan.bi@intel.com>; Dong, Guo <guo.dong@intel.com>; Wu, Hao A <hao.a.wu@intel.com>; Wang, Jian J <jian.j.wang@intel.com>; Julien Grall <julien@xen.org>; Leif Lindholm <leif@nuviainc.com>; Ma, Maurice <maurice.ma@intel.com>
> Subject: Re: [edk2-devel] [PATCH v13 00/46] SEV-ES guest support
> 
> On 08/06/20 17:12, Tom Lendacky wrote:
>> It looks like all the patches will have a Reviewed-by: or Acked-by: 
>> tag now. I just need to submit another version with the fix to address 
>> the
>> IA32 issue that Laszlo found. After that, assuming it is looking good 
>> to include this series, there is still the question of the 
>> edk2-platform changes that are needed because of the library prereqs 
>> introduced by this series.
>>
>> I've updated the appropriate .dsc files in the edk2-platform tree 
>> locally and successfully test built some of the configs in the Intel 
>> sub-directory (KabylakeRvp3, WhiskeylakeURvp, CometlakeURvp).
>>
>> How would we approach applying this series and providing the updates 
>> to the edk2-platform at the same time?
> 
> I don't have much experience with edk2-platforms. If I recall correctly, the edk2 series is supposed to be merged in two parts, between which edk2-platforms is supposed to be updated (meaning various edk2-platform patches and an edk2 submodule update). In other words, if there is a way to lay out the edk2 and edk2-platforms patches as one grand series, then that order should be followed with the merges.
> 
> In your v14 blurb, you can perhaps identify this split point (if there is one), plus the edk2-platforms series that should be merged there, in the middle.
> 
> Thanks
> Laszlo
> 
> [Liming] First, please send the patches to edk2-platform tree, and make sure they pass review also. The merge process is just Laszlo propose. The first 10 patches in Edk2 will be merged first, then edk2-platforms changes will be merged, last the remaining changes in Edk2 will be merged. After your patches are ready for merge, I can help merge them in edk2 and edk2-platforms.

I'll send the edk2-platforms patch for review as a reply to patch 10 of
the series (the split point).

Thanks,
Tom

> 
> Thanks
> Liming
>