[edk2-devel] [PATCH v3 0/2] Support MM based variable services in PEI for ARM

Kun Qin posted 2 patches 10 months ago
Failed in applying to current master (apply log)
ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c          | 221 ++++++++++++
MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c   | 381 ++++++++++++++++++++
ArmPkg/ArmPkg.dsc                                               |   2 +
ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf        |  40 ++
MdeModulePkg/MdeModulePkg.dsc                                   |   1 +
MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h   | 134 +++++++
MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf |  40 ++
7 files changed, 819 insertions(+)
create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c
create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c
create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf
create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h
create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf
[edk2-devel] [PATCH v3 0/2] Support MM based variable services in PEI for ARM
Posted by Kun Qin 10 months ago
REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4464

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

The main changes between v2 and v3 patches are:
  - Removed header file for MM communicate PEI module
  - Added constant qualifiers to applicable entries
  - Added reviewed-by tag

The change was verified on QEMU based ARM sbsa platform and proprietary
hardware platform.

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

Cc: Hao A Wu <hao.a.wu@intel.com>
Cc: Liming Gao <gaoliming@byosoft.com.cn>
Cc: Jian J Wang <jian.j.wang@intel.com>
Cc: Leif Lindholm <quic_llindhol@quicinc.com>
Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
Cc: Sami Mujawar <sami.mujawar@arm.com>

Kun Qin (2):
  ArmPkg: MmCommunicationPei: Introduce MM communicate in PEI
  MdeModulePkg: Variable: Introduce MM based variable read service in
    PEI

 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c          | 221 ++++++++++++
 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c   | 381 ++++++++++++++++++++
 ArmPkg/ArmPkg.dsc                                               |   2 +
 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf        |  40 ++
 MdeModulePkg/MdeModulePkg.dsc                                   |   1 +
 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h   | 134 +++++++
 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf |  40 ++
 7 files changed, 819 insertions(+)
 create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c
 create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c
 create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf
 create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h
 create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf

-- 
2.41.0.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106443): https://edk2.groups.io/g/devel/message/106443
Mute This Topic: https://groups.io/mt/99820282/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/2] Support MM based variable services in PEI for ARM
Posted by Ard Biesheuvel 10 months ago
On Wed, 28 Jun 2023 at 01:14, Kun Qin <kuqin12@gmail.com> wrote:
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4464
>
> This patch series is a follow-up of previous submission:
> https://edk2.groups.io/g/devel/message/106372
>
> The main changes between v2 and v3 patches are:
>   - Removed header file for MM communicate PEI module
>   - Added constant qualifiers to applicable entries
>   - Added reviewed-by tag
>
> The change was verified on QEMU based ARM sbsa platform and proprietary
> hardware platform.
>
> Patch v3 branch: https://github.com/kuqin12/edk2/tree/arm_var_pei_v3
>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Leif Lindholm <quic_llindhol@quicinc.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Sami Mujawar <sami.mujawar@arm.com>
>
> Kun Qin (2):
>   ArmPkg: MmCommunicationPei: Introduce MM communicate in PEI
>   MdeModulePkg: Variable: Introduce MM based variable read service in
>     PEI
>

Thanks, I've queued this up now.


>  ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c          | 221 ++++++++++++
>  MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c   | 381 ++++++++++++++++++++
>  ArmPkg/ArmPkg.dsc                                               |   2 +
>  ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf        |  40 ++
>  MdeModulePkg/MdeModulePkg.dsc                                   |   1 +
>  MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h   | 134 +++++++
>  MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf |  40 ++
>  7 files changed, 819 insertions(+)
>  create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c
>  create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c
>  create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf
>  create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h
>  create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf
>
> --
> 2.41.0.windows.1
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#106480): https://edk2.groups.io/g/devel/message/106480
Mute This Topic: https://groups.io/mt/99820282/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/2] Support MM based variable services in PEI for ARM
Posted by Kun Qin 10 months ago
Thank you so much for the quick turnaround!

Regards,
Kun

On 6/28/2023 11:44 AM, Ard Biesheuvel wrote:
> On Wed, 28 Jun 2023 at 01:14, Kun Qin <kuqin12@gmail.com> wrote:
>> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4464
>>
>> This patch series is a follow-up of previous submission:
>> https://edk2.groups.io/g/devel/message/106372
>>
>> The main changes between v2 and v3 patches are:
>>    - Removed header file for MM communicate PEI module
>>    - Added constant qualifiers to applicable entries
>>    - Added reviewed-by tag
>>
>> The change was verified on QEMU based ARM sbsa platform and proprietary
>> hardware platform.
>>
>> Patch v3 branch: https://github.com/kuqin12/edk2/tree/arm_var_pei_v3
>>
>> Cc: Hao A Wu <hao.a.wu@intel.com>
>> Cc: Liming Gao <gaoliming@byosoft.com.cn>
>> Cc: Jian J Wang <jian.j.wang@intel.com>
>> Cc: Leif Lindholm <quic_llindhol@quicinc.com>
>> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
>> Cc: Sami Mujawar <sami.mujawar@arm.com>
>>
>> Kun Qin (2):
>>    ArmPkg: MmCommunicationPei: Introduce MM communicate in PEI
>>    MdeModulePkg: Variable: Introduce MM based variable read service in
>>      PEI
>>
> Thanks, I've queued this up now.
>
>
>>   ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c          | 221 ++++++++++++
>>   MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c   | 381 ++++++++++++++++++++
>>   ArmPkg/ArmPkg.dsc                                               |   2 +
>>   ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf        |  40 ++
>>   MdeModulePkg/MdeModulePkg.dsc                                   |   1 +
>>   MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h   | 134 +++++++
>>   MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf |  40 ++
>>   7 files changed, 819 insertions(+)
>>   create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.c
>>   create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.c
>>   create mode 100644 ArmPkg/Drivers/MmCommunicationPei/MmCommunicationPei.inf
>>   create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.h
>>   create mode 100644 MdeModulePkg/Universal/Variable/MmVariablePei/MmVariablePei.inf
>>
>> --
>> 2.41.0.windows.1
>>


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