On Mon, Mar 02, 2020 at 14:00:54 +0100, Ard Biesheuvel wrote:
> On Mon, 2 Mar 2020 at 13:57, Leif Lindholm <l...@nuviainc.com> wrote:
> >
> > On Wed, Feb 26, 2020 at 13:41:03 +0100, Ard Biesheuvel wrote:
> > > Commit 2fe25a74d6fee3c2 ("ArmPkg/MmCommunicationDxe: relay architected PI
> > > events to MM context") update the ARM specific standalone MM client
> > > driver to register for certain events in the entrypoint code, but did
> > > so in a way that makes the entrypoint always return with an error.
> > >
> > > Instead, return EFI_SUCCESS if registering for those events succeeds,
> > > and back out the registrations that did succeed if one fails, and
> > > return an error.
> > >
> > > Fixes: 2fe25a74d6fee3c2 ("ArmPkg/MmCommunicationDxe: relay architected PI
> > > events to MM context")
> > > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org>
> >
> > Reviewed-by: Leif Lindholm <l...@nuviainc.com>
> >
>
> Thanks.
>
> > Are you looking for this one to make the stable tag? You're not
> > explicitly asking for it, but you're cc:ing Liming.
> >
>
> I don't think it is terribly useful to get this into the stable tag,
> given that the standalone MM support is not complete yet, and stalled
> on a PI spec update.
> But by the same reasoning, it could go in as well, since it is an
> obvious bugfix, and the risk is low.
>
> I don't really care either way tbh, I just wanted to make Liming aware of it.
Thanks for explanation.
I'd say leave it then - if tag gets pushed Wednesday as I think the
current plan is, you can push it then.
/
Leif
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#55180): https://edk2.groups.io/g/devel/message/55180
Mute This Topic: https://groups.io/mt/71564397/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
