From mboxrd@z Thu Jan  1 00:00:00 1970
Subject: Re: [edk2-devel] [PATCH v1 0/1] Introduce DxeMmUnblockMemoryLib Interface
To: Wu, Hao A <hao.a.wu@intel.com>,devel@edk2.groups.io
From: "Kun Qin" <kun.q@outlook.com>
X-Originating-Location: Bothell, Washington, US (50.35.88.161)
X-Originating-Platform: Windows Chrome 88
User-Agent: GROUPS.IO Web Poster
MIME-Version: 1.0
Date: Thu, 04 Feb 2021 19:10:56 -0800
References: <BN8PR11MB3666BD13671E72A63364214ACAB29@BN8PR11MB3666.namprd11.prod.outlook.com>
In-Reply-To: <BN8PR11MB3666BD13671E72A63364214ACAB29@BN8PR11MB3666.namprd11.prod.outlook.com>
Message-ID: <32563.1612494656588537457@groups.io>
Content-Type: multipart/alternative; boundary="ZVGIBb6ey0qdi4wCO8XY"

--ZVGIBb6ey0qdi4wCO8XY
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi Hao,

As mentioned in the cover letter and in BZ-3168, VariableStandaloneMm and =
Tcg2Smm would need this capability to unblock certain regions in order to a=
ccess either variable runtime cache or NVS region patched into ACPI table, =
if non-MMRAM region is blocked for access.

Just as a preview (error handling needs better polishing), the example usa=
ges for the 2 drivers above could be found here:
1. Variable DXE SMM change that unblocks runtime cache regions is posted h=
ere: https://github.com/kuqin12/mu_basecore/commit/189f90318d1256c2e72a1a67=
d31e3176588d8e5b
2. Tcg2Smm change example, which breaks one module into 2 and then unblock=
 NVS region, is posted here: https://github.com/kuqin12/mu_basecore/commit/=
c0784f95ffe31d4d65f6230657adfc63294a693d

The change should be relatively minor but critical for the standalone Mm m=
odel.

Thanks,
Kun

--ZVGIBb6ey0qdi4wCO8XY
Content-Type: text/html; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

Hi Hao,<br /><br />As mentioned in the cover letter and in BZ-3168, Variabl=
eStandaloneMm and Tcg2Smm would need this capability to unblock certain reg=
ions in order to access either variable runtime cache or NVS region patched=
 into ACPI table, if non-MMRAM region is blocked for access.<br /><br />Jus=
t as a preview (error handling needs better polishing), the example usages =
for the 2 drivers above could be found here:<br />
<div style=3D"line-height: 1.42857;">1. Variable DXE SMM change that unblo=
cks runtime cache regions is posted here: <a href=3D"https://github.com/kuq=
in12/mu_basecore/commit/189f90318d1256c2e72a1a67d31e3176588d8e5b" target=3D=
"_blank" rel=3D"noopener">https://github.com/kuqin12/mu_basecore/commit/189=
f90318d1256c2e72a1a67d31e3176588d8e5b</a></div>
<div>2. Tcg2Smm change example, which breaks one module into 2 and then un=
block NVS region, is posted here: <a href=3D"https://github.com/kuqin12/mu_=
basecore/commit/c0784f95ffe31d4d65f6230657adfc63294a693d" target=3D"_blank"=
 rel=3D"noopener">https://github.com/kuqin12/mu_basecore/commit/c0784f95ffe=
31d4d65f6230657adfc63294a693d</a></div>
<div><br />The change should be relatively minor but critical for the stan=
dalone Mm model.<br /><br />Thanks,<br />Kun</div>

--ZVGIBb6ey0qdi4wCO8XY--