* Re: [edk2-devel] [patch V2 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
[not found] <166E9BC61BB3701A.10450@groups.io>
@ 2021-03-23 2:05 ` Dandan Bi
2021-03-23 8:39 ` 回复: " gaoliming
0 siblings, 1 reply; 3+ messages in thread
From: Dandan Bi @ 2021-03-23 2:05 UTC (permalink / raw)
To: devel@edk2.groups.io, Bi, Dandan
Cc: Andrew Fish, Leif Lindholm, Laszlo Ersek, Kinney, Michael D,
Liming Gao, Liu, Zhiguang
Hi All,
Here is the branch for these changes in Edk2.
https://github.com/dandanbi/edk2/tree/RegisterFilterLibV2
https://github.com/tianocore/edk2/pull/1509
Hi Mike, Liming and Zhiguang,
Could you help review the first and last 2 patches in MdePkg?
Since we should submit the first 2 patches firstly before the changes of dsc files .
Thanks,
Dandan
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan
> Bi
> Sent: Monday, March 22, 2021 4:09 PM
> To: devel@edk2.groups.io
> Cc: Andrew Fish <afish@apple.com>; Leif Lindholm <leif@nuviainc.com>;
> Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> Liu, Zhiguang <zhiguang.liu@intel.com>
> Subject: [edk2-devel] [patch V2 00/29] Add a new library class
> RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
> RFC: https://edk2.groups.io/g/devel/message/72530
>
> Patch 1 is to add RegisterFilterLib Library Class in edk2 to filter/trace port
> IO/MMIO/MSR access and add a RegisterFilterLibNull instance.
> Patch 2 is to add the MdeLibs.dsc.inc file to MdePkg for some default libraries
> provided by MdePkg and add RegisterFilterLib into it as the first version of
> MdeLibs.dsc.inc.
> Last 2 patches are to update APIs in IoLib and BaseLib to filter/trace port
> IO/MMIO/MSR access.
> Remaining patches are to update related dsc files to consume
> MdeLibs.dsc.inc for RegisterFilterLib.
>
> Will submit patch 1 and 2 firstly.
> And then update related dsc files in edk2 and edk2platform repo to consume
> MdeLibs.dsc.inc for RegisterFilterLib.
> At last will submit the patches to update IoLib and BaseLib to filter/trace port
> IO/MMIO/MSR access.
>
> --
> V2:
> Introduce MdeLibs.dsc.inc and add RegisterFilterLib into it as the first version
> of MdeLibs.dsc.inc.
> Update Platform dsc to consume the MdeLibs.dsc.inc.
> Add the description for the return flag in FilterBeforeXXXX functions in
> header file source code.
> Extend the years for Intel copyright.
> Add mssing change the dsc files in OvmfPkg.
>
> Cc: Andrew Fish <afish@apple.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
>
> Dandan Bi (29):
> MdePkg: Add RegisterFilterLib class and NULL instance
> MdePkg: Add MdeLibs.dsc.inc file to MdePkg
> ArmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> ArmPlatformPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> CryptoPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> DynamicTablesPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> EmbeddedPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> EmulatorPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> FatPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> FmpDevicePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> IntelFsp2Pkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> IntelFsp2WrapperPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> MdeModulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> MdePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> NetworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> OvmfPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> PcAtChipsetPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> RedfishPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> SecurityPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> ShellPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> SignedCapsulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> SourceLevelDebugPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> StandaloneMmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> UefiCpuPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> UefiPayloadPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> MdePkg/IoLib: Filter/trace port IO/MMIO access
> MdePkg/Baseib: Filter/trace MSR access for IA32/X64
>
> ArmPkg/ArmPkg.dsc | 2 +
> ArmPlatformPkg/ArmPlatformPkg.dsc | 2 +
> ArmVirtPkg/ArmVirt.dsc.inc | 4 +-
> CryptoPkg/CryptoPkg.dsc | 4 +-
> DynamicTablesPkg/DynamicTablesPkg.dsc | 2 +
> EmbeddedPkg/EmbeddedPkg.dsc | 4 +-
> EmulatorPkg/EmulatorPkg.dsc | 4 +-
> FatPkg/FatPkg.dsc | 4 +-
> FmpDevicePkg/FmpDevicePkg.dsc | 4 +-
> IntelFsp2Pkg/IntelFsp2Pkg.dsc | 4 +-
> IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc | 2 +
> IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc | 4 +-
> MdeModulePkg/MdeModulePkg.dsc | 4 +-
> MdePkg/Include/Library/RegisterFilterLib.h | 243 ++++++++++++++++
> .../BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf | 3 +-
> .../BaseIoLibIntrinsicArmVirt.inf | 3 +-
> .../BaseIoLibIntrinsicInternal.h | 3 +-
> .../BaseIoLibIntrinsicSev.inf | 3 +-
> MdePkg/Library/BaseIoLibIntrinsic/IoLib.c | 95 ++++--
> .../Library/BaseIoLibIntrinsic/IoLibArmVirt.c | 82 +++++-
> MdePkg/Library/BaseIoLibIntrinsic/IoLibGcc.c | 58 +++-
> MdePkg/Library/BaseIoLibIntrinsic/IoLibMsc.c | 81 ++++--
> MdePkg/Library/BaseIoLibIntrinsic/IoLibNoIo.c | 74 ++++-
> MdePkg/Library/BaseLib/BaseLib.inf | 5 +-
> MdePkg/Library/BaseLib/Ia32/GccInlinePriv.c | 38 ++-
> MdePkg/Library/BaseLib/Ia32/ReadMsr64.c | 38 ++-
> MdePkg/Library/BaseLib/Ia32/WriteMsr64.c | 22 +-
> MdePkg/Library/BaseLib/X64/GccInlinePriv.c | 43 ++-
> MdePkg/Library/BaseLib/X64/ReadMsr64.c | 15 +-
> MdePkg/Library/BaseLib/X64/WriteMsr64.c | 13 +-
> .../RegisterFilterLibNull.c | 271 ++++++++++++++++++
> .../RegisterFilterLibNull.inf | 23 ++
> .../RegisterFilterLibNull.uni | 13 +
> MdePkg/MdeLibs.dsc.inc | 14 +
> MdePkg/MdePkg.dec | 7 +-
> MdePkg/MdePkg.dsc | 6 +-
> NetworkPkg/NetworkPkg.dsc | 4 +-
> OvmfPkg/AmdSev/AmdSevX64.dsc | 4 +-
> OvmfPkg/Bhyve/BhyveX64.dsc | 4 +-
> OvmfPkg/OvmfPkgIa32.dsc | 4 +-
> OvmfPkg/OvmfPkgIa32X64.dsc | 4 +-
> OvmfPkg/OvmfPkgX64.dsc | 4 +-
> OvmfPkg/OvmfXen.dsc | 4 +-
> PcAtChipsetPkg/PcAtChipsetPkg.dsc | 4 +-
> RedfishPkg/RedfishPkg.dsc | 4 +-
> SecurityPkg/SecurityPkg.dsc | 4 +-
> ShellPkg/ShellPkg.dsc | 4 +-
> SignedCapsulePkg/SignedCapsulePkg.dsc | 4 +-
> SourceLevelDebugPkg/SourceLevelDebugPkg.dsc | 4 +-
> StandaloneMmPkg/StandaloneMmPkg.dsc | 4 +-
> UefiCpuPkg/UefiCpuPkg.dsc | 4 +-
> UefiPayloadPkg/UefiPayloadPkg.dsc | 4 +-
> .../UnitTestFrameworkPkgTarget.dsc.inc | 4 +-
> 53 files changed, 1120 insertions(+), 145 deletions(-) create mode 100644
> MdePkg/Include/Library/RegisterFilterLib.h
> create mode 100644
> MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.c
> create mode 100644
> MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.inf
> create mode 100644
> MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.uni
> create mode 100644 MdePkg/MdeLibs.dsc.inc
>
> --
> 2.18.0.windows.1
>
>
>
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* 回复: [edk2-devel] [patch V2 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
2021-03-23 2:05 ` [edk2-devel] [patch V2 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access Dandan Bi
@ 2021-03-23 8:39 ` gaoliming
2021-03-25 6:18 ` Dandan Bi
0 siblings, 1 reply; 3+ messages in thread
From: gaoliming @ 2021-03-23 8:39 UTC (permalink / raw)
To: 'Bi, Dandan', devel
Cc: 'Andrew Fish', 'Leif Lindholm',
'Laszlo Ersek', 'Kinney, Michael D',
'Liu, Zhiguang'
Dandan:
This change is good to me. Reviewed-by: Liming Gao <liming.gao@intel.com>
for the patch set.
Thanks
Liming
> -----邮件原件-----
> 发件人: Bi, Dandan <dandan.bi@intel.com>
> 发送时间: 2021年3月23日 10:06
> 收件人: devel@edk2.groups.io; Bi, Dandan <dandan.bi@intel.com>
> 抄送: Andrew Fish <afish@apple.com>; Leif Lindholm <leif@nuviainc.com>;
> Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>; Liu,
> Zhiguang <zhiguang.liu@intel.com>
> 主题: RE: [edk2-devel] [patch V2 00/29] Add a new library class
> RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
>
> Hi All,
>
> Here is the branch for these changes in Edk2.
> https://github.com/dandanbi/edk2/tree/RegisterFilterLibV2
> https://github.com/tianocore/edk2/pull/1509
>
> Hi Mike, Liming and Zhiguang,
>
> Could you help review the first and last 2 patches in MdePkg?
> Since we should submit the first 2 patches firstly before the changes of
dsc
> files .
>
>
> Thanks,
> Dandan
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan
> > Bi
> > Sent: Monday, March 22, 2021 4:09 PM
> > To: devel@edk2.groups.io
> > Cc: Andrew Fish <afish@apple.com>; Leif Lindholm <leif@nuviainc.com>;
> > Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> > Liu, Zhiguang <zhiguang.liu@intel.com>
> > Subject: [edk2-devel] [patch V2 00/29] Add a new library class
> > RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
> >
> > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
> > RFC: https://edk2.groups.io/g/devel/message/72530
> >
> > Patch 1 is to add RegisterFilterLib Library Class in edk2 to
filter/trace port
> > IO/MMIO/MSR access and add a RegisterFilterLibNull instance.
> > Patch 2 is to add the MdeLibs.dsc.inc file to MdePkg for some default
> libraries
> > provided by MdePkg and add RegisterFilterLib into it as the first
version of
> > MdeLibs.dsc.inc.
> > Last 2 patches are to update APIs in IoLib and BaseLib to filter/trace
port
> > IO/MMIO/MSR access.
> > Remaining patches are to update related dsc files to consume
> > MdeLibs.dsc.inc for RegisterFilterLib.
> >
> > Will submit patch 1 and 2 firstly.
> > And then update related dsc files in edk2 and edk2platform repo to
consume
> > MdeLibs.dsc.inc for RegisterFilterLib.
> > At last will submit the patches to update IoLib and BaseLib to
filter/trace
> port
> > IO/MMIO/MSR access.
> >
> > --
> > V2:
> > Introduce MdeLibs.dsc.inc and add RegisterFilterLib into it as the first
> version
> > of MdeLibs.dsc.inc.
> > Update Platform dsc to consume the MdeLibs.dsc.inc.
> > Add the description for the return flag in FilterBeforeXXXX functions in
> > header file source code.
> > Extend the years for Intel copyright.
> > Add mssing change the dsc files in OvmfPkg.
> >
> > Cc: Andrew Fish <afish@apple.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> >
> > Dandan Bi (29):
> > MdePkg: Add RegisterFilterLib class and NULL instance
> > MdePkg: Add MdeLibs.dsc.inc file to MdePkg
> > ArmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > ArmPlatformPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > CryptoPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > DynamicTablesPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > EmbeddedPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > EmulatorPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > FatPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > FmpDevicePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > IntelFsp2Pkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > IntelFsp2WrapperPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > MdeModulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > MdePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > NetworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > OvmfPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > PcAtChipsetPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > RedfishPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > SecurityPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > ShellPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > SignedCapsulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > SourceLevelDebugPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > StandaloneMmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > UefiCpuPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > UefiPayloadPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > MdePkg/IoLib: Filter/trace port IO/MMIO access
> > MdePkg/Baseib: Filter/trace MSR access for IA32/X64
> >
> > ArmPkg/ArmPkg.dsc | 2 +
> > ArmPlatformPkg/ArmPlatformPkg.dsc | 2 +
> > ArmVirtPkg/ArmVirt.dsc.inc | 4 +-
> > CryptoPkg/CryptoPkg.dsc | 4 +-
> > DynamicTablesPkg/DynamicTablesPkg.dsc | 2 +
> > EmbeddedPkg/EmbeddedPkg.dsc | 4 +-
> > EmulatorPkg/EmulatorPkg.dsc | 4 +-
> > FatPkg/FatPkg.dsc | 4 +-
> > FmpDevicePkg/FmpDevicePkg.dsc | 4 +-
> > IntelFsp2Pkg/IntelFsp2Pkg.dsc | 4 +-
> > IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc | 2 +
> > IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc | 4 +-
> > MdeModulePkg/MdeModulePkg.dsc | 4 +-
> > MdePkg/Include/Library/RegisterFilterLib.h | 243
> ++++++++++++++++
> > .../BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf | 3 +-
> > .../BaseIoLibIntrinsicArmVirt.inf | 3 +-
> > .../BaseIoLibIntrinsicInternal.h | 3 +-
> > .../BaseIoLibIntrinsicSev.inf | 3 +-
> > MdePkg/Library/BaseIoLibIntrinsic/IoLib.c | 95 ++++--
> > .../Library/BaseIoLibIntrinsic/IoLibArmVirt.c | 82 +++++-
> > MdePkg/Library/BaseIoLibIntrinsic/IoLibGcc.c | 58 +++-
> > MdePkg/Library/BaseIoLibIntrinsic/IoLibMsc.c | 81 ++++--
> > MdePkg/Library/BaseIoLibIntrinsic/IoLibNoIo.c | 74 ++++-
> > MdePkg/Library/BaseLib/BaseLib.inf | 5 +-
> > MdePkg/Library/BaseLib/Ia32/GccInlinePriv.c | 38 ++-
> > MdePkg/Library/BaseLib/Ia32/ReadMsr64.c | 38 ++-
> > MdePkg/Library/BaseLib/Ia32/WriteMsr64.c | 22 +-
> > MdePkg/Library/BaseLib/X64/GccInlinePriv.c | 43 ++-
> > MdePkg/Library/BaseLib/X64/ReadMsr64.c | 15 +-
> > MdePkg/Library/BaseLib/X64/WriteMsr64.c | 13 +-
> > .../RegisterFilterLibNull.c | 271
> ++++++++++++++++++
> > .../RegisterFilterLibNull.inf | 23 ++
> > .../RegisterFilterLibNull.uni | 13 +
> > MdePkg/MdeLibs.dsc.inc | 14 +
> > MdePkg/MdePkg.dec | 7 +-
> > MdePkg/MdePkg.dsc | 6 +-
> > NetworkPkg/NetworkPkg.dsc | 4 +-
> > OvmfPkg/AmdSev/AmdSevX64.dsc | 4 +-
> > OvmfPkg/Bhyve/BhyveX64.dsc | 4 +-
> > OvmfPkg/OvmfPkgIa32.dsc | 4 +-
> > OvmfPkg/OvmfPkgIa32X64.dsc | 4 +-
> > OvmfPkg/OvmfPkgX64.dsc | 4 +-
> > OvmfPkg/OvmfXen.dsc | 4 +-
> > PcAtChipsetPkg/PcAtChipsetPkg.dsc | 4 +-
> > RedfishPkg/RedfishPkg.dsc | 4 +-
> > SecurityPkg/SecurityPkg.dsc | 4 +-
> > ShellPkg/ShellPkg.dsc | 4 +-
> > SignedCapsulePkg/SignedCapsulePkg.dsc | 4 +-
> > SourceLevelDebugPkg/SourceLevelDebugPkg.dsc | 4 +-
> > StandaloneMmPkg/StandaloneMmPkg.dsc | 4 +-
> > UefiCpuPkg/UefiCpuPkg.dsc | 4 +-
> > UefiPayloadPkg/UefiPayloadPkg.dsc | 4 +-
> > .../UnitTestFrameworkPkgTarget.dsc.inc | 4 +-
> > 53 files changed, 1120 insertions(+), 145 deletions(-) create mode
> 100644
> > MdePkg/Include/Library/RegisterFilterLib.h
> > create mode 100644
> > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.c
> > create mode 100644
> > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.inf
> > create mode 100644
> > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.uni
> > create mode 100644 MdePkg/MdeLibs.dsc.inc
> >
> > --
> > 2.18.0.windows.1
> >
> >
> >
> >
> >
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [edk2-devel] [patch V2 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
2021-03-23 8:39 ` 回复: " gaoliming
@ 2021-03-25 6:18 ` Dandan Bi
0 siblings, 0 replies; 3+ messages in thread
From: Dandan Bi @ 2021-03-25 6:18 UTC (permalink / raw)
To: gaoliming, devel@edk2.groups.io
Cc: 'Andrew Fish', 'Leif Lindholm',
'Laszlo Ersek', Kinney, Michael D, Liu, Zhiguang
Thank you Liming.
If no more comments, I plan to submit following 2 patches firstly tomorrow.
[patch V2 01/29] MdePkg: Add RegisterFilterLib class and NULL instance (https://edk2.groups.io/g/devel/message/73072)
[patch V2 02/29] MdePkg: Add MdeLibs.dsc.inc file to MdePkg (https://edk2.groups.io/g/devel/message/73073)
Then we can submit following dsc patches and IoLib and BaseLib update patches.
Please let me know if you have any concern.
Thanks,
Dandan
> -----Original Message-----
> From: gaoliming <gaoliming@byosoft.com.cn>
> Sent: Tuesday, March 23, 2021 4:40 PM
> To: Bi, Dandan <dandan.bi@intel.com>; devel@edk2.groups.io
> Cc: 'Andrew Fish' <afish@apple.com>; 'Leif Lindholm' <leif@nuviainc.com>;
> 'Laszlo Ersek' <lersek@redhat.com>; Kinney, Michael D
> <michael.d.kinney@intel.com>; Liu, Zhiguang <zhiguang.liu@intel.com>
> Subject: 回复: [edk2-devel] [patch V2 00/29] Add a new library class
> RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
>
> Dandan:
> This change is good to me. Reviewed-by: Liming Gao <liming.gao@intel.com>
> for the patch set.
>
> Thanks
> Liming
> > -----邮件原件-----
> > 发件人: Bi, Dandan <dandan.bi@intel.com>
> > 发送时间: 2021年3月23日 10:06
> > 收件人: devel@edk2.groups.io; Bi, Dandan <dandan.bi@intel.com>
> > 抄送: Andrew Fish <afish@apple.com>; Leif Lindholm <leif@nuviainc.com>;
> > Laszlo Ersek <lersek@redhat.com>; Kinney, Michael D
> > <michael.d.kinney@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> > Liu, Zhiguang <zhiguang.liu@intel.com>
> > 主题: RE: [edk2-devel] [patch V2 00/29] Add a new library class
> > RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
> >
> > Hi All,
> >
> > Here is the branch for these changes in Edk2.
> > https://github.com/dandanbi/edk2/tree/RegisterFilterLibV2
> > https://github.com/tianocore/edk2/pull/1509
> >
> > Hi Mike, Liming and Zhiguang,
> >
> > Could you help review the first and last 2 patches in MdePkg?
> > Since we should submit the first 2 patches firstly before the changes
> > of
> dsc
> > files .
> >
> >
> > Thanks,
> > Dandan
> > > -----Original Message-----
> > > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> > > Dandan Bi
> > > Sent: Monday, March 22, 2021 4:09 PM
> > > To: devel@edk2.groups.io
> > > Cc: Andrew Fish <afish@apple.com>; Leif Lindholm
> > > <leif@nuviainc.com>; Laszlo Ersek <lersek@redhat.com>; Kinney,
> > > Michael D <michael.d.kinney@intel.com>; Liming Gao
> > > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> > > Subject: [edk2-devel] [patch V2 00/29] Add a new library class
> > > RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
> > >
> > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3246
> > > RFC: https://edk2.groups.io/g/devel/message/72530
> > >
> > > Patch 1 is to add RegisterFilterLib Library Class in edk2 to
> filter/trace port
> > > IO/MMIO/MSR access and add a RegisterFilterLibNull instance.
> > > Patch 2 is to add the MdeLibs.dsc.inc file to MdePkg for some
> > > default
> > libraries
> > > provided by MdePkg and add RegisterFilterLib into it as the first
> version of
> > > MdeLibs.dsc.inc.
> > > Last 2 patches are to update APIs in IoLib and BaseLib to
> > > filter/trace
> port
> > > IO/MMIO/MSR access.
> > > Remaining patches are to update related dsc files to consume
> > > MdeLibs.dsc.inc for RegisterFilterLib.
> > >
> > > Will submit patch 1 and 2 firstly.
> > > And then update related dsc files in edk2 and edk2platform repo to
> consume
> > > MdeLibs.dsc.inc for RegisterFilterLib.
> > > At last will submit the patches to update IoLib and BaseLib to
> filter/trace
> > port
> > > IO/MMIO/MSR access.
> > >
> > > --
> > > V2:
> > > Introduce MdeLibs.dsc.inc and add RegisterFilterLib into it as the
> > > first
> > version
> > > of MdeLibs.dsc.inc.
> > > Update Platform dsc to consume the MdeLibs.dsc.inc.
> > > Add the description for the return flag in FilterBeforeXXXX
> > > functions in header file source code.
> > > Extend the years for Intel copyright.
> > > Add mssing change the dsc files in OvmfPkg.
> > >
> > > Cc: Andrew Fish <afish@apple.com>
> > > Cc: Leif Lindholm <leif@nuviainc.com>
> > > Cc: Laszlo Ersek <lersek@redhat.com>
> > > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > > Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> > >
> > > Dandan Bi (29):
> > > MdePkg: Add RegisterFilterLib class and NULL instance
> > > MdePkg: Add MdeLibs.dsc.inc file to MdePkg
> > > ArmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > ArmPlatformPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > CryptoPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > DynamicTablesPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > EmbeddedPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > EmulatorPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > FatPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > FmpDevicePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > IntelFsp2Pkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > IntelFsp2WrapperPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > MdeModulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > MdePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > NetworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > OvmfPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > PcAtChipsetPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > RedfishPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > SecurityPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > ShellPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > SignedCapsulePkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > SourceLevelDebugPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > StandaloneMmPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > UefiCpuPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > UefiPayloadPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > > MdePkg/IoLib: Filter/trace port IO/MMIO access
> > > MdePkg/Baseib: Filter/trace MSR access for IA32/X64
> > >
> > > ArmPkg/ArmPkg.dsc | 2 +
> > > ArmPlatformPkg/ArmPlatformPkg.dsc | 2 +
> > > ArmVirtPkg/ArmVirt.dsc.inc | 4 +-
> > > CryptoPkg/CryptoPkg.dsc | 4 +-
> > > DynamicTablesPkg/DynamicTablesPkg.dsc | 2 +
> > > EmbeddedPkg/EmbeddedPkg.dsc | 4 +-
> > > EmulatorPkg/EmulatorPkg.dsc | 4 +-
> > > FatPkg/FatPkg.dsc | 4 +-
> > > FmpDevicePkg/FmpDevicePkg.dsc | 4 +-
> > > IntelFsp2Pkg/IntelFsp2Pkg.dsc | 4 +-
> > > IntelFsp2Pkg/Tools/Tests/QemuFspPkg.dsc | 2 +
> > > IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc | 4 +-
> > > MdeModulePkg/MdeModulePkg.dsc | 4 +-
> > > MdePkg/Include/Library/RegisterFilterLib.h | 243
> > ++++++++++++++++
> > > .../BaseIoLibIntrinsic/BaseIoLibIntrinsic.inf | 3 +-
> > > .../BaseIoLibIntrinsicArmVirt.inf | 3 +-
> > > .../BaseIoLibIntrinsicInternal.h | 3 +-
> > > .../BaseIoLibIntrinsicSev.inf | 3 +-
> > > MdePkg/Library/BaseIoLibIntrinsic/IoLib.c | 95 ++++--
> > > .../Library/BaseIoLibIntrinsic/IoLibArmVirt.c | 82 +++++-
> > > MdePkg/Library/BaseIoLibIntrinsic/IoLibGcc.c | 58 +++-
> > > MdePkg/Library/BaseIoLibIntrinsic/IoLibMsc.c | 81 ++++--
> > > MdePkg/Library/BaseIoLibIntrinsic/IoLibNoIo.c | 74 ++++-
> > > MdePkg/Library/BaseLib/BaseLib.inf | 5 +-
> > > MdePkg/Library/BaseLib/Ia32/GccInlinePriv.c | 38 ++-
> > > MdePkg/Library/BaseLib/Ia32/ReadMsr64.c | 38 ++-
> > > MdePkg/Library/BaseLib/Ia32/WriteMsr64.c | 22 +-
> > > MdePkg/Library/BaseLib/X64/GccInlinePriv.c | 43 ++-
> > > MdePkg/Library/BaseLib/X64/ReadMsr64.c | 15 +-
> > > MdePkg/Library/BaseLib/X64/WriteMsr64.c | 13 +-
> > > .../RegisterFilterLibNull.c | 271
> > ++++++++++++++++++
> > > .../RegisterFilterLibNull.inf | 23 ++
> > > .../RegisterFilterLibNull.uni | 13 +
> > > MdePkg/MdeLibs.dsc.inc | 14 +
> > > MdePkg/MdePkg.dec | 7 +-
> > > MdePkg/MdePkg.dsc | 6 +-
> > > NetworkPkg/NetworkPkg.dsc | 4 +-
> > > OvmfPkg/AmdSev/AmdSevX64.dsc | 4 +-
> > > OvmfPkg/Bhyve/BhyveX64.dsc | 4 +-
> > > OvmfPkg/OvmfPkgIa32.dsc | 4 +-
> > > OvmfPkg/OvmfPkgIa32X64.dsc | 4 +-
> > > OvmfPkg/OvmfPkgX64.dsc | 4 +-
> > > OvmfPkg/OvmfXen.dsc | 4 +-
> > > PcAtChipsetPkg/PcAtChipsetPkg.dsc | 4 +-
> > > RedfishPkg/RedfishPkg.dsc | 4 +-
> > > SecurityPkg/SecurityPkg.dsc | 4 +-
> > > ShellPkg/ShellPkg.dsc | 4 +-
> > > SignedCapsulePkg/SignedCapsulePkg.dsc | 4 +-
> > > SourceLevelDebugPkg/SourceLevelDebugPkg.dsc | 4 +-
> > > StandaloneMmPkg/StandaloneMmPkg.dsc | 4 +-
> > > UefiCpuPkg/UefiCpuPkg.dsc | 4 +-
> > > UefiPayloadPkg/UefiPayloadPkg.dsc | 4 +-
> > > .../UnitTestFrameworkPkgTarget.dsc.inc | 4 +-
> > > 53 files changed, 1120 insertions(+), 145 deletions(-) create mode
> > 100644
> > > MdePkg/Include/Library/RegisterFilterLib.h
> > > create mode 100644
> > > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.c
> > > create mode 100644
> > > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.inf
> > > create mode 100644
> > > MdePkg/Library/RegisterFilterLibNull/RegisterFilterLibNull.uni
> > > create mode 100644 MdePkg/MdeLibs.dsc.inc
> > >
> > > --
> > > 2.18.0.windows.1
> > >
> > >
> > >
> > >
> > >
>
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-03-25 6:18 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <166E9BC61BB3701A.10450@groups.io>
2021-03-23 2:05 ` [edk2-devel] [patch V2 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access Dandan Bi
2021-03-23 8:39 ` 回复: " gaoliming
2021-03-25 6:18 ` Dandan Bi
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox