Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 1 May 2020 08:12:24 -0700
From:      Conrad Meyer <cem@freebsd.org>
To:        Jung-uk Kim <jkim@freebsd.org>
Cc:        src-committers@freebsd.org, svn-src-all@freebsd.org,  svn-src-head@freebsd.org
Subject:   Re: svn commit: r360517 - in head/sys/contrib/dev/acpica: . common compiler components/debugger components/disassembler components/dispatcher components/executer include
Message-ID:  <CAG6CVpVCJXCPm9-75kFOkG2UWhaZhb0h-=X7OdhSu-ys8gz2Jw@mail.gmail.com>
In-Reply-To: <202005010126.0411QaEF047318@repo.freebsd.org>
References:  <202005010126.0411QaEF047318@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
Thanks, this looks like a big improvement!

On Thu, Apr 30, 2020 at 18:26 Jung-uk Kim <jkim@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



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAG6CVpVCJXCPm9-75kFOkG2UWhaZhb0h-=X7OdhSu-ys8gz2Jw>