[edk2-devel] [PATCH v2 1/1] OvmfPkg/PlatformPei: drop S3Verification()

[Ard Biesheuvel]

On Wed, 24 May 2023 at 14:29, Ard Biesheuvel <ardb at kernel.org> wrote:
>
> On Wed, 24 May 2023 at 14:28, Laszlo Ersek <lersek at redhat.com> wrote:
> >
> > On 5/24/23 11:11, Gerd Hoffmann wrote:
> > > Not needed any more, SMM + 64-bit PEI + S3 suspend works now.  While
> > > being at it also remove it from Bhyve (where it is dead code).
> > >
> > > Fixed by commits:
> > >  - 8bd2028f9ac3 ("MdeModulePkg: Supporting S3 in 64bit PEI")
> > >  - 6acf72901a2e ("UefiCpuPkg: Supporting S3 in 64bit PEI")
> > > See also https://bugzilla.tianocore.org/show_bug.cgi?id=4195
> > >
> > > Signed-off-by: Gerd Hoffmann <kraxel at redhat.com>
> > > Reviewed-by: Ard Biesheuvel <ardb at kernel.org>
> > > Reviewed-by: Laszlo Ersek <lersek at redhat.com>
> > > Reviewed-by: Ray Ni <ray.ni at intel.com>
> > > ---
> > >  OvmfPkg/Bhyve/PlatformPei/Platform.c | 29 --------------------------
> > >  OvmfPkg/PlatformPei/Platform.c       | 31 ----------------------------
> > >  2 files changed, 60 deletions(-)
> >
> > I disagree with this (v2) update. Bhyve platform code and QEMU platform
> > code should not be fused into a common patch, if there's a way to avoid
> > that. The reviewers for these C files are also different. If we want to
> > modify Bhyve as a courtesy, we can certainly propose a *separate* patch
> > for that.
> >
> > If others approve this patch, I can't block it from going in; since I've
> > been CC'd (and I thought I'd comment this time), this is my view on it.
> >
>
> Thanks Laszlo. I'll split them out when applying.

Merged as #4441


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#105376): https://edk2.groups.io/g/devel/message/105376
Mute This Topic: https://groups.io/mt/99104839/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-