public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* Re: [edk2-devel] [patch V4 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
       [not found] <16707FF9B84D29FC.5213@groups.io>
@ 2021-03-29  2:55 ` Dandan Bi
  2021-03-29  5:20   ` 回复: " gaoliming
       [not found] ` <1670B0B0AC65D7DB.438@groups.io>
  1 sibling, 1 reply; 3+ messages in thread
From: Dandan Bi @ 2021-03-29  2:55 UTC (permalink / raw)
  To: devel@edk2.groups.io, Bi, Dandan
  Cc: Kinney, Michael D, Sean Brogan, Bret Barkelew, Leif Lindholm,
	Ard Biesheuvel, Laszlo Ersek, Yao, Jiewen, Wang, Jian J,
	Sami Mujawar, Alexei Fedorov, Andrew Fish, Ni, Ray, Chiu, Chasel,
	Desimone, Nathaniel L, Zeng, Star, Gao, Zhichao, Wu, Hao A,
	Ma, Maurice, Dong, Guo, You, Benjamin, Dong, Eric, Maciej Rabeda,
	Wu, Jiaxin, Fu, Siyuan, Justen, Jordan L, Supreeth Venkatesh,
	Abner Chang, Nickle Wang, Liming Gao, Liu, Zhiguang

Hi Maintainers/Reviewers,

For this change serial,  could you help review the patch in your packages in edk2/edk2-platform repo? Thanks in advance.


Thanks,
Dandan
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan
> Bi
> Sent: Sunday, March 28, 2021 8:02 PM
> To: devel@edk2.groups.io
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Sean Brogan
> <sean.brogan@microsoft.com>; Bret Barkelew
> <Bret.Barkelew@microsoft.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> Biesheuvel <ardb+tianocore@kernel.org>; Laszlo Ersek
> <lersek@redhat.com>; Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Sami Mujawar <Sami.Mujawar@arm.com>; Alexei
> Fedorov <Alexei.Fedorov@arm.com>; Andrew Fish <afish@apple.com>; Ni,
> Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>; Dong,
> Eric <eric.dong@intel.com>; Maciej Rabeda
> <maciej.rabeda@linux.intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>; Fu,
> Siyuan <siyuan.fu@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>;
> Supreeth Venkatesh <supreeth.venkatesh@arm.com>; Abner Chang
> <abner.chang@hpe.com>; Nickle Wang <nickle.wang@hpe.com>; Liming
> Gao <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> Subject: [edk2-devel] [patch V4 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
> Branch for this change:
> https://github.com/dandanbi/edk2/tree/RegisterFilterLibV3
> 
> 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.
> 
> ---
> V4:
> Consume the MdeLibs.dsc.inc for RegisterFilterLibNull in dsc file rather than
> in xxx.dsc.inc.
> Following 2 patchws were updated for consuming MdeLibs.dsc.inc in
> platform dsc file.
>   ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
>   UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> Other patches are the same with V3.
> ---
> V3:
> 1. Don't assume Library instances in MdeLibs.dsc.inc are must in common
> [LibraryClasses] section.
>    Can specify different [LibraryClasses] section for the libary instances in
> MdeLibs.dsc.inc.
> 2. Update plaftom dsc to consume the MdeLibs.dsc outside the
> [LibraryClasses] section.
> 3. Pick up the R-B for patch 1, patch 28, patch 29.
> 
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Sean Brogan <sean.brogan@microsoft.com>
> Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> Cc: Leif Lindholm <leif@nuviainc.com>
> Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> Cc: Laszlo Ersek <lersek@redhat.com>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Cc: Jian J Wang <jian.j.wang@intel.com>
> Cc: Sami Mujawar <Sami.Mujawar@arm.com>
> Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
> Cc: Andrew Fish <afish@apple.com>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Chasel Chiu <chasel.chiu@intel.com>
> Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Cc: Hao A Wu <hao.a.wu@intel.com>
> Cc: Maurice Ma <maurice.ma@intel.com>
> Cc: Guo Dong <guo.dong@intel.com>
> Cc: Benjamin You <benjamin.you@intel.com>
> Cc: Eric Dong <eric.dong@intel.com>
> Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Cc: Siyuan Fu <siyuan.fu@intel.com>
> Cc: Jordan Justen <jordan.l.justen@intel.com>
> Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> Cc: Abner Chang <abner.chang@hpe.com>
> Cc: Nickle Wang <nickle.wang@hpe.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                       |   5 +-
>  DynamicTablesPkg/DynamicTablesPkg.dsc         |   2 +
>  EmbeddedPkg/EmbeddedPkg.dsc                   |   5 +-
>  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                        |  15 +
>  MdePkg/MdePkg.dec                             |   7 +-
>  MdePkg/MdePkg.dsc                             |   6 +-
>  NetworkPkg/NetworkPkg.dsc                     |   4 +-
>  OvmfPkg/AmdSev/AmdSevX64.dsc                  |   5 +-
>  OvmfPkg/Bhyve/BhyveX64.dsc                    |   5 +-
>  OvmfPkg/OvmfPkgIa32.dsc                       |   5 +-
>  OvmfPkg/OvmfPkgIa32X64.dsc                    |   5 +-
>  OvmfPkg/OvmfPkgX64.dsc                        |   5 +-
>  OvmfPkg/OvmfXen.dsc                           |   5 +-
>  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           |   5 +-
>  UefiCpuPkg/UefiCpuPkg.dsc                     |   4 +-
>  UefiPayloadPkg/UefiPayloadPkg.dsc             |   5 +-
>  .../UnitTestFrameworkPkgTarget.dsc.inc        |   4 +-
>  53 files changed, 1131 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 V4 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
  2021-03-29  2:55 ` [edk2-devel] [patch V4 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access Dandan Bi
@ 2021-03-29  5:20   ` gaoliming
  0 siblings, 0 replies; 3+ messages in thread
From: gaoliming @ 2021-03-29  5:20 UTC (permalink / raw)
  To: devel, dandan.bi
  Cc: 'Kinney, Michael D', 'Sean Brogan',
	'Bret Barkelew', 'Leif Lindholm',
	'Ard Biesheuvel', 'Laszlo Ersek',
	'Yao, Jiewen', 'Wang, Jian J',
	'Sami Mujawar', 'Alexei Fedorov',
	'Andrew Fish', 'Ni, Ray', 'Chiu, Chasel',
	'Desimone, Nathaniel L', 'Zeng, Star',
	'Gao, Zhichao', 'Wu, Hao A',
	'Ma, Maurice', 'Dong, Guo',
	'You, Benjamin', 'Dong, Eric',
	'Maciej Rabeda', 'Wu, Jiaxin',
	'Fu, Siyuan', 'Justen, Jordan L',
	'Supreeth Venkatesh', 'Abner Chang',
	'Nickle Wang', 'Liu, Zhiguang'

Dandan:
  Thanks for your quick update. I agree this change in MdeLibs.dsc.inc does
make sense. 

  For this patch set, Reviewed-by: Liming Gao <gaoliming@byosoft.com.cn>

Thanks
Liming
> -----邮件原件-----
> 发件人: devel@edk2.groups.io <devel@edk2.groups.io> 代表 Dandan Bi
> 发送时间: 2021年3月29日 10:56
> 收件人: devel@edk2.groups.io; Bi, Dandan <dandan.bi@intel.com>
> 抄送: Kinney, Michael D <michael.d.kinney@intel.com>; Sean Brogan
> <sean.brogan@microsoft.com>; Bret Barkelew
> <Bret.Barkelew@microsoft.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> Biesheuvel <ardb+tianocore@kernel.org>; Laszlo Ersek <lersek@redhat.com>;
> Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J <jian.j.wang@intel.com>;
> Sami Mujawar <Sami.Mujawar@arm.com>; Alexei Fedorov
> <Alexei.Fedorov@arm.com>; Andrew Fish <afish@apple.com>; Ni, Ray
> <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>; Dong, Eric
> <eric.dong@intel.com>; Maciej Rabeda <maciej.rabeda@linux.intel.com>;
> Wu, Jiaxin <jiaxin.wu@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>;
Justen,
> Jordan L <jordan.l.justen@intel.com>; Supreeth Venkatesh
> <supreeth.venkatesh@arm.com>; Abner Chang <abner.chang@hpe.com>;
> Nickle Wang <nickle.wang@hpe.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> 主题: Re: [edk2-devel] [patch V4 00/29] Add a new library class
> RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
> 
> Hi Maintainers/Reviewers,
> 
> For this change serial,  could you help review the patch in your packages
in
> edk2/edk2-platform repo? Thanks in advance.
> 
> 
> Thanks,
> Dandan
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan
> > Bi
> > Sent: Sunday, March 28, 2021 8:02 PM
> > To: devel@edk2.groups.io
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Sean Brogan
> > <sean.brogan@microsoft.com>; Bret Barkelew
> > <Bret.Barkelew@microsoft.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> > Biesheuvel <ardb+tianocore@kernel.org>; Laszlo Ersek
> > <lersek@redhat.com>; Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> > <jian.j.wang@intel.com>; Sami Mujawar <Sami.Mujawar@arm.com>;
> Alexei
> > Fedorov <Alexei.Fedorov@arm.com>; Andrew Fish <afish@apple.com>; Ni,
> > Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> > Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> > <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Wu, Hao A
> > <hao.a.wu@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> > <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>; Dong,
> > Eric <eric.dong@intel.com>; Maciej Rabeda
> > <maciej.rabeda@linux.intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>; Fu,
> > Siyuan <siyuan.fu@intel.com>; Justen, Jordan L
> <jordan.l.justen@intel.com>;
> > Supreeth Venkatesh <supreeth.venkatesh@arm.com>; Abner Chang
> > <abner.chang@hpe.com>; Nickle Wang <nickle.wang@hpe.com>; Liming
> > Gao <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> > Subject: [edk2-devel] [patch V4 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
> > Branch for this change:
> > https://github.com/dandanbi/edk2/tree/RegisterFilterLibV3
> >
> > 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.
> >
> > ---
> > V4:
> > Consume the MdeLibs.dsc.inc for RegisterFilterLibNull in dsc file rather
than
> > in xxx.dsc.inc.
> > Following 2 patchws were updated for consuming MdeLibs.dsc.inc in
> > platform dsc file.
> >   ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> >   UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > Other patches are the same with V3.
> > ---
> > V3:
> > 1. Don't assume Library instances in MdeLibs.dsc.inc are must in common
> > [LibraryClasses] section.
> >    Can specify different [LibraryClasses] section for the libary
instances in
> > MdeLibs.dsc.inc.
> > 2. Update plaftom dsc to consume the MdeLibs.dsc outside the
> > [LibraryClasses] section.
> > 3. Pick up the R-B for patch 1, patch 28, patch 29.
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Sean Brogan <sean.brogan@microsoft.com>
> > Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Sami Mujawar <Sami.Mujawar@arm.com>
> > Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
> > Cc: Andrew Fish <afish@apple.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Star Zeng <star.zeng@intel.com>
> > Cc: Zhichao Gao <zhichao.gao@intel.com>
> > Cc: Hao A Wu <hao.a.wu@intel.com>
> > Cc: Maurice Ma <maurice.ma@intel.com>
> > Cc: Guo Dong <guo.dong@intel.com>
> > Cc: Benjamin You <benjamin.you@intel.com>
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
> > Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> > Cc: Siyuan Fu <siyuan.fu@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> > Cc: Abner Chang <abner.chang@hpe.com>
> > Cc: Nickle Wang <nickle.wang@hpe.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                       |   5 +-
> >  DynamicTablesPkg/DynamicTablesPkg.dsc         |   2 +
> >  EmbeddedPkg/EmbeddedPkg.dsc                   |   5 +-
> >  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                        |  15 +
> >  MdePkg/MdePkg.dec                             |   7 +-
> >  MdePkg/MdePkg.dsc                             |   6 +-
> >  NetworkPkg/NetworkPkg.dsc                     |   4 +-
> >  OvmfPkg/AmdSev/AmdSevX64.dsc                  |   5 +-
> >  OvmfPkg/Bhyve/BhyveX64.dsc                    |   5 +-
> >  OvmfPkg/OvmfPkgIa32.dsc                       |   5 +-
> >  OvmfPkg/OvmfPkgIa32X64.dsc                    |   5 +-
> >  OvmfPkg/OvmfPkgX64.dsc                        |   5 +-
> >  OvmfPkg/OvmfXen.dsc                           |   5 +-
> >  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           |   5 +-
> >  UefiCpuPkg/UefiCpuPkg.dsc                     |   4 +-
> >  UefiPayloadPkg/UefiPayloadPkg.dsc             |   5 +-
> >  .../UnitTestFrameworkPkgTarget.dsc.inc        |   4 +-
> >  53 files changed, 1131 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 V4 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
       [not found] ` <1670B0B0AC65D7DB.438@groups.io>
@ 2021-03-30 13:24   ` Dandan Bi
  0 siblings, 0 replies; 3+ messages in thread
From: Dandan Bi @ 2021-03-30 13:24 UTC (permalink / raw)
  To: devel@edk2.groups.io
  Cc: Kinney, Michael D, Sean Brogan, Bret Barkelew, Leif Lindholm,
	Ard Biesheuvel, Laszlo Ersek, Yao, Jiewen, Wang, Jian J,
	Sami Mujawar, Alexei Fedorov, Andrew Fish, Ni, Ray, Chiu, Chasel,
	Desimone, Nathaniel L, Zeng, Star, Gao, Zhichao, Wu, Hao A,
	Ma, Maurice, Dong, Guo, You, Benjamin, Dong, Eric, Maciej Rabeda,
	Wu, Jiaxin, Fu, Siyuan, Justen, Jordan L, Supreeth Venkatesh,
	Abner Chang, Nickle Wang, Liming Gao, Liu, Zhiguang

Hi All,

Thank you for the patch review.
I have submitted patch 1 &2 via following commits.
https://github.com/tianocore/edk2/commit/1c11e7a2142b1406ccff5e0af0893c94472871c8
https://github.com/tianocore/edk2/commit/98ff7e3c6373d3e0590f953459b6234ab12edce6

I plan to submit dsc changes tomorrow afternoon with R-B got from Liming, Mike, and some maintainers.
If you have any concern, please let me know. 


Thanks,
Dandan
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Dandan
> Bi
> Sent: Monday, March 29, 2021 10:56 AM
> To: devel@edk2.groups.io; Bi, Dandan <dandan.bi@intel.com>
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Sean Brogan
> <sean.brogan@microsoft.com>; Bret Barkelew
> <Bret.Barkelew@microsoft.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> Biesheuvel <ardb+tianocore@kernel.org>; Laszlo Ersek
> <lersek@redhat.com>; Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> <jian.j.wang@intel.com>; Sami Mujawar <Sami.Mujawar@arm.com>; Alexei
> Fedorov <Alexei.Fedorov@arm.com>; Andrew Fish <afish@apple.com>; Ni,
> Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>; Desimone,
> Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Wu, Hao A
> <hao.a.wu@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>; Dong,
> Eric <eric.dong@intel.com>; Maciej Rabeda
> <maciej.rabeda@linux.intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>; Fu,
> Siyuan <siyuan.fu@intel.com>; Justen, Jordan L <jordan.l.justen@intel.com>;
> Supreeth Venkatesh <supreeth.venkatesh@arm.com>; Abner Chang
> <abner.chang@hpe.com>; Nickle Wang <nickle.wang@hpe.com>; Liming
> Gao <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> Subject: Re: [edk2-devel] [patch V4 00/29] Add a new library class
> RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access
> 
> Hi Maintainers/Reviewers,
> 
> For this change serial,  could you help review the patch in your packages in
> edk2/edk2-platform repo? Thanks in advance.
> 
> 
> Thanks,
> Dandan
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of
> Dandan
> > Bi
> > Sent: Sunday, March 28, 2021 8:02 PM
> > To: devel@edk2.groups.io
> > Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Sean Brogan
> > <sean.brogan@microsoft.com>; Bret Barkelew
> > <Bret.Barkelew@microsoft.com>; Leif Lindholm <leif@nuviainc.com>; Ard
> > Biesheuvel <ardb+tianocore@kernel.org>; Laszlo Ersek
> > <lersek@redhat.com>; Yao, Jiewen <jiewen.yao@intel.com>; Wang, Jian J
> > <jian.j.wang@intel.com>; Sami Mujawar <Sami.Mujawar@arm.com>;
> Alexei
> > Fedorov <Alexei.Fedorov@arm.com>; Andrew Fish <afish@apple.com>; Ni,
> > Ray <ray.ni@intel.com>; Chiu, Chasel <chasel.chiu@intel.com>;
> > Desimone, Nathaniel L <nathaniel.l.desimone@intel.com>; Zeng, Star
> > <star.zeng@intel.com>; Gao, Zhichao <zhichao.gao@intel.com>; Wu, Hao
> A
> > <hao.a.wu@intel.com>; Ma, Maurice <maurice.ma@intel.com>; Dong,
> Guo
> > <guo.dong@intel.com>; You, Benjamin <benjamin.you@intel.com>; Dong,
> > Eric <eric.dong@intel.com>; Maciej Rabeda
> > <maciej.rabeda@linux.intel.com>; Wu, Jiaxin <jiaxin.wu@intel.com>; Fu,
> > Siyuan <siyuan.fu@intel.com>; Justen, Jordan L
> > <jordan.l.justen@intel.com>; Supreeth Venkatesh
> > <supreeth.venkatesh@arm.com>; Abner Chang <abner.chang@hpe.com>;
> > Nickle Wang <nickle.wang@hpe.com>; Liming Gao
> > <gaoliming@byosoft.com.cn>; Liu, Zhiguang <zhiguang.liu@intel.com>
> > Subject: [edk2-devel] [patch V4 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
> > Branch for this change:
> > https://github.com/dandanbi/edk2/tree/RegisterFilterLibV3
> >
> > 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.
> >
> > ---
> > V4:
> > Consume the MdeLibs.dsc.inc for RegisterFilterLibNull in dsc file
> > rather than in xxx.dsc.inc.
> > Following 2 patchws were updated for consuming MdeLibs.dsc.inc in
> > platform dsc file.
> >   ArmVirtPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> >   UnitTestFrameworkPkg: Consume MdeLibs.dsc.inc for RegisterFilterLib
> > Other patches are the same with V3.
> > ---
> > V3:
> > 1. Don't assume Library instances in MdeLibs.dsc.inc are must in
> > common [LibraryClasses] section.
> >    Can specify different [LibraryClasses] section for the libary
> > instances in MdeLibs.dsc.inc.
> > 2. Update plaftom dsc to consume the MdeLibs.dsc outside the
> > [LibraryClasses] section.
> > 3. Pick up the R-B for patch 1, patch 28, patch 29.
> >
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Sean Brogan <sean.brogan@microsoft.com>
> > Cc: Bret Barkelew <Bret.Barkelew@microsoft.com>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Laszlo Ersek <lersek@redhat.com>
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Cc: Jian J Wang <jian.j.wang@intel.com>
> > Cc: Sami Mujawar <Sami.Mujawar@arm.com>
> > Cc: Alexei Fedorov <Alexei.Fedorov@arm.com>
> > Cc: Andrew Fish <afish@apple.com>
> > Cc: Ray Ni <ray.ni@intel.com>
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Star Zeng <star.zeng@intel.com>
> > Cc: Zhichao Gao <zhichao.gao@intel.com>
> > Cc: Hao A Wu <hao.a.wu@intel.com>
> > Cc: Maurice Ma <maurice.ma@intel.com>
> > Cc: Guo Dong <guo.dong@intel.com>
> > Cc: Benjamin You <benjamin.you@intel.com>
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Maciej Rabeda <maciej.rabeda@linux.intel.com>
> > Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> > Cc: Siyuan Fu <siyuan.fu@intel.com>
> > Cc: Jordan Justen <jordan.l.justen@intel.com>
> > Cc: Supreeth Venkatesh <supreeth.venkatesh@arm.com>
> > Cc: Abner Chang <abner.chang@hpe.com>
> > Cc: Nickle Wang <nickle.wang@hpe.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                       |   5 +-
> >  DynamicTablesPkg/DynamicTablesPkg.dsc         |   2 +
> >  EmbeddedPkg/EmbeddedPkg.dsc                   |   5 +-
> >  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                        |  15 +
> >  MdePkg/MdePkg.dec                             |   7 +-
> >  MdePkg/MdePkg.dsc                             |   6 +-
> >  NetworkPkg/NetworkPkg.dsc                     |   4 +-
> >  OvmfPkg/AmdSev/AmdSevX64.dsc                  |   5 +-
> >  OvmfPkg/Bhyve/BhyveX64.dsc                    |   5 +-
> >  OvmfPkg/OvmfPkgIa32.dsc                       |   5 +-
> >  OvmfPkg/OvmfPkgIa32X64.dsc                    |   5 +-
> >  OvmfPkg/OvmfPkgX64.dsc                        |   5 +-
> >  OvmfPkg/OvmfXen.dsc                           |   5 +-
> >  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           |   5 +-
> >  UefiCpuPkg/UefiCpuPkg.dsc                     |   4 +-
> >  UefiPayloadPkg/UefiPayloadPkg.dsc             |   5 +-
> >  .../UnitTestFrameworkPkgTarget.dsc.inc        |   4 +-
> >  53 files changed, 1131 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-30 13:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <16707FF9B84D29FC.5213@groups.io>
2021-03-29  2:55 ` [edk2-devel] [patch V4 00/29] Add a new library class RegisterFilterLib in edk2 to filter/trace port IO/MMIO/MSR access Dandan Bi
2021-03-29  5:20   ` 回复: " gaoliming
     [not found] ` <1670B0B0AC65D7DB.438@groups.io>
2021-03-30 13:24   ` Dandan Bi

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox