On 8/9/20 9:41 PM, Gao, Liming wrote: > > > -----Original Message----- > From: Laszlo Ersek <[email protected]> > Sent: 2020年8月6日 23:39 > To: Tom Lendacky <[email protected]>; [email protected] > Cc: Brijesh Singh <[email protected]>; Ard Biesheuvel > <[email protected]>; Dong, Eric <[email protected]>; Justen, Jordan L > <[email protected]>; Gao, Liming <[email protected]>; Kinney, > Michael D <[email protected]>; Ni, Ray <[email protected]>; Andrew > Fish <[email protected]>; Anthony Perard <[email protected]>; You, > Benjamin <[email protected]>; Bi, Dandan <[email protected]>; Dong, > Guo <[email protected]>; Wu, Hao A <[email protected]>; Wang, Jian J > <[email protected]>; Julien Grall <[email protected]>; Leif Lindholm > <[email protected]>; Ma, Maurice <[email protected]> > 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: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
