On Tue, 31 Oct 2023 at 13:30, Laszlo Ersek <ler...@redhat.com> wrote: > > On 10/30/23 15:36, Ard Biesheuvel wrote: > > On Fri, 27 Oct 2023 at 07:43, duntan <dun....@intel.com> wrote: > >> > >> Use BaseIoLibIntrinsic.inf in dsc files. The > >> BaseIoLibIntrinsic supports Tdx and sev now. > >> The BaseIoLibIntrinsicSev will be removed in > >> the following patches. > >> > >> Signed-off-by: Dun Tan <dun....@intel.com> > >> Cc: Ard Biesheuvel <ardb+tianoc...@kernel.org> > >> Cc: Jiewen Yao <jiewen....@intel.com> > >> Cc: Jordan Justen <jordan.l.jus...@intel.com> > >> Cc: Gerd Hoffmann <kra...@redhat.com> > >> Cc: Ray Ni <ray...@intel.com> > > > > Reviewed-by: Ard Biesheuvel <a...@kernel.org> > > While I agree that this patch *in itself* is (quite obviously) formally > correct, semantically it is wrong for OVMF; which is why I have NACKed > the whole series; see <https://edk2.groups.io/g/devel/message/110242>. > > (Just highlighting it here too, lest it get lost in the review thread.) >
Apologies, I missed that. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#110410): https://edk2.groups.io/g/devel/message/110410 Mute This Topic: https://groups.io/mt/102215668/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
