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
>
-=-=-=-=-=-=-=-=-=-=-=-
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/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-