svn commit: r360517 - in head/sys/contrib/dev/acpica: . common compiler components/debugger components/disassembler components/dispatcher components/executer include

[Conrad Meyer]

Thanks, this looks like a big improvement!

On Thu, Apr 30, 2020 at 18:26 Jung-uk Kim <jkim at freebsd.org> wrote:

> Author: jkim
> Date: Fri May  1 01:26:36 2020
> New Revision: 360517
> URL: https://svnweb.freebsd.org/changeset/base/360517
>
> Log:
>   MFV:  r360512
>
>   Merge ACPICA 20200430.
>
> --- head/sys/contrib/dev/acpica/changes.txt     Fri May  1 00:41:41 2020
>       (r360516)
> +++ head/sys/contrib/dev/acpica/changes.txt     Fri May  1 01:26:36 2020
>       (r360517)
> @@ -1,4 +1,61 @@
>  ----------------------------------------
> +
> +
> +30 April 2020. Summary of changes for version 20200430:
> ...
> +2) iASL Compiler/Disassembler and ACPICA tools:
> +
> +iASL DataTable Compiler:  Fixed a segfault on errors that aren't directly
> +associated with a field.
> +
> +Disassembler: has been made more resilient so that it will continue to
> +parse AML even if the AML generates ACPI namespace errors. This enables
> +iASL to disassemble some AML that may have been compiled using older
> +versions of iASL that no longer compile with newer versions of iASL.
> +
> +iASL: Fixed the required parameters for _NIH and _NIG. Previously, there
> +was a mixup where _NIG required one parameter and _NIH required zero
> +parameters. This change swaps these parameter requirements. Now it is
> +required that _NIH must be called with one parameter and _NIG requires
> +zero parameters.
> +
> +iASL: Allow use of undefined externals as long as they are protected by
> +an if (CondRefOf (...)) block when compiling multiple definition blocks.
> +
> +iASL: Fixed the type override behavior of named objects that are declared
> +as External. External declarations will no longer override the type of
> +the actual definition if it already exists.
> +
> +AcpiNames: Added setargv.obj to the MSVC 2017 link sequence to enable
> +command line wildcard support on Windows. Note: the AcpiNames utility is
> +essentially redundant with the AcpiExec utility (using the "namespace"
> +command) and is therefore deprecated. It will be removed in future
> +releases of ACPICA.
> +
> +Disassembler: ignore AE_ALREADY_EXISTS status when parsing create*
> +operators. The disassembler is intended to emit existing ASL code as-is.
> +Therefore, error messages emitted during disassembly should be ignored or
> +handled in a way such that the disassembler can continue to parse the
> +AML. This change ignores AE_ALREADY_EXISTS errors during the deferred Op
> +parsing for create operators in order to complete parsing ASL termlists.


All of the above should improve acpidump disassembler utility significantly.

Best,
Conrad