[v3] MM communicate functionality in variable policy

[edk2-devel] [PATCH v3 0/6] MM communicate functionality in variable policy

Posted by Kun Qin 2 years, 3 months ago

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751

This patch series is a follow up of previous submission:
https://edk2.groups.io/g/devel/message/85116

All v3 patches added "Reviewed-By" tags collected during v2 series.

Patch v3 branch: https://github.com/kuqin12/edk2/tree/mm_communicate_check_v3

Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Zhiguang Liu <zhiguang.liu@intel.com>
Cc: Leif Lindholm <leif@nuviainc.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
Cc: Michael Kubacki <michael.kubacki@microsoft.com>
Cc: Sami Mujawar <sami.mujawar@arm.com>

Kun Qin (6):
  MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message
    Length
  MdePkg: MmCommunication2: Update MM communicate2 function description
  ArmPkg: MmCommunicationDxe: MM communicate function argument
    attributes
  ArmPkg: MmCommunicationDxe: Update MM communicate `CommBuffer**`
    checks
  ArmPkg: MmCommunicationDxe: Update MM communicate `CommSize` check
  ArmPkg: MmCommunicationDxe: Update MM communicate `MessageLength`
    check

 ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c               | 46 ++++++++++++--------
 MdeModulePkg/Universal/Variable/RuntimeDxe/VariablePolicySmmDxe.c | 10 ++---
 MdePkg/Include/Protocol/MmCommunication2.h                        | 13 +++---
 3 files changed, 41 insertions(+), 28 deletions(-)

-- 
2.34.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86065): https://edk2.groups.io/g/devel/message/86065
Mute This Topic: https://groups.io/mt/88680217/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

[edk2-devel] 回复: [PATCH v3 0/6] MM communicate functionality in variable policy

Posted by gaoliming 2 years, 3 months ago

Hi, all
  If no other comment, I will merge this patch set tomorrow. 

Thanks
Liming
> -----邮件原件-----
> 发件人: Kun Qin <kuqin12@gmail.com>
> 发送时间: 2022年1月26日 3:39
> 收件人: devel@edk2.groups.io
> 抄送: Jian J Wang <jian.j.wang@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Hao A Wu <hao.a.wu@intel.com>; Michael D
> Kinney <michael.d.kinney@intel.com>; Zhiguang Liu
> <zhiguang.liu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> Biesheuvel <ardb+tianocore@kernel.org>; Bret Barkelew
> <Bret.Barkelew@microsoft.com>; Michael Kubacki
> <michael.kubacki@microsoft.com>; Sami Mujawar <sami.mujawar@arm.com>
> 主题: [PATCH v3 0/6] MM communicate functionality in variable policy
> 
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751
> 
> This patch series is a follow up of previous submission:
> https://edk2.groups.io/g/devel/message/85116
> 
> All v3 patches added "Reviewed-By" tags collected during v2 series.
> 
> Patch v3 branch:
> https://github.com/kuqin12/edk2/tree/mm_communicate_check_v3
> 
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> Cc: Michael Kubacki <michael.kubacki@microsoft.com>
> Cc: Sami Mujawar <sami.mujawar@arm.com>
> 
> Kun Qin (6):
>   MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message
>     Length
>   MdePkg: MmCommunication2: Update MM communicate2 function
> description
>   ArmPkg: MmCommunicationDxe: MM communicate function argument
>     attributes
>   ArmPkg: MmCommunicationDxe: Update MM communicate
> `CommBuffer**`
>     checks
>   ArmPkg: MmCommunicationDxe: Update MM communicate `CommSize`
> check
>   ArmPkg: MmCommunicationDxe: Update MM communicate
> `MessageLength`
>     check
> 
>  ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c
> | 46 ++++++++++++--------
>  MdeModulePkg/Universal/Variable/RuntimeDxe/VariablePolicySmmDxe.c |
> 10 ++---
>  MdePkg/Include/Protocol/MmCommunication2.h
> | 13 +++---
>  3 files changed, 41 insertions(+), 28 deletions(-)
> 
> --
> 2.34.1.windows.1





-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86074): https://edk2.groups.io/g/devel/message/86074
Mute This Topic: https://groups.io/mt/88687597/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-

Re: [edk2-devel] 回复: [PATCH v3 0/6] MM communicate functionality in variable policy

Posted by Kun Qin 2 years, 2 months ago

Hi Liming,

Thanks for the help!

Regards,
Kun

On 01/25/2022 17:41, gaoliming wrote:
> Hi, all
>    If no other comment, I will merge this patch set tomorrow.
> 
> Thanks
> Liming
>> -----邮件原件-----
>> 发件人: Kun Qin <kuqin12@gmail.com>
>> 发送时间: 2022年1月26日 3:39
>> 收件人: devel@edk2.groups.io
>> 抄送: Jian J Wang <jian.j.wang@intel.com>; Liming Gao
>> <gaoliming@byosoft.com.cn>; Hao A Wu <hao.a.wu@intel.com>; Michael D
>> Kinney <michael.d.kinney@intel.com>; Zhiguang Liu
>> <zhiguang.liu@intel.com>; Leif Lindholm <leif@nuviainc.com>; Ard
>> Biesheuvel <ardb+tianocore@kernel.org>; Bret Barkelew
>> <Bret.Barkelew@microsoft.com>; Michael Kubacki
>> <michael.kubacki@microsoft.com>; Sami Mujawar <sami.mujawar@arm.com>
>> 主题: [PATCH v3 0/6] MM communicate functionality in variable policy
>>
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3709
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3751
>>
>> This patch series is a follow up of previous submission:
>> https://edk2.groups.io/g/devel/message/85116
>>
>> All v3 patches added "Reviewed-By" tags collected during v2 series.
>>
>> Patch v3 branch:
>> https://github.com/kuqin12/edk2/tree/mm_communicate_check_v3
>>
>> Cc: Jian J Wang <jian.j.wang@intel.com>
>> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>> Cc: Hao A Wu <hao.a.wu@intel.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
>> Cc: Leif Lindholm <leif@nuviainc.com>
>> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
>> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
>> Cc: Michael Kubacki <michael.kubacki@microsoft.com>
>> Cc: Sami Mujawar <sami.mujawar@arm.com>
>>
>> Kun Qin (6):
>>    MdeModulePkg: VariableSmmRuntimeDxe: Fix Variable Policy Message
>>      Length
>>    MdePkg: MmCommunication2: Update MM communicate2 function
>> description
>>    ArmPkg: MmCommunicationDxe: MM communicate function argument
>>      attributes
>>    ArmPkg: MmCommunicationDxe: Update MM communicate
>> `CommBuffer**`
>>      checks
>>    ArmPkg: MmCommunicationDxe: Update MM communicate `CommSize`
>> check
>>    ArmPkg: MmCommunicationDxe: Update MM communicate
>> `MessageLength`
>>      check
>>
>>   ArmPkg/Drivers/MmCommunicationDxe/MmCommunication.c
>> | 46 ++++++++++++--------
>>   MdeModulePkg/Universal/Variable/RuntimeDxe/VariablePolicySmmDxe.c |
>> 10 ++---
>>   MdePkg/Include/Protocol/MmCommunication2.h
>> | 13 +++---
>>   3 files changed, 41 insertions(+), 28 deletions(-)
>>
>> --
>> 2.34.1.windows.1
> 
> 
> 
> 
> 
> 
> 
> 


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#86127): https://edk2.groups.io/g/devel/message/86127
Mute This Topic: https://groups.io/mt/88687597/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-