Hi Michael, I cannot find the shellpkg change of this patch set. Can you share your poll request for review?
Thanks, Zhichao > -----Original Message----- > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Michael > Kubacki > Sent: Tuesday, October 24, 2023 2:16 AM > To: devel@edk2.groups.io > Cc: Anatol Belski <anbel...@linux.microsoft.com>; Anthony Perard > <anthony.per...@citrix.com>; Bi, Dandan <dandan...@intel.com>; Gerd > Hoffmann <kra...@redhat.com>; Wu, Hao A <hao.a...@intel.com>; Wang, > Jian J <jian.j.w...@intel.com>; Jianyong Wu <jianyong...@arm.com>; Yao, > Jiewen <jiewen....@intel.com>; Justen, Jordan L > <jordan.l.jus...@intel.com>; Julien Grall <jul...@xen.org>; Gao, Liming > <gaolim...@byosoft.com.cn>; Kinney, Michael D > <michael.d.kin...@intel.com>; Gao, Zhichao <zhichao....@intel.com> > Subject: Re: [edk2-devel] [PATCH v1 0/3] Add Variable Policy Audit and Shell > Command > > A reminder to review this series. > > Regards, > Michael > > On 9/19/2023 10:32 AM, Michael Kubacki wrote: > > From: Michael Kubacki <michael.kuba...@microsoft.com> > > > > Adds a new module (dynamic shell command) to ShellPkg that lists > > variable policy information for all UEFI variables on the system. > > > > Some other UEFI variable related functionality is also included to > > give a greater sense of platform UEFI variable state. > > > > Like all dynamic shell commands, a platform only needs to include > > VariablePolicyDynamicCommand.inf in their flash image to have the > > command registered in their UEFI shell. > > > > The shell command is added to OvmfPkg so it is available in an easily > > obtainable virtual platform. > > > > This is being made available to ease auditing and enabling of UEFI > > variable policy to encourage its adoption for securing UEFI variables. > > > > --- > > > > Command Help: > > > > Lists UEFI variable policy information. > > > > VARPOLICY [-p] [-s] [-v] > > > > -p - The policy flag will print variable policy info for each variable. > > > > -s - The stats flag will print overall UEFI variable policy statistics. > > > > -v - The verbose flag indicates all known information should be printed. > > > > This includes a dump of the corresponding UEFI variable data in > > addition to all other UEFI variable policy information. > > > > EXAMPLES: > > > > * To dump all active UEFI variables: > > fs0:\> varpolicy > > > > * To include UEFI variable policy information: > > varpolicy -p > > > > * To include UEFI variable statistics: > > varpolicy -s > > > > Press ENTER to continue or 'Q' break: > > > > --- > > > > Images showing example output are available in the PR: > > https://github.com/tianocore/edk2/pull/4835 > > > > Cc: Anatol Belski <anbel...@linux.microsoft.com> > > Cc: Anthony Perard <anthony.per...@citrix.com> > > Cc: Dandan Bi <dandan...@intel.com> > > Cc: Gerd Hoffmann <kra...@redhat.com> > > Cc: Hao A Wu <hao.a...@intel.com> > > Cc: Jian J Wang <jian.j.w...@intel.com> > > Cc: Jianyong Wu <jianyong...@arm.com> > > Cc: Jiewen Yao <jiewen....@intel.com> > > Cc: Jordan Justen <jordan.l.jus...@intel.com> > > Cc: Julien Grall <jul...@xen.org> > > Cc: Liming Gao <gaolim...@byosoft.com.cn> > > Cc: Michael D Kinney <michael.d.kin...@intel.com> > > Cc: Zhichao Gao <zhichao....@intel.com> > > > > Michael Kubacki (3): > > MdeModulePkg/VariablePolicy: Add more granular variable policy > > querying > > ShellPkg: Add varpolicy dynamic shell command > > OvmfPkg: Add varpolicy shell command > > > > MdeModulePkg/Library/VarCheckPolicyLib/VarCheckPolicyLib.c > | 174 +++- > > MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.c > | 304 ++++++ > > MdeModulePkg/Universal/Variable/RuntimeDxe/VariableDxe.c > | 4 +- > > MdeModulePkg/Universal/Variable/RuntimeDxe/VariablePolicySmmDxe.c > | 346 ++++++- > > > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.c | 1009 ++++++++++++++++++++ > > MdeModulePkg/Include/Guid/VarCheckPolicyMmi.h > | 39 +- > > MdeModulePkg/Include/Library/VariablePolicyLib.h > > | > 107 +++ > > MdeModulePkg/Include/Protocol/VariablePolicy.h > > | > 133 ++- > > MdeModulePkg/Library/VariablePolicyLib/VariablePolicyLib.inf > | 1 + > > OvmfPkg/CloudHv/CloudHvX64.dsc > > | 4 + > > OvmfPkg/Microvm/MicrovmX64.dsc > > | 4 + > > OvmfPkg/OvmfPkgIa32.dsc > > | 4 + > > OvmfPkg/OvmfPkgIa32X64.dsc > > | 4 + > > OvmfPkg/OvmfPkgX64.dsc > > | 4 + > > OvmfPkg/OvmfXen.dsc > > | 4 + > > > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.h | 94 ++ > > > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.inf | 56 ++ > > > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.uni | 86 ++ > > ShellPkg/ShellPkg.dsc > > | 1 + > > 19 files changed, 2332 insertions(+), 46 deletions(-) > > create mode 100644 > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.c > > create mode 100644 > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.h > > create mode 100644 > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > DynamicCommand.inf > > create mode 100644 > > > ShellPkg/DynamicCommand/VariablePolicyDynamicCommand/VariablePolicy > Dyn > > amicCommand.uni > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#109963): https://edk2.groups.io/g/devel/message/109963 Mute This Topic: https://groups.io/mt/102141559/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-