Re: [PATCH] ShellPkg: Add acpiview tool to dump ACPI tables

[Evan Lloyd <Evan.Lloyd@arm.com>]

Hi Jiewen.
I hope I have our forename there, not your surname - it can be difficult to tell :-)
I've trimmed a lot out  of the history below, for clarity.

>
>From: Yao, Jiewen [mailto:jiewen.yao@intel.com]
>Sent: 20 December 2016 16:35
>To: Evan Lloyd; Ni, Ruiyu; Carsey, Jaben; Sami Mujawar
>Cc: Carsey, Jaben; edk2-devel@ml01.01.org; Leif Lindholm; ard.biesheuvel@linaro.org
>Subject: RE: [edk2] [PATCH] ShellPkg: Add acpiview tool to dump ACPI tables
>
>Thank you, Lloyd.
>
>Comment inline.
>
...
>[Jiewen] Got it.
>As you mention, I hope it can be in SHELL spec, too.
>
>So that we can have a standard way to dump all table information.
>
>For X86 system, I have written similar tool to dump X86 related info.
>I just check in to https://github.com/jyao1/EdkiiShellTool/tree/master/AcpiToolPkg.
>It is also BSD license code. I tried by best to dump info for *all* the ACPI table.
>But I do not validate ARM system. We complement to each other. ?

Yes, we obviously need to get together over this.

>...
>[Jiewen] Let me summarize current status:
>1)       We know there is strong requirement to dump ACPI table in UEFI shell environment, for debug purposes.
>2)       UEFI shell specification does not contain any ACPI dump command. (Although it has SmbiosDump)
>3)       ACPICA.ORG has sample code to dump ACPI.
>> > >Binary can be downloaded from: https://acpica.org/downloads/uefi-support
>> > >Source can be downloaded from: https://github.com/acpica/acpica
>4)       This patch provides dump ACPI information with consistency check. But limitation is: it only supports the limited table on ARM platform. NO support for X86 platform.
>5)       I have similar code to dump ACPI. But the limitation is: it only validated on X86 platform. It is not validated on ARM platform.
>
>
>So I would like to propose:
>1)       DOCUMENTATION: Can we co-work to submit an ECR for SHELL, to add AcpiDump command? So that people may get standard dump log in the future.

This sounds fine, but I'd suggest getting it into the Shell spec is much lower
priority than making the tool available.  We believe the checking aspect will
help people detect errors at an early stage, so want to "get it out there".

>2)       CODE: It seems both of our code is POC quality and has partial validation only. It might not be suitable to check in EDKII immediately
>I would like to propose we co-work in EDKII staging tree - https://github.com/tianocore/edk2-staging/ (We have 4 features there.)

That sounds excellent.  It will be a great thing to have ARM/Intel (or Intel/ARM if you prefer) cooperation on this.

>We can align our design, then I can help validate X86 and you can help validate ARM.
>Then we can submit a complete patch to EDKII ShellPkg.
>
>What about your idea?

I think this potentially helps everyone in UEFI.
Sadly, we are about to shut down for Christmas, and will not be back until 2017-01-03.  (I'm not really sad about it.)
From looking at your code, I think we will be able to learn a lot about EDK2 programming from you.
I suggest we can discuss the design and code merge aspects off list. (And make details of the results public in condensed form.)

Do I need to ask anyone to give me commit rights to https://github.com/tianocore/edk2-staging?
Or is it pull request based?

Regards, and Merry Christmas,
Evan

>
 Thank you
>> Yao Jiewen
>
>>> > > ...
>> > > >> Guid("CE165669-3EF3-493F-B85D-6190EE5B9759")
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.