On 8/9/20 9:41 PM, Gao, Liming wrote:
> 
> 
> -----Original Message-----
> From: Laszlo Ersek <ler...@redhat.com> 
> Sent: 2020年8月6日 23:39
> To: Tom Lendacky <thomas.lenda...@amd.com>; devel@edk2.groups.io
> Cc: Brijesh Singh <brijesh.si...@amd.com>; Ard Biesheuvel 
> <ard.biesheu...@arm.com>; Dong, Eric <eric.d...@intel.com>; Justen, Jordan L 
> <jordan.l.jus...@intel.com>; Gao, Liming <liming....@intel.com>; Kinney, 
> Michael D <michael.d.kin...@intel.com>; Ni, Ray <ray...@intel.com>; Andrew 
> Fish <af...@apple.com>; Anthony Perard <anthony.per...@citrix.com>; You, 
> Benjamin <benjamin....@intel.com>; Bi, Dandan <dandan...@intel.com>; Dong, 
> Guo <guo.d...@intel.com>; Wu, Hao A <hao.a...@intel.com>; Wang, Jian J 
> <jian.j.w...@intel.com>; Julien Grall <jul...@xen.org>; Leif Lindholm 
> <l...@nuviainc.com>; Ma, Maurice <maurice...@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
> 

-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#63912): https://edk2.groups.io/g/devel/message/63912
Mute This Topic: https://groups.io/mt/75892660/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to