From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) by mx.groups.io with SMTP id smtpd.web10.14237.1583950854957041570 for ; Wed, 11 Mar 2020 11:20:55 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@linaro.org header.s=google header.b=c6AskWvW; spf=pass (domain: linaro.org, ip: 209.85.128.42, mailfrom: ard.biesheuvel@linaro.org) Received: by mail-wm1-f42.google.com with SMTP id f7so3220382wml.4 for ; Wed, 11 Mar 2020 11:20:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=DWLFw2RX7DIk5wfzWiCLduUumA6u6NxT47bJ4W8Y2Qg=; b=c6AskWvWrQBNFvXINAvXtLIPJ1FBzUGW0Z7pHoI3mDoBRo1f39OA4Z0UwgFmNxnBP5 5H/88MYS6UTdEupdHvDSu2UokaXBwQHXuVKWKbSKnZhqNKEJdRSNl5/LT9Bkx7UX4yMa 4cyJzrhhc4R/Djw6o45dDms9v9VxFusLWi0ZbN4b7xLjeEmjRWoo7Hke1XVoY0CRF8+C uy49HBpgDFFyA6k2vKv5BuOShDctEkDDjCZpoJnio3GXF+mLlCZAWZzByEq/GMtTrE0v ehs+Yid21cYu1pntFgeVtMchI72xUJWxC5NWChKZQcojdrEMtyIJiEX8MlOghkUWag37 ECUg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=DWLFw2RX7DIk5wfzWiCLduUumA6u6NxT47bJ4W8Y2Qg=; b=VBhZfvVH16Zp2xwcC32kSSL9iMH1jTzgP7BqVs8J7oDXkgPKrFeIBq2g09w1bxJmOG 2m6/zMbTTpnZZ32gTKM+f48JfwilWgx4ibG4YAWmDQh7792Vkz4UrBoCU65UzMfR8bOx 4gwQ80s5Vf6whzUW22wVpwQqukGs48Js1ItzWhEpsLKUuXW5jiyYbPA/65kl3GCgRDWJ wPXBLGG1bAyBXSB2FueRXl7wzWW0rpQiERENLWkv0OEKIuZ639pVOpb17eWxMXphOmVH AJmbra3yz7vm/wLniydZt2ga9STZxUmL46XMvi7ckJi5bkR6XWn+MJqcrlEzZTneU1In JlfA== X-Gm-Message-State: ANhLgQ3H6GsBIgq0hKHGGQeTaNM0K1k5u83Nb4d2rK6CkhN4YwpgCUsv 7Ir2XklY8NrGg9j8bakO5gtRuxCnU291E5sN6Bs6aQ== X-Google-Smtp-Source: ADFU+vsaRedKcKXa7fjrH8Kq/ZMWMu4xBj4BT379O0BoDQcNEDOtK0FOXo2g7IbeC2XM43lMlQ8tK7eGscueAj7awYw= X-Received: by 2002:a1c:2d88:: with SMTP id t130mr59348wmt.68.1583950852570; Wed, 11 Mar 2020 11:20:52 -0700 (PDT) MIME-Version: 1.0 References: <20191206152946.624-1-ard.biesheuvel@arm.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E58A232@SHSMSX104.ccr.corp.intel.com> In-Reply-To: From: "Ard Biesheuvel" Date: Wed, 11 Mar 2020 14:20:40 -0400 Message-ID: Subject: Re: [piwg] Re: [PATCH 0/7] New implementation of MM communicate for standalone MM To: "Doran, Mark" Cc: "Gao, Liming" , Stuart Yoder , "Zimmer, Vincent" , "piwg@uefi.org" , Ard Biesheuvel , edk2-devel-groups-io , "Kinney, Michael D" , Leif Lindholm , "Wu, Hao A" , "Dong, Eric" , "Ni, Ray" , Achin Gupta , "Yao, Jiewen" Content-Type: multipart/alternative; boundary="000000000000e8d97905a0984bdb" --000000000000e8d97905a0984bdb Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable thanks Mark this is about mantis ticket 1993, which is a prerequisite for building anything based on standalone mm that involves a va switch at runtime i don't care deeply about the pace at which things move, as long as they move at all, so I'm happy to settle for whatever is workable for the editorial team On Wed, Mar 11, 2020, 14:12 Doran, Mark wrote: > Apologies for the delay -- I just found this by accident looking at the > PIWG email archive for something else which prompted me to find the note > below in the spam trap for some reason. Anyhow... > > UEFI spec publications and errata are kind of demand-driven - we don't > keep a regular schedule so much as make one for an errata or new spec > version when a contributor requests that they want one to meet a product > deliverable or some equivalent external event that needs to read on an > update. Lazy evaluation if you will. > > Which is to say right now there isn't a PI 1.7A schedule but we can make > one =F0=9F=98=8A > > To be sure we have this right is there one specific PI ECR we're talking > about or more than one for this?? If we can identify which peg we need p= ut > into an errata version to hang these code changes from then I can get wit= h > Vincent and we can have the work group crank this handle. > > That said... > > The one caveat that applies right this second is that the conversion of > the spec sources is in flight. Our tech writer may tell me that there ar= e > some related limitations on when exactly we can practically make an > update. I don't want to pollute the open lists copied here with that UEF= I > Forum internal housekeeping but I did want to mention that as a factor by > way of not wanting to over-promise an instant fix. > > -- > Cheers, > > Mark. > > -----Original Message----- > From: piwg@uefi.org On Behalf Of Ard Biesheuvel > Sent: Friday, March 6, 2020 2:38 AM > To: Gao, Liming ; Stuart Yoder ; > Zimmer, Vincent ; piwg@uefi.org > Cc: Ard Biesheuvel ; edk2-devel-groups-io < > devel@edk2.groups.io>; Kinney, Michael D ; > Leif Lindholm ; Wu, Hao A ; > Dong, Eric ; Ni, Ray ; Achin Gupta > ; Yao, Jiewen > Subject: [piwg] Re: [PATCH 0/7] New implementation of MM communicate for > standalone MM > > (adding PIWG and some other folks to cc) > > On Mon, 6 Jan 2020 at 02:16, Gao, Liming wrote: > > > > Ard: > > The changes are good to me. But, I think this change will not be added > into MdePkg until PI1.7 errata A is published. > > > > Hello all, > > Due to the Huawei situation, and now the fact that there is little > movement in the PIWG, these changes have been in limbo for over a year no= w. > > Is there any way we can progress with this independently? Or push on with > the publication of PI 1.7 errata A ? > > Thanks, > Ard. > > > > > -----Original Message----- > > From: Ard Biesheuvel > > Sent: 2020=E5=B9=B41=E6=9C=883=E6=97=A5 21:03 > > To: Ard Biesheuvel > > Cc: edk2-devel-groups-io ; Kinney, Michael D > > ; Leif Lindholm > > ; Gao, Liming ; Wu, > > Hao A ; Dong, Eric ; Ni, Ray > > ; Achin Gupta ; Yao, Jiewen > > > > Subject: Re: [PATCH 0/7] New implementation of MM communicate for > > standalone MM > > > > On Fri, 6 Dec 2019 at 16:29, Ard Biesheuvel > wrote: > > > > > > From: Ard Biesheuvel > > > > > > This series implements the changes proposed by Mantis ticket 1993, > > > which describes how we need to modify the MM communicate protocol to > > > make it compatible with standalone MM based DXE_RUNTIME_DRIVER MM > > > client using virtually remapped memory. > > > > > > These changes have been accepted as errata for PI spec v1.7 > > > > > > Cc: Michael D Kinney > > > Cc: Leif Lindholm > > > Cc: Liming Gao > > > Cc: Hao A Wu > > > Cc: Eric Dong > > > Cc: Ray Ni > > > Cc: Achin Gupta > > > Cc: Jiewen Yao > > > > > > > Ping? > > > > > > > Ard Biesheuvel (7): > > > MdePkg: introduce MM communicate 2 protocol > > > MdeModulePkg/SmmIpl: expose MM communicate 2 protocol > > > ArmPkg/MmCommunicationDxe: expose MM Communicate 2 protocol > > > MdeModulePkg/VariableInfo: switch to MM communicate 2 protocol > > > MdeModulePkg/FaultTolerantWriteSmmDxe: switch to MM communicate 2 > > > protocol > > > MdeModulePkg/VariableSmmRuntimeDxe: switch to MM communicate 2 > > > protocol > > > StandaloneMmPkg: switch to MM communicate 2 protocol > > > > > > .../MmCommunicationDxe/MmCommunication.c | 79 +++++++++--------= -- > > > .../MmCommunicationDxe/MmCommunication.inf | 6 +- > > > .../Application/VariableInfo/VariableInfo.c | 19 +++-- > > > .../Application/VariableInfo/VariableInfo.inf | 2 +- > > > MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c | 79 +++++++++++++++++= ++ > > > MdeModulePkg/Core/PiSmmCore/PiSmmIpl.inf | 1 + > > > MdeModulePkg/Include/Guid/SmmVariableCommon.h | 6 +- > > > .../FaultTolerantWriteSmmCommon.h | 2 +- > > > .../FaultTolerantWriteSmmDxe.c | 25 +++--- > > > .../FaultTolerantWriteSmmDxe.h | 2 +- > > > .../FaultTolerantWriteSmmDxe.inf | 4 +- > > > .../RuntimeDxe/VariableSmmRuntimeDxe.c | 39 ++++----- > > > .../RuntimeDxe/VariableSmmRuntimeDxe.inf | 4 +- > > > MdePkg/Include/Protocol/MmCommunication2.h | 69 ++++++++++++++++ > > > MdePkg/MdePkg.dec | 7 ++ > > > StandaloneMmPkg/Core/StandaloneMmCore.h | 2 +- > > > .../StandaloneMmCpu/AArch64/StandaloneMmCpu.h | 2 +- > > > 17 files changed, 254 insertions(+), 94 deletions(-) create mode > > > 100644 MdePkg/Include/Protocol/MmCommunication2.h > > > > > > -- > > > 2.17.1 > > > > --000000000000e8d97905a0984bdb Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
thanks Mark

= this is about mantis ticket 1993, which is a prerequisite for building anyt= hing based on standalone mm that involves a va switch at runtime

i don't care deeply about the = pace at which things move, as long as they move at all, so I'm happy to= settle for whatever is workable for the editorial team


On Wed, Mar 11, 2020, 14:12 Doran, Mark <mark.doran@intel.com> wrote:
Apologies for the delay -- I just found this by a= ccident looking at the PIWG email archive for something else which prompted= me to find the note below in the spam trap for some reason.=C2=A0 Anyhow..= .

UEFI spec publications and errata are kind of demand-driven - we don't = keep a regular schedule so much as make one for an errata or new spec versi= on when a contributor requests that they want one to meet a product deliver= able or some equivalent external event that needs to read on an update.=C2= =A0 Lazy evaluation if you will.

Which is to say right now there isn't a PI 1.7A schedule but we can mak= e one =F0=9F=98=8A=C2=A0

To be sure we have this right is there one specific PI ECR we're talkin= g about or more than one for this??=C2=A0 If we can identify which peg we n= eed put into an errata version to hang these code changes from then I can g= et with Vincent and we can have the work group crank this handle.

That said...

The one caveat that applies right this second is that the conversion of the= spec sources is in flight.=C2=A0 Our tech writer may tell me that there ar= e some related limitations on when exactly we can practically make an updat= e.=C2=A0 I don't want to pollute the open lists copied here with that U= EFI Forum internal housekeeping but I did want to mention that as a factor = by way of not wanting to over-promise an instant fix.

--
Cheers,

Mark.

-----Original Message-----
From: piwg@uefi.org <piwg@uefi.org> On Behalf Of Ard Biesheuvel
Sent: Friday, March 6, 2020 2:38 AM
To: Gao, Liming <liming.gao@intel.com>; Stuart Yoder <Stua= rt.Yoder@arm.com>; Zimmer, Vincent <vincent.zimmer@intel.co= m>; piwg@uefi.org
Cc: Ard Biesheuvel <ard.biesheuvel@arm.com>; edk2-devel-group= s-io <devel@edk2.groups.io>; Kinney, Michael D <mich= ael.d.kinney@intel.com>; Leif Lindholm <leif.lindholm@linar= o.org>; Wu, Hao A <hao.a.wu@intel.com>; Dong, Eric <<= a href=3D"mailto:eric.dong@intel.com" target=3D"_blank" rel=3D"noreferrer">= eric.dong@intel.com>; Ni, Ray <ray.ni@intel.com>; Achin Gupt= a <achin.gupta@arm.com>; Yao, Jiewen <jiewen.yao@intel.com<= /a>>
Subject: [piwg] Re: [PATCH 0/7] New implementation of MM communicate for st= andalone MM

(adding PIWG and some other folks to cc)

On Mon, 6 Jan 2020 at 02:16, Gao, Liming <liming.gao@intel.com>= wrote:
>
> Ard:
>=C2=A0 The changes are good to me. But, I think this change will not be= added into MdePkg until PI1.7 errata A is published.
>

Hello all,

Due to the Huawei situation, and now the fact that there is little movement= in the PIWG, these changes have been in limbo for over a year now.

Is there any way we can progress with this independently? Or push on with t= he publication of PI 1.7 errata A ?

Thanks,
Ard.



> -----Original Message-----
> From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Sent: 2020=E5=B9=B41=E6=9C=883=E6=97=A5 21:03
> To: Ard Biesheuvel <ard.biesheuvel@arm.com>
> Cc: edk2-devel-groups-io <devel@edk2.groups.io>; Kinney, M= ichael D
> <michael.d.kinney@intel.com>; Leif Lindholm
> <leif.lindholm@linaro.org>; Gao, Liming <limin= g.gao@intel.com>; Wu,
> Hao A <hao.a.wu@intel.com>; Dong, Eric <eric.dong@intel= .com>; Ni, Ray
> <ray.ni@intel.com>; Achin Gupta <achin.gupta@arm.com&g= t;; Yao, Jiewen
> <jiewen.yao@intel.com>
> Subject: Re: [PATCH 0/7] New implementation of MM communicate for
> standalone MM
>
> On Fri, 6 Dec 2019 at 16:29, Ard Biesheuvel <ard.biesheuvel@arm= .com> wrote:
> >
> > From: Ard Biesheuvel <ard.biesheuvel@linaro.org>=
> >
> > This series implements the changes proposed by Mantis ticket 1993= ,
> > which describes how we need to modify the MM communicate protocol= to
> > make it compatible with standalone MM based DXE_RUNTIME_DRIVER MM=
> > client using virtually remapped memory.
> >
> > These changes have been accepted as errata for PI spec v1.7
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com&g= t;
> > Cc: Leif Lindholm <leif.lindholm@linaro.org>
> > Cc: Liming Gao <liming.gao@intel.com>
> > Cc: Hao A Wu <hao.a.wu@intel.com>
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Achin Gupta <achin.gupta@arm.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> >
>
> Ping?
>
>
> > Ard Biesheuvel (7):
> >=C2=A0 =C2=A0MdePkg: introduce MM communicate 2 protocol
> >=C2=A0 =C2=A0MdeModulePkg/SmmIpl: expose MM communicate 2 protocol=
> >=C2=A0 =C2=A0ArmPkg/MmCommunicationDxe: expose MM Communicate 2 pr= otocol
> >=C2=A0 =C2=A0MdeModulePkg/VariableInfo: switch to MM communicate 2= protocol
> >=C2=A0 =C2=A0MdeModulePkg/FaultTolerantWriteSmmDxe: switch to MM c= ommunicate 2
> >=C2=A0 =C2=A0 =C2=A0protocol
> >=C2=A0 =C2=A0MdeModulePkg/VariableSmmRuntimeDxe: switch to MM comm= unicate 2
> >=C2=A0 =C2=A0 =C2=A0protocol
> >=C2=A0 =C2=A0StandaloneMmPkg: switch to MM communicate 2 protocol<= br> > >
> >=C2=A0 .../MmCommunicationDxe/MmCommunication.c=C2=A0 =C2=A0 =C2= =A0 | 79 +++++++++----------
> >=C2=A0 .../MmCommunicationDxe/MmCommunication.inf=C2=A0 =C2=A0 |= =C2=A0 6 +-
> >=C2=A0 .../Application/VariableInfo/VariableInfo.c=C2=A0 =C2=A0| 1= 9 +++--
> >=C2=A0 .../Application/VariableInfo/VariableInfo.inf |=C2=A0 2 +-<= br> > >=C2=A0 MdeModulePkg/Core/PiSmmCore/PiSmmIpl.c=C2=A0 =C2=A0 =C2=A0 = =C2=A0 | 79 +++++++++++++++++++
> >=C2=A0 MdeModulePkg/Core/PiSmmCore/PiSmmIpl.inf=C2=A0 =C2=A0 =C2= =A0 |=C2=A0 1 +
> >=C2=A0 MdeModulePkg/Include/Guid/SmmVariableCommon.h |=C2=A0 6 +-<= br> > >=C2=A0 .../FaultTolerantWriteSmmCommon.h=C2=A0 =C2=A0 =C2=A0 =C2= =A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 2 +-
> >=C2=A0 .../FaultTolerantWriteSmmDxe.c=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 | 25 +++---
> >=C2=A0 .../FaultTolerantWriteSmmDxe.h=C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 |=C2=A0 2 +-
> >=C2=A0 .../FaultTolerantWriteSmmDxe.inf=C2=A0 =C2=A0 =C2=A0 =C2=A0= =C2=A0 =C2=A0 =C2=A0 |=C2=A0 4 +-
> >=C2=A0 .../RuntimeDxe/VariableSmmRuntimeDxe.c=C2=A0 =C2=A0 =C2=A0 = =C2=A0 | 39 ++++-----
> >=C2=A0 .../RuntimeDxe/VariableSmmRuntimeDxe.inf=C2=A0 =C2=A0 =C2= =A0 |=C2=A0 4 +-
> >=C2=A0 MdePkg/Include/Protocol/MmCommunication2.h=C2=A0 =C2=A0 | 6= 9 ++++++++++++++++
> >=C2=A0 MdePkg/MdePkg.dec=C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 = =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0 =C2=A0|=C2=A0 7 ++<= br> > >=C2=A0 StandaloneMmPkg/Core/StandaloneMmCore.h=C2=A0 =C2=A0 =C2=A0= =C2=A0|=C2=A0 2 +-
> >=C2=A0 .../StandaloneMmCpu/AArch64/StandaloneMmCpu.h |=C2=A0 2 +-<= br> > >=C2=A0 17 files changed, 254 insertions(+), 94 deletions(-)=C2=A0 = create mode
> > 100644 MdePkg/Include/Protocol/MmCommunication2.h
> >
> > --
> > 2.17.1
> >
--000000000000e8d97905a0984bdb--