Re: [edk2-devel] Proposing v3 of MM communicate buffer

2024-08-12 Thread Kun Qin via groups.io
4bit PEI + 64bit DXE + 64bit MM. Does the proposal try to solve some ARM problem? Can you explain the necessity? I would like to avoid the complicated interfaces which do not solve a practical problem. Thanks, Ray ________________ From: devel@edk2.groups.io<mailto:

Re: [edk2-devel] Proposing v3 of MM communicate buffer

2024-08-09 Thread Kun Qin via groups.io
faces which do not solve a practical problem. Thanks, Ray ________________ From: devel@edk2.groups.io<mailto:devel@edk2.groups.io> mailto:devel@edk2.groups.io>> on behalf of Kun Qin via groups.io mailto:Kun.Qin=microsoft@groups.io>> Sent: Thursday, A

Re: [edk2-devel] Proposing v3 of MM communicate buffer

2024-08-08 Thread Kun Qin via groups.io
>> on behalf of Kun Qin via groups.io mailto:Kun.Qin=microsoft@groups.io>> Sent: Thursday, August 8, 2024 2:14 To: devel@edk2.groups.io<mailto:devel@edk2.groups.io> mailto:devel@edk2.groups.io>> Subject: [edk2-devel] Proposing v3 of MM communicate buffer Hi all, I a

Re: [edk2-devel] Proposing v3 of MM communicate buffer

2024-08-07 Thread Kun Qin via groups.io
Adding stakeholders. Regards, Kun From: Kun Qin Sent: Wednesday, August 7, 2024 11:15 AM To: devel@edk2.groups.io Subject: Proposing v3 of MM communicate buffer Hi all, I am trying to propose a change into PI spec and would like to gather some feedback in this forum. Essentially, the current

[edk2-devel] Proposing v3 of MM communicate buffer

2024-08-07 Thread Kun Qin via groups.io
Hi all, I am trying to propose a change into PI spec and would like to gather some feedback in this forum. Essentially, the current communicate header contains a UINTN field in place, which is causing programing errors when trying to communicate the message between different operation mode (i.

Re: [edk2-devel] [PATCH 7/9] MdeModulePkg:Consume gEdkiiVariableRuntimeCacheInfoHobGuid

2024-05-17 Thread Kun Qin via groups.io
Hi Ray & Dun, Thanks for adding me to the patch. I think the proposed solution should work. One question, which is actually on the hob creator patch (https://edk2.groups.io/g/devel/message/119022), is that I understand the hob creation depends on "gEfiPeiMemoryDiscoveredPpiGuid", but does this

Re: 回复: [edk2-devel] [PATCHv3 1/1] MdePkg/UefiDevicePathLib: Add support for PEIMs

2022-07-11 Thread Kun Qin via groups.io
Liming/Mike, I think making MM instance to be a base version makes sense, too. Thanks for the suggestion. Regards, Kun -Original Message- From: gaoliming Sent: Sunday, July 10, 2022 11:49 PM To: devel@edk2.groups.io; Kinney, Michael D ; 'Albecki, Mateusz' Cc: 'Liu, Zhiguang' ; Kun Q

Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg/UnitTestLib: Correct dereferred pointer.

2020-04-13 Thread Kun Qin via groups.io
Hi Guomin, Could you please point me to the proposed change? Thanks, Kun From: Jiang, Guomin Sent: Wednesday, April 8, 2020 6:30 PM To: Sean Brogan ; devel@edk2.groups.io Cc: Kun Qin Subject: [EXTERNAL] RE: [edk2-devel] [PATCH] UnitTestFrameworkPkg/UnitTestLib: Correct dereferred pointer. Hi

Re: [edk2-devel] [PATCH] UnitTestFrameworkPkg/UnitTestLib: Correct dereferred pointer.

2020-04-13 Thread Kun Qin via groups.io
Hi Guomin, Thanks for point the commit below to me. I agree that the change is functionally the same, and I am also with Sean that the local variable seems redundant. But is there other reason we used this local variable in the first place? I thought the first implementation

Re: [edk2-devel] [PATCH] FmpDevicePkg/FmpDxe: Fix uninitialized pointer dereference

2020-03-23 Thread Kun Qin via Groups.Io
Hi Guomin, Thanks for reaching out. I did encounter a GP fault because of this issue: If Line 582 is triggered when the first Fmp->GetImageInfo failed, this specific mFmpImage