On 04/26/16 03:50, Yao, Jiewen wrote:
> Comments below:
> 
>> -----Original Message-----
>> From: Laszlo Ersek [mailto:[email protected]]
>> Sent: Tuesday, April 26, 2016 12:32 AM
>> To: Yao, Jiewen <[email protected]>
>> Cc: [email protected]; Gao, Liming <[email protected]>; Tian,
>> Feng <[email protected]>; Zeng, Star <[email protected]>; Dong, Eric
>> <[email protected]>; Justen, Jordan L <[email protected]>
>> Subject: Re: [PATCH 00/12] Enhance SMM Communication by using fixed
>> comm buffer.
>>
>> On 04/25/16 17:17, Yao, Jiewen wrote:

>>> I will work out a branch in public repo tomorrow.
>>
>> Thank you; I plan to test it, and follow up here.
> 
> [Jiewen] Thanks great help from "Wu, Hao A" <[email protected]>, I created a 
> branch.
> You can get it from https://github.com/jyao1/edk2/tree/SmmComm-jyao1
> 
> I made 3 update based on feedback so far.
> 1) Patch 09/12. The previous commit message is wrong and is fixed now. No 
> real code change. Only commit log change.
> 2) Patch 02/12. I rename the driver name from SmmCommunicationBuffer to 
> SmmCommunicationBufferDxe. It follows EDKII driver naming style. No real code 
> change. Only file name change.
> 3) Patch 03/12. I fixed UNI file to UTF8 format. (The tool is great). No 
> content change. Only format change.

For patch #4 (subject "MdeModulePkg-MemoryProfile(2): Add
SMRAM_PROFILE_COMMAND_GET_PROFILE_DATA_BY_OFFSET in PiSmmCore.",
currently commit 926bbe31676c on your branch):

Regression-tested-by: Laszlo Ersek <[email protected]>

For patch #12 (subject "MdePkg-SmmMemLib: Enhance
SmmIsBufferOutsideSmmValid() check for fixed comm buffer.", currently
commit 4dd8354a7e18 on your branch):

Regression-tested-by: Laszlo Ersek <[email protected]>

(I tested normal boot and S3 suspend-resume on the Q35 machine type,
using Ia32/Fedora, Ia32X64/Fedora, and Ia32X64/Windows-8.1.)

Thanks!
Laszlo
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to