public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
From: "Sam Kaynor" <sam.kaynor@arm.com>
To: gaoliming <gaoliming@byosoft.com.cn>,
	"devel@edk2.groups.io" <devel@edk2.groups.io>
Cc: 'Ray Ni' <ray.ni@intel.com>,
	'Zhichao Gao' <zhichao.gao@intel.com>,
	'Michael D Kinney' <michael.d.kinney@intel.com>,
	'Zhiguang Liu' <zhiguang.liu@intel.com>,
	Stuart Yoder <Stuart.Yoder@arm.com>
Subject: Re: [edk2-devel] [PATCH v7 0/5] Adding support for veborse UEFI Table dumping to Dmem.c
Date: Thu, 9 May 2024 20:51:10 +0000	[thread overview]
Message-ID: <PAWPR08MB10239FEBC0ED180509E517A419AE62@PAWPR08MB10239.eurprd08.prod.outlook.com> (raw)
In-Reply-To: <035c01da9f5f$efa18360$cee48a20$@byosoft.com.cn>

Liming,

I have not made a Pull Request for this patch set, should I make one?

Thanks,
Sam

-----Original Message-----
From: gaoliming <gaoliming@byosoft.com.cn>
Sent: Sunday, May 5, 2024 9:49 PM
To: Sam Kaynor <Sam.Kaynor@arm.com>; devel@edk2.groups.io
Cc: 'Ray Ni' <ray.ni@intel.com>; 'Zhichao Gao' <zhichao.gao@intel.com>; 'Michael D Kinney' <michael.d.kinney@intel.com>; 'Zhiguang Liu' <zhiguang.liu@intel.com>; Stuart Yoder <Stuart.Yoder@arm.com>
Subject: 回复: [PATCH v7 0/5] Adding support for veborse UEFI Table dumping to Dmem.c

Sam:

Thanks for your update. This version patch is good to me. Reviewed-by:
Liming Gao <gaoliming@byosoft.com.cn>

Besides, is there a Pull Request for this patch set?

Thanks
Liming
> -----邮件原件-----
> 发件人: Sam Kaynor <Sam.Kaynor@arm.com>
> 发送时间: 2024年5月1日 22:58
> 收件人: devel@edk2.groups.io
> 抄送: Ray Ni <ray.ni@intel.com>; Zhichao Gao <zhichao.gao@intel.com>;
> Michael D Kinney <michael.d.kinney@intel.com>; Liming Gao
> <gaoliming@byosoft.com.cn>; Zhiguang Liu <zhiguang.liu@intel.com>;
> Stuart Yoder <stuart.yoder@arm.com>
> 主题: [PATCH v7 0/5] Adding support for veborse UEFI Table dumping to
> Dmem.c
>
> This adds an additional option to the dmem UEFI shell command for
> verbose dumping of a couple of UEFI tables.
>
> The tables specified are the RT Properties Table, the Image Execution
Table,
> and the Conformance Profiles table.
>
> Supporting objects are added in the .inf and .uni files relating to
> Dmem.c
and
> an additional header file for the Conformance Profiles table was added.
>
> REF: https://bugzilla.tianocore.org/show_bug.cgi?id=4352
>
> v6->v7:
> - Additional patch to separate UEFI and EBBR Guids
>
> Cc: Ray Ni <ray.ni@intel.com>
> Cc: Zhichao Gao <zhichao.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Liming Gao <gaoliming@byosoft.com.cn>
> Cc: Zhiguang Liu <zhiguang.liu@intel.com>
> Cc: Stuart Yoder <stuart.yoder@arm.com>
> Signed-off-by: Sam Kaynor <Sam.Kaynor@arm.com>
>
> Sam Kaynor (5):
>   ShellPkg: UefiShellDebug1CommandsLib: Dumping RT Properties in
> Dmem.c
>   ShellPkg: UefiShellDebug1CommandsLib: Image Execution Table in Dmem.c
>   MdePkg: Adding support for EFI_CONFORMANCE_PROFILE_TABLE
>   MdePkg: Adding EBBR EFI_CONFORMANCE_PROFILE_TABLE GUIDs
>   ShellPkg: UefiShellDebug1CommandsLib: Conformance Profiles in Dmem.c
>
>  MdePkg/MdePkg.dec
> |  10 +
>
> ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsL
> ib.inf |   4 +
>  MdePkg/Include/Guid/ConformanceProfiles.h
> |  63 ++++
>  ShellPkg/Library/UefiShellDebug1CommandsLib/Dmem.c
> | 321 ++++++++++++++++++--
>
> ShellPkg/Library/UefiShellDebug1CommandsLib/UefiShellDebug1CommandsL
> ib.uni |  31 +-
>  5 files changed, 402 insertions(+), 27 deletions(-)  create mode
> 100644 MdePkg/Include/Guid/ConformanceProfiles.h
>
> --
> 2.34.1



IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium. Thank you.


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#118771): https://edk2.groups.io/g/devel/message/118771
Mute This Topic: https://groups.io/mt/106009401/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-



  reply	other threads:[~2024-05-09 20:51 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-05-01 14:58 [edk2-devel] [PATCH v7 0/5] Adding support for veborse UEFI Table dumping to Dmem.c Sam Kaynor
2024-05-01 14:58 ` [edk2-devel] [PATCH v7 1/5] ShellPkg: UefiShellDebug1CommandsLib: Dumping RT Properties in Dmem.c Sam Kaynor
2024-05-01 14:58 ` [edk2-devel] [PATCH v7 2/5] ShellPkg: UefiShellDebug1CommandsLib: Image Execution Table " Sam Kaynor
2024-05-01 14:58 ` [edk2-devel] [PATCH v7 3/5] MdePkg: Adding support for EFI_CONFORMANCE_PROFILE_TABLE Sam Kaynor
2024-05-01 14:58 ` [edk2-devel] [PATCH v7 4/5] MdePkg: Adding EBBR EFI_CONFORMANCE_PROFILE_TABLE GUIDs Sam Kaynor
2024-05-01 14:58 ` [edk2-devel] [PATCH v7 5/5] ShellPkg: UefiShellDebug1CommandsLib: Conformance Profiles in Dmem.c Sam Kaynor
2024-05-06  2:48 ` [edk2-devel] 回复: [PATCH v7 0/5] Adding support for veborse UEFI Table dumping to Dmem.c gaoliming via groups.io
2024-05-09 20:51   ` Sam Kaynor [this message]
2024-05-10  0:47     ` 回复: [edk2-devel] " gaoliming via groups.io

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-list from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=PAWPR08MB10239FEBC0ED180509E517A419AE62@PAWPR08MB10239.eurprd08.prod.outlook.com \
    --to=devel@edk2.groups.io \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox