ACPICA: Implement deferred resolution of reference package elements
authorBob Moore <robert.moore@intel.com>
Thu, 3 Aug 2017 06:27:22 +0000 (14:27 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 3 Aug 2017 21:34:17 +0000 (23:34 +0200)
ACPICA commit 0c08790c56fdf0dc081ae869495a09d8c4230854

This change defers the resolution of package elements that
are named references until after the entire namespace has been
loaded from the definition block. This allows such references
to be in fact forward references for both module level code
and control methods.

Link: https://github.com/acpica/acpica/commit/0c08790c
Signed-off-by: Bob Moore <robert.moore@intel.com>
Signed-off-by: Lv Zheng <lv.zheng@intel.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
16 files changed:
drivers/acpi/acpica/Makefile
drivers/acpi/acpica/acdispat.h
drivers/acpi/acpica/aclocal.h
drivers/acpi/acpica/acobject.h
drivers/acpi/acpica/acutils.h
drivers/acpi/acpica/dsobject.c
drivers/acpi/acpica/dsopcode.c
drivers/acpi/acpica/dspkginit.c [new file with mode: 0644]
drivers/acpi/acpica/nsarguments.c
drivers/acpi/acpica/nsinit.c
drivers/acpi/acpica/nsnames.c
drivers/acpi/acpica/psloop.c
drivers/acpi/acpica/utmisc.c
drivers/acpi/acpica/utobject.c
drivers/acpi/acpica/utstate.c
drivers/acpi/acpica/uttrack.c

index b125bdd3d58bd097a8698109dc242b627aa2e41a..1709551bc4aa8bf518ba4eac1e30c2996692d5ca 100644 (file)
@@ -18,6 +18,7 @@ acpi-y :=             \
        dsmthdat.o      \
        dsobject.o      \
        dsopcode.o      \
+       dspkginit.o     \
        dsutils.o       \
        dswexec.o       \
        dswload.o       \
index 0d95c85cce066d5345f95581709adf9826cc0d36..f8f3a6e74128a05e379808003939cfcebb85520d 100644 (file)
@@ -237,6 +237,11 @@ acpi_ds_initialize_objects(u32 table_index,
  * dsobject - Parser/Interpreter interface - object initialization and conversion
  */
 acpi_status
+acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
+                             union acpi_parse_object *op,
+                             union acpi_operand_object **obj_desc_ptr);
+
+acpi_status
 acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,
                                  union acpi_parse_object *op,
                                  u32 buffer_length,
@@ -258,6 +263,14 @@ acpi_ds_create_node(struct acpi_walk_state *walk_state,
                    struct acpi_namespace_node *node,
                    union acpi_parse_object *op);
 
+/*
+ * dspkginit - Package object initialization
+ */
+acpi_status
+acpi_ds_init_package_element(u8 object_type,
+                            union acpi_operand_object *source_object,
+                            union acpi_generic_state *state, void *context);
+
 /*
  * dsutils - Parser/Interpreter interface utility routines
  */
index bae943b5f377356b1f927a53bdeadd3cc0e05c88..0d45b8bb1678945eb296cdabce11f229cb349b65 100644 (file)
@@ -605,7 +605,7 @@ struct acpi_update_state {
  * Pkg state - used to traverse nested package structures
  */
 struct acpi_pkg_state {
-       ACPI_STATE_COMMON u16 index;
+       ACPI_STATE_COMMON u32 index;
        union acpi_operand_object *source_object;
        union acpi_operand_object *dest_object;
        struct acpi_walk_state *walk_state;
index 549f227348cd8874a7471cab00a7de588cde78ae..5226146190bf92124a42cf66cbf4b9013a0e80a5 100644 (file)
@@ -339,11 +339,12 @@ struct acpi_object_addr_handler {
 struct acpi_object_reference {
        ACPI_OBJECT_COMMON_HEADER u8 class;     /* Reference Class */
        u8 target_type;         /* Used for Index Op */
-       u8 reserved;
+       u8 resolved;            /* Reference has been resolved to a value */
        void *object;           /* name_op=>HANDLE to obj, index_op=>union acpi_operand_object */
        struct acpi_namespace_node *node;       /* ref_of or Namepath */
        union acpi_operand_object **where;      /* Target of Index */
        u8 *index_pointer;      /* Used for Buffers and Strings */
+       u8 *aml;                /* Used for deferred resolution of the ref */
        u32 value;              /* Used for Local/Arg/Index/ddb_handle */
 };
 
index 0a56014055849962ff7411b54f8aafdcb28e2f56..745134ade35fd72972efd708d0cb66c550a46037 100644 (file)
@@ -516,7 +516,7 @@ union acpi_generic_state *acpi_ut_create_update_state(union acpi_operand_object
 
 union acpi_generic_state *acpi_ut_create_pkg_state(void *internal_object,
                                                   void *external_object,
-                                                  u16 index);
+                                                  u32 index);
 
 acpi_status
 acpi_ut_create_update_state_and_push(union acpi_operand_object *object,
index 7df3152ed8569dd1a60bb96dd0f36d23f812c10e..82448551781b43bf3d276df6ee9378416cb2a08b 100644 (file)
 #define _COMPONENT          ACPI_DISPATCHER
 ACPI_MODULE_NAME("dsobject")
 
-/* Local prototypes */
-static acpi_status
-acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
-                             union acpi_parse_object *op,
-                             union acpi_operand_object **obj_desc_ptr);
-
 #ifndef ACPI_NO_METHOD_EXECUTION
 /*******************************************************************************
  *
@@ -73,15 +67,13 @@ acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
  *              Simple objects are any objects other than a package object!
  *
  ******************************************************************************/
-
-static acpi_status
+acpi_status
 acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
                              union acpi_parse_object *op,
                              union acpi_operand_object **obj_desc_ptr)
 {
        union acpi_operand_object *obj_desc;
        acpi_status status;
-       acpi_object_type type;
 
        ACPI_FUNCTION_TRACE(ds_build_internal_object);
 
@@ -89,140 +81,47 @@ acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
        if (op->common.aml_opcode == AML_INT_NAMEPATH_OP) {
                /*
                 * This is a named object reference. If this name was
-                * previously looked up in the namespace, it was stored in this op.
-                * Otherwise, go ahead and look it up now
+                * previously looked up in the namespace, it was stored in
+                * this op. Otherwise, go ahead and look it up now
                 */
                if (!op->common.node) {
-                       status = acpi_ns_lookup(walk_state->scope_info,
-                                               op->common.value.string,
-                                               ACPI_TYPE_ANY,
-                                               ACPI_IMODE_EXECUTE,
-                                               ACPI_NS_SEARCH_PARENT |
-                                               ACPI_NS_DONT_OPEN_SCOPE, NULL,
-                                               ACPI_CAST_INDIRECT_PTR(struct
-                                                                      acpi_namespace_node,
-                                                                      &(op->
-                                                                        common.
-                                                                        node)));
-                       if (ACPI_FAILURE(status)) {
-
-                               /* Check if we are resolving a named reference within a package */
-
-                               if ((status == AE_NOT_FOUND)
-                                   && (acpi_gbl_enable_interpreter_slack)
-                                   &&
-                                   ((op->common.parent->common.aml_opcode ==
-                                     AML_PACKAGE_OP)
-                                    || (op->common.parent->common.aml_opcode ==
-                                        AML_VARIABLE_PACKAGE_OP))) {
-                                       /*
-                                        * We didn't find the target and we are populating elements
-                                        * of a package - ignore if slack enabled. Some ASL code
-                                        * contains dangling invalid references in packages and
-                                        * expects that no exception will be issued. Leave the
-                                        * element as a null element. It cannot be used, but it
-                                        * can be overwritten by subsequent ASL code - this is
-                                        * typically the case.
-                                        */
-                                       ACPI_DEBUG_PRINT((ACPI_DB_INFO,
-                                                         "Ignoring unresolved reference in package [%4.4s]\n",
-                                                         walk_state->
-                                                         scope_info->scope.
-                                                         node->name.ascii));
-
-                                       return_ACPI_STATUS(AE_OK);
-                               } else {
-                                       ACPI_ERROR_NAMESPACE(op->common.value.
-                                                            string, status);
-                               }
-
-                               return_ACPI_STATUS(status);
-                       }
-               }
-
-               /* Special object resolution for elements of a package */
-
-               if ((op->common.parent->common.aml_opcode == AML_PACKAGE_OP) ||
-                   (op->common.parent->common.aml_opcode ==
-                    AML_VARIABLE_PACKAGE_OP)) {
-                       /*
-                        * Attempt to resolve the node to a value before we insert it into
-                        * the package. If this is a reference to a common data type,
-                        * resolve it immediately. According to the ACPI spec, package
-                        * elements can only be "data objects" or method references.
-                        * Attempt to resolve to an Integer, Buffer, String or Package.
-                        * If cannot, return the named reference (for things like Devices,
-                        * Methods, etc.) Buffer Fields and Fields will resolve to simple
-                        * objects (int/buf/str/pkg).
-                        *
-                        * NOTE: References to things like Devices, Methods, Mutexes, etc.
-                        * will remain as named references. This behavior is not described
-                        * in the ACPI spec, but it appears to be an oversight.
-                        */
-                       obj_desc =
-                           ACPI_CAST_PTR(union acpi_operand_object,
-                                         op->common.node);
-
-                       status =
-                           acpi_ex_resolve_node_to_value(ACPI_CAST_INDIRECT_PTR
-                                                         (struct
-                                                          acpi_namespace_node,
-                                                          &obj_desc),
-                                                         walk_state);
-                       if (ACPI_FAILURE(status)) {
-                               return_ACPI_STATUS(status);
-                       }
-
-                       /*
-                        * Special handling for Alias objects. We need to setup the type
-                        * and the Op->Common.Node to point to the Alias target. Note,
-                        * Alias has at most one level of indirection internally.
-                        */
-                       type = op->common.node->type;
-                       if (type == ACPI_TYPE_LOCAL_ALIAS) {
-                               type = obj_desc->common.type;
-                               op->common.node =
-                                   ACPI_CAST_PTR(struct acpi_namespace_node,
-                                                 op->common.node->object);
-                       }
-
-                       switch (type) {
-                               /*
-                                * For these types, we need the actual node, not the subobject.
-                                * However, the subobject did not get an extra reference count above.
-                                *
-                                * TBD: should ex_resolve_node_to_value be changed to fix this?
-                                */
-                       case ACPI_TYPE_DEVICE:
-                       case ACPI_TYPE_THERMAL:
-
-                               acpi_ut_add_reference(op->common.node->object);
 
-                               /*lint -fallthrough */
-                               /*
-                                * For these types, we need the actual node, not the subobject.
-                                * The subobject got an extra reference count in ex_resolve_node_to_value.
-                                */
-                       case ACPI_TYPE_MUTEX:
-                       case ACPI_TYPE_METHOD:
-                       case ACPI_TYPE_POWER:
-                       case ACPI_TYPE_PROCESSOR:
-                       case ACPI_TYPE_EVENT:
-                       case ACPI_TYPE_REGION:
-
-                               /* We will create a reference object for these types below */
-                               break;
+                       /* Check if we are resolving a named reference within a package */
 
-                       default:
+                       if ((op->common.parent->common.aml_opcode ==
+                            AML_PACKAGE_OP)
+                           || (op->common.parent->common.aml_opcode ==
+                               AML_VARIABLE_PACKAGE_OP)) {
                                /*
-                                * All other types - the node was resolved to an actual
-                                * object, we are done.
+                                * We won't resolve package elements here, we will do this
+                                * after all ACPI tables are loaded into the namespace. This
+                                * behavior supports both forward references to named objects
+                                * and external references to objects in other tables.
                                 */
-                               goto exit;
+                               goto create_new_object;
+                       } else {
+                               status = acpi_ns_lookup(walk_state->scope_info,
+                                                       op->common.value.string,
+                                                       ACPI_TYPE_ANY,
+                                                       ACPI_IMODE_EXECUTE,
+                                                       ACPI_NS_SEARCH_PARENT |
+                                                       ACPI_NS_DONT_OPEN_SCOPE,
+                                                       NULL,
+                                                       ACPI_CAST_INDIRECT_PTR
+                                                       (struct
+                                                        acpi_namespace_node,
+                                                        &(op->common.node)));
+                               if (ACPI_FAILURE(status)) {
+                                       ACPI_ERROR_NAMESPACE(op->common.value.
+                                                            string, status);
+                                       return_ACPI_STATUS(status);
+                               }
                        }
                }
        }
 
+create_new_object:
+
        /* Create and init a new internal ACPI object */
 
        obj_desc = acpi_ut_create_internal_object((acpi_ps_get_opcode_info
@@ -240,7 +139,27 @@ acpi_ds_build_internal_object(struct acpi_walk_state *walk_state,
                return_ACPI_STATUS(status);
        }
 
-exit:
+       /*
+        * Handling for unresolved package reference elements.
+        * These are elements that are namepaths.
+        */
+       if ((op->common.parent->common.aml_opcode == AML_PACKAGE_OP) ||
+           (op->common.parent->common.aml_opcode == AML_VARIABLE_PACKAGE_OP)) {
+               obj_desc->reference.resolved = TRUE;
+
+               if ((op->common.aml_opcode == AML_INT_NAMEPATH_OP) &&
+                   !obj_desc->reference.node) {
+                       /*
+                        * Name was unresolved above.
+                        * Get the prefix node for later lookup
+                        */
+                       obj_desc->reference.node =
+                           walk_state->scope_info->scope.node;
+                       obj_desc->reference.aml = op->common.aml;
+                       obj_desc->reference.resolved = FALSE;
+               }
+       }
+
        *obj_desc_ptr = obj_desc;
        return_ACPI_STATUS(status);
 }
@@ -349,200 +268,6 @@ acpi_ds_build_internal_buffer_obj(struct acpi_walk_state *walk_state,
        return_ACPI_STATUS(AE_OK);
 }
 
-/*******************************************************************************
- *
- * FUNCTION:    acpi_ds_build_internal_package_obj
- *
- * PARAMETERS:  walk_state      - Current walk state
- *              op              - Parser object to be translated
- *              element_count   - Number of elements in the package - this is
- *                                the num_elements argument to Package()
- *              obj_desc_ptr    - Where the ACPI internal object is returned
- *
- * RETURN:      Status
- *
- * DESCRIPTION: Translate a parser Op package object to the equivalent
- *              namespace object
- *
- * NOTE: The number of elements in the package will be always be the num_elements
- * count, regardless of the number of elements in the package list. If
- * num_elements is smaller, only that many package list elements are used.
- * if num_elements is larger, the Package object is padded out with
- * objects of type Uninitialized (as per ACPI spec.)
- *
- * Even though the ASL compilers do not allow num_elements to be smaller
- * than the Package list length (for the fixed length package opcode), some
- * BIOS code modifies the AML on the fly to adjust the num_elements, and
- * this code compensates for that. This also provides compatibility with
- * other AML interpreters.
- *
- ******************************************************************************/
-
-acpi_status
-acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
-                                  union acpi_parse_object *op,
-                                  u32 element_count,
-                                  union acpi_operand_object **obj_desc_ptr)
-{
-       union acpi_parse_object *arg;
-       union acpi_parse_object *parent;
-       union acpi_operand_object *obj_desc = NULL;
-       acpi_status status = AE_OK;
-       u32 i;
-       u16 index;
-       u16 reference_count;
-
-       ACPI_FUNCTION_TRACE(ds_build_internal_package_obj);
-
-       /* Find the parent of a possibly nested package */
-
-       parent = op->common.parent;
-       while ((parent->common.aml_opcode == AML_PACKAGE_OP) ||
-              (parent->common.aml_opcode == AML_VARIABLE_PACKAGE_OP)) {
-               parent = parent->common.parent;
-       }
-
-       /*
-        * If we are evaluating a Named package object "Name (xxxx, Package)",
-        * the package object already exists, otherwise it must be created.
-        */
-       obj_desc = *obj_desc_ptr;
-       if (!obj_desc) {
-               obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_PACKAGE);
-               *obj_desc_ptr = obj_desc;
-               if (!obj_desc) {
-                       return_ACPI_STATUS(AE_NO_MEMORY);
-               }
-
-               obj_desc->package.node = parent->common.node;
-       }
-
-       /*
-        * Allocate the element array (array of pointers to the individual
-        * objects) based on the num_elements parameter. Add an extra pointer slot
-        * so that the list is always null terminated.
-        */
-       obj_desc->package.elements = ACPI_ALLOCATE_ZEROED(((acpi_size)
-                                                          element_count +
-                                                          1) * sizeof(void *));
-
-       if (!obj_desc->package.elements) {
-               acpi_ut_delete_object_desc(obj_desc);
-               return_ACPI_STATUS(AE_NO_MEMORY);
-       }
-
-       obj_desc->package.count = element_count;
-
-       /*
-        * Initialize the elements of the package, up to the num_elements count.
-        * Package is automatically padded with uninitialized (NULL) elements
-        * if num_elements is greater than the package list length. Likewise,
-        * Package is truncated if num_elements is less than the list length.
-        */
-       arg = op->common.value.arg;
-       arg = arg->common.next;
-       for (i = 0; arg && (i < element_count); i++) {
-               if (arg->common.aml_opcode == AML_INT_RETURN_VALUE_OP) {
-                       if (arg->common.node->type == ACPI_TYPE_METHOD) {
-                               /*
-                                * A method reference "looks" to the parser to be a method
-                                * invocation, so we special case it here
-                                */
-                               arg->common.aml_opcode = AML_INT_NAMEPATH_OP;
-                               status =
-                                   acpi_ds_build_internal_object(walk_state,
-                                                                 arg,
-                                                                 &obj_desc->
-                                                                 package.
-                                                                 elements[i]);
-                       } else {
-                               /* This package element is already built, just get it */
-
-                               obj_desc->package.elements[i] =
-                                   ACPI_CAST_PTR(union acpi_operand_object,
-                                                 arg->common.node);
-                       }
-               } else {
-                       status =
-                           acpi_ds_build_internal_object(walk_state, arg,
-                                                         &obj_desc->package.
-                                                         elements[i]);
-               }
-
-               if (*obj_desc_ptr) {
-
-                       /* Existing package, get existing reference count */
-
-                       reference_count =
-                           (*obj_desc_ptr)->common.reference_count;
-                       if (reference_count > 1) {
-
-                               /* Make new element ref count match original ref count */
-
-                               for (index = 0; index < (reference_count - 1);
-                                    index++) {
-                                       acpi_ut_add_reference((obj_desc->
-                                                              package.
-                                                              elements[i]));
-                               }
-                       }
-               }
-
-               arg = arg->common.next;
-       }
-
-       /* Check for match between num_elements and actual length of package_list */
-
-       if (arg) {
-               /*
-                * num_elements was exhausted, but there are remaining elements in the
-                * package_list. Truncate the package to num_elements.
-                *
-                * Note: technically, this is an error, from ACPI spec: "It is an error
-                * for NumElements to be less than the number of elements in the
-                * PackageList". However, we just print a message and
-                * no exception is returned. This provides Windows compatibility. Some
-                * BIOSs will alter the num_elements on the fly, creating this type
-                * of ill-formed package object.
-                */
-               while (arg) {
-                       /*
-                        * We must delete any package elements that were created earlier
-                        * and are not going to be used because of the package truncation.
-                        */
-                       if (arg->common.node) {
-                               acpi_ut_remove_reference(ACPI_CAST_PTR
-                                                        (union
-                                                         acpi_operand_object,
-                                                         arg->common.node));
-                               arg->common.node = NULL;
-                       }
-
-                       /* Find out how many elements there really are */
-
-                       i++;
-                       arg = arg->common.next;
-               }
-
-               ACPI_INFO(("Actual Package length (%u) is larger than "
-                          "NumElements field (%u), truncated",
-                          i, element_count));
-       } else if (i < element_count) {
-               /*
-                * Arg list (elements) was exhausted, but we did not reach num_elements count.
-                * Note: this is not an error, the package is padded out with NULLs.
-                */
-               ACPI_DEBUG_PRINT((ACPI_DB_INFO,
-                                 "Package List length (%u) smaller than NumElements "
-                                 "count (%u), padded with null elements\n",
-                                 i, element_count));
-       }
-
-       obj_desc->package.flags |= AOPOBJ_DATA_VALID;
-       op->common.node = ACPI_CAST_PTR(struct acpi_namespace_node, obj_desc);
-       return_ACPI_STATUS(status);
-}
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ds_create_node
@@ -662,11 +387,20 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
 
        case ACPI_TYPE_PACKAGE:
                /*
-                * Defer evaluation of Package term_arg operand
+                * Defer evaluation of Package term_arg operand and all
+                * package elements. (01/2017): We defer the element
+                * resolution to allow forward references from the package
+                * in order to provide compatibility with other ACPI
+                * implementations.
                 */
                obj_desc->package.node =
                    ACPI_CAST_PTR(struct acpi_namespace_node,
                                  walk_state->operands[0]);
+
+               if (!op->named.data) {
+                       return_ACPI_STATUS(AE_OK);
+               }
+
                obj_desc->package.aml_start = op->named.data;
                obj_desc->package.aml_length = op->named.length;
                break;
@@ -818,9 +552,11 @@ acpi_ds_init_object_from_op(struct acpi_walk_state *walk_state,
                                /* Node was saved in Op */
 
                                obj_desc->reference.node = op->common.node;
-                               obj_desc->reference.object =
-                                   op->common.node->object;
                                obj_desc->reference.class = ACPI_REFCLASS_NAME;
+                               if (op->common.node) {
+                                       obj_desc->reference.object =
+                                           op->common.node->object;
+                               }
                                break;
 
                        case AML_DEBUG_OP:
index dfc3c25a083dde54383e8d3dadb2e63eb42095de..0336df7ac47dd1f4af1f732593ed41ab884de1ba 100644 (file)
@@ -599,6 +599,15 @@ acpi_ds_eval_data_object_operands(struct acpi_walk_state *walk_state,
         */
        walk_state->operand_index = walk_state->num_operands;
 
+       /* Ignore if child is not valid */
+
+       if (!op->common.value.arg) {
+               ACPI_ERROR((AE_INFO,
+                           "Dispatch: Missing child while executing TermArg for %X",
+                           op->common.aml_opcode));
+               return_ACPI_STATUS(AE_OK);
+       }
+
        status = acpi_ds_create_operand(walk_state, op->common.value.arg, 1);
        if (ACPI_FAILURE(status)) {
                return_ACPI_STATUS(status);
diff --git a/drivers/acpi/acpica/dspkginit.c b/drivers/acpi/acpica/dspkginit.c
new file mode 100644 (file)
index 0000000..6d487ed
--- /dev/null
@@ -0,0 +1,496 @@
+/******************************************************************************
+ *
+ * Module Name: dspkginit - Completion of deferred package initialization
+ *
+ *****************************************************************************/
+
+/*
+ * Copyright (C) 2000 - 2017, Intel Corp.
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions
+ * are met:
+ * 1. Redistributions of source code must retain the above copyright
+ *    notice, this list of conditions, and the following disclaimer,
+ *    without modification.
+ * 2. Redistributions in binary form must reproduce at minimum a disclaimer
+ *    substantially similar to the "NO WARRANTY" disclaimer below
+ *    ("Disclaimer") and any redistribution must be conditioned upon
+ *    including a substantially similar Disclaimer requirement for further
+ *    binary redistribution.
+ * 3. Neither the names of the above-listed copyright holders nor the names
+ *    of any contributors may be used to endorse or promote products derived
+ *    from this software without specific prior written permission.
+ *
+ * Alternatively, this software may be distributed under the terms of the
+ * GNU General Public License ("GPL") version 2 as published by the Free
+ * Software Foundation.
+ *
+ * NO WARRANTY
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
+ * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
+ * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTIBILITY AND FITNESS FOR
+ * A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
+ * HOLDERS OR CONTRIBUTORS BE LIABLE FOR SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS
+ * OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION)
+ * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT,
+ * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING
+ * IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
+ * POSSIBILITY OF SUCH DAMAGES.
+ */
+
+#include <acpi/acpi.h>
+#include "accommon.h"
+#include "acnamesp.h"
+#include "amlcode.h"
+#include "acdispat.h"
+#include "acinterp.h"
+
+#define _COMPONENT          ACPI_NAMESPACE
+ACPI_MODULE_NAME("dspkginit")
+
+/* Local prototypes */
+static void
+acpi_ds_resolve_package_element(union acpi_operand_object **element);
+
+/*******************************************************************************
+ *
+ * FUNCTION:    acpi_ds_build_internal_package_obj
+ *
+ * PARAMETERS:  walk_state      - Current walk state
+ *              op              - Parser object to be translated
+ *              element_count   - Number of elements in the package - this is
+ *                                the num_elements argument to Package()
+ *              obj_desc_ptr    - Where the ACPI internal object is returned
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Translate a parser Op package object to the equivalent
+ *              namespace object
+ *
+ * NOTE: The number of elements in the package will be always be the num_elements
+ * count, regardless of the number of elements in the package list. If
+ * num_elements is smaller, only that many package list elements are used.
+ * if num_elements is larger, the Package object is padded out with
+ * objects of type Uninitialized (as per ACPI spec.)
+ *
+ * Even though the ASL compilers do not allow num_elements to be smaller
+ * than the Package list length (for the fixed length package opcode), some
+ * BIOS code modifies the AML on the fly to adjust the num_elements, and
+ * this code compensates for that. This also provides compatibility with
+ * other AML interpreters.
+ *
+ ******************************************************************************/
+
+acpi_status
+acpi_ds_build_internal_package_obj(struct acpi_walk_state *walk_state,
+                                  union acpi_parse_object *op,
+                                  u32 element_count,
+                                  union acpi_operand_object **obj_desc_ptr)
+{
+       union acpi_parse_object *arg;
+       union acpi_parse_object *parent;
+       union acpi_operand_object *obj_desc = NULL;
+       acpi_status status = AE_OK;
+       u16 reference_count;
+       u32 index;
+       u32 i;
+
+       ACPI_FUNCTION_TRACE(ds_build_internal_package_obj);
+
+       /* Find the parent of a possibly nested package */
+
+       parent = op->common.parent;
+       while ((parent->common.aml_opcode == AML_PACKAGE_OP) ||
+              (parent->common.aml_opcode == AML_VARIABLE_PACKAGE_OP)) {
+               parent = parent->common.parent;
+       }
+
+       /*
+        * If we are evaluating a Named package object of the form:
+        *      Name (xxxx, Package)
+        * the package object already exists, otherwise it must be created.
+        */
+       obj_desc = *obj_desc_ptr;
+       if (!obj_desc) {
+               obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_PACKAGE);
+               *obj_desc_ptr = obj_desc;
+               if (!obj_desc) {
+                       return_ACPI_STATUS(AE_NO_MEMORY);
+               }
+
+               obj_desc->package.node = parent->common.node;
+       }
+
+       if (obj_desc->package.flags & AOPOBJ_DATA_VALID) {      /* Just in case */
+               return_ACPI_STATUS(AE_OK);
+       }
+
+       /*
+        * Allocate the element array (array of pointers to the individual
+        * objects) based on the num_elements parameter. Add an extra pointer slot
+        * so that the list is always null terminated.
+        */
+       obj_desc->package.elements = ACPI_ALLOCATE_ZEROED(((acpi_size)
+                                                          element_count +
+                                                          1) * sizeof(void *));
+
+       if (!obj_desc->package.elements) {
+               acpi_ut_delete_object_desc(obj_desc);
+               return_ACPI_STATUS(AE_NO_MEMORY);
+       }
+
+       obj_desc->package.count = element_count;
+       arg = op->common.value.arg;
+       arg = arg->common.next;
+
+       if (arg) {
+               obj_desc->package.flags |= AOPOBJ_DATA_VALID;
+       }
+
+       /*
+        * Initialize the elements of the package, up to the num_elements count.
+        * Package is automatically padded with uninitialized (NULL) elements
+        * if num_elements is greater than the package list length. Likewise,
+        * Package is truncated if num_elements is less than the list length.
+        */
+       for (i = 0; arg && (i < element_count); i++) {
+               if (arg->common.aml_opcode == AML_INT_RETURN_VALUE_OP) {
+                       if (arg->common.node->type == ACPI_TYPE_METHOD) {
+                               /*
+                                * A method reference "looks" to the parser to be a method
+                                * invocation, so we special case it here
+                                */
+                               arg->common.aml_opcode = AML_INT_NAMEPATH_OP;
+                               status =
+                                   acpi_ds_build_internal_object(walk_state,
+                                                                 arg,
+                                                                 &obj_desc->
+                                                                 package.
+                                                                 elements[i]);
+                       } else {
+                               /* This package element is already built, just get it */
+
+                               obj_desc->package.elements[i] =
+                                   ACPI_CAST_PTR(union acpi_operand_object,
+                                                 arg->common.node);
+                       }
+               } else {
+                       status =
+                           acpi_ds_build_internal_object(walk_state, arg,
+                                                         &obj_desc->package.
+                                                         elements[i]);
+                       if (status == AE_NOT_FOUND) {
+                               ACPI_ERROR((AE_INFO, "%-48s",
+                                           "****DS namepath not found"));
+                       }
+
+                       /*
+                        * Initialize this package element. This function handles the
+                        * resolution of named references within the package.
+                        */
+                       acpi_ds_init_package_element(0,
+                                                    obj_desc->package.
+                                                    elements[i], NULL,
+                                                    &obj_desc->package.
+                                                    elements[i]);
+               }
+
+               if (*obj_desc_ptr) {
+
+                       /* Existing package, get existing reference count */
+
+                       reference_count =
+                           (*obj_desc_ptr)->common.reference_count;
+                       if (reference_count > 1) {
+
+                               /* Make new element ref count match original ref count */
+                               /* TBD: Probably need an acpi_ut_add_references function */
+
+                               for (index = 0;
+                                    index < ((u32)reference_count - 1);
+                                    index++) {
+                                       acpi_ut_add_reference((obj_desc->
+                                                              package.
+                                                              elements[i]));
+                               }
+                       }
+               }
+
+               arg = arg->common.next;
+       }
+
+       /* Check for match between num_elements and actual length of package_list */
+
+       if (arg) {
+               /*
+                * num_elements was exhausted, but there are remaining elements in
+                * the package_list. Truncate the package to num_elements.
+                *
+                * Note: technically, this is an error, from ACPI spec: "It is an
+                * error for NumElements to be less than the number of elements in
+                * the PackageList". However, we just print a message and no
+                * exception is returned. This provides compatibility with other
+                * ACPI implementations. Some firmware implementations will alter
+                * the num_elements on the fly, possibly creating this type of
+                * ill-formed package object.
+                */
+               while (arg) {
+                       /*
+                        * We must delete any package elements that were created earlier
+                        * and are not going to be used because of the package truncation.
+                        */
+                       if (arg->common.node) {
+                               acpi_ut_remove_reference(ACPI_CAST_PTR
+                                                        (union
+                                                         acpi_operand_object,
+                                                         arg->common.node));
+                               arg->common.node = NULL;
+                       }
+
+                       /* Find out how many elements there really are */
+
+                       i++;
+                       arg = arg->common.next;
+               }
+
+               ACPI_INFO(("Actual Package length (%u) is larger than "
+                          "NumElements field (%u), truncated",
+                          i, element_count));
+       } else if (i < element_count) {
+               /*
+                * Arg list (elements) was exhausted, but we did not reach
+                * num_elements count.
+                *
+                * Note: this is not an error, the package is padded out
+                * with NULLs.
+                */
+               ACPI_DEBUG_PRINT((ACPI_DB_INFO,
+                                 "Package List length (%u) smaller than NumElements "
+                                 "count (%u), padded with null elements\n",
+                                 i, element_count));
+       }
+
+       obj_desc->package.flags |= AOPOBJ_DATA_VALID;
+       op->common.node = ACPI_CAST_PTR(struct acpi_namespace_node, obj_desc);
+       return_ACPI_STATUS(status);
+}
+
+/*******************************************************************************
+ *
+ * FUNCTION:    acpi_ds_init_package_element
+ *
+ * PARAMETERS:  acpi_pkg_callback
+ *
+ * RETURN:      Status
+ *
+ * DESCRIPTION: Resolve a named reference element within a package object
+ *
+ ******************************************************************************/
+
+acpi_status
+acpi_ds_init_package_element(u8 object_type,
+                            union acpi_operand_object *source_object,
+                            union acpi_generic_state *state, void *context)
+{
+       union acpi_operand_object **element_ptr;
+
+       if (!source_object) {
+               return (AE_OK);
+       }
+
+       /*
+        * The following code is a bit of a hack to workaround a (current)
+        * limitation of the acpi_pkg_callback interface. We need a pointer
+        * to the location within the element array because a new object
+        * may be created and stored there.
+        */
+       if (context) {
+
+               /* A direct call was made to this function */
+
+               element_ptr = (union acpi_operand_object **)context;
+       } else {
+               /* Call came from acpi_ut_walk_package_tree */
+
+               element_ptr = state->pkg.this_target_obj;
+       }
+
+       /* We are only interested in reference objects/elements */
+
+       if (source_object->common.type == ACPI_TYPE_LOCAL_REFERENCE) {
+
+               /* Attempt to resolve the (named) reference to a namespace node */
+
+               acpi_ds_resolve_package_element(element_ptr);
+       } else if (source_object->common.type == ACPI_TYPE_PACKAGE) {
+               source_object->package.flags |= AOPOBJ_DATA_VALID;
+       }
+
+       return (AE_OK);
+}
+
+/*******************************************************************************
+ *
+ * FUNCTION:    acpi_ds_resolve_package_element
+ *
+ * PARAMETERS:  element_ptr         - Pointer to a reference object
+ *
+ * RETURN:      Possible new element is stored to the indirect element_ptr
+ *
+ * DESCRIPTION: Resolve a package element that is a reference to a named
+ *              object.
+ *
+ ******************************************************************************/
+
+static void
+acpi_ds_resolve_package_element(union acpi_operand_object **element_ptr)
+{
+       acpi_status status;
+       union acpi_generic_state scope_info;
+       union acpi_operand_object *element = *element_ptr;
+       struct acpi_namespace_node *resolved_node;
+       char *external_path = NULL;
+       acpi_object_type type;
+
+       ACPI_FUNCTION_TRACE(ds_resolve_package_element);
+
+       /* Check if reference element is already resolved */
+
+       if (element->reference.resolved) {
+               return_VOID;
+       }
+
+       /* Element must be a reference object of correct type */
+
+       scope_info.scope.node = element->reference.node;        /* Prefix node */
+
+       status = acpi_ns_lookup(&scope_info, (char *)element->reference.aml,    /* Pointer to AML path */
+                               ACPI_TYPE_ANY, ACPI_IMODE_EXECUTE,
+                               ACPI_NS_SEARCH_PARENT | ACPI_NS_DONT_OPEN_SCOPE,
+                               NULL, &resolved_node);
+       if (ACPI_FAILURE(status)) {
+               status = acpi_ns_externalize_name(ACPI_UINT32_MAX,
+                                                 (char *)element->reference.
+                                                 aml, NULL, &external_path);
+
+               ACPI_EXCEPTION((AE_INFO, status,
+                               "Could not find/resolve named package element: %s",
+                               external_path));
+
+               ACPI_FREE(external_path);
+               *element_ptr = NULL;
+               return_VOID;
+       } else if (resolved_node->type == ACPI_TYPE_ANY) {
+
+               /* Named reference not resolved, return a NULL package element */
+
+               ACPI_ERROR((AE_INFO,
+                           "Could not resolve named package element [%4.4s] in [%4.4s]",
+                           resolved_node->name.ascii,
+                           scope_info.scope.node->name.ascii));
+               *element_ptr = NULL;
+               return_VOID;
+       }
+#if 0
+       else if (resolved_node->flags & ANOBJ_TEMPORARY) {
+               /*
+                * A temporary node found here indicates that the reference is
+                * to a node that was created within this method. We are not
+                * going to allow it (especially if the package is returned
+                * from the method) -- the temporary node will be deleted out
+                * from under the method. (05/2017).
+                */
+               ACPI_ERROR((AE_INFO,
+                           "Package element refers to a temporary name [%4.4s], "
+                           "inserting a NULL element",
+                           resolved_node->name.ascii));
+               *element_ptr = NULL;
+               return_VOID;
+       }
+#endif
+
+       /*
+        * Special handling for Alias objects. We need resolved_node to point
+        * to the Alias target. This effectively "resolves" the alias.
+        */
+       if (resolved_node->type == ACPI_TYPE_LOCAL_ALIAS) {
+               resolved_node = ACPI_CAST_PTR(struct acpi_namespace_node,
+                                             resolved_node->object);
+       }
+
+       /* Update the reference object */
+
+       element->reference.resolved = TRUE;
+       element->reference.node = resolved_node;
+       type = element->reference.node->type;
+
+       /*
+        * Attempt to resolve the node to a value before we insert it into
+        * the package. If this is a reference to a common data type,
+        * resolve it immediately. According to the ACPI spec, package
+        * elements can only be "data objects" or method references.
+        * Attempt to resolve to an Integer, Buffer, String or Package.
+        * If cannot, return the named reference (for things like Devices,
+        * Methods, etc.) Buffer Fields and Fields will resolve to simple
+        * objects (int/buf/str/pkg).
+        *
+        * NOTE: References to things like Devices, Methods, Mutexes, etc.
+        * will remain as named references. This behavior is not described
+        * in the ACPI spec, but it appears to be an oversight.
+        */
+       status = acpi_ex_resolve_node_to_value(&resolved_node, NULL);
+       if (ACPI_FAILURE(status)) {
+               return_VOID;
+       }
+#if 0
+/* TBD - alias support */
+       /*
+        * Special handling for Alias objects. We need to setup the type
+        * and the Op->Common.Node to point to the Alias target. Note,
+        * Alias has at most one level of indirection internally.
+        */
+       type = op->common.node->type;
+       if (type == ACPI_TYPE_LOCAL_ALIAS) {
+               type = obj_desc->common.type;
+               op->common.node = ACPI_CAST_PTR(struct acpi_namespace_node,
+                                               op->common.node->object);
+       }
+#endif
+
+       switch (type) {
+               /*
+                * These object types are a result of named references, so we will
+                * leave them as reference objects. In other words, these types
+                * have no intrinsic "value".
+                */
+       case ACPI_TYPE_DEVICE:
+       case ACPI_TYPE_THERMAL:
+
+               /* TBD: This may not be necesssary */
+
+               acpi_ut_add_reference(resolved_node->object);
+               break;
+
+       case ACPI_TYPE_MUTEX:
+       case ACPI_TYPE_METHOD:
+       case ACPI_TYPE_POWER:
+       case ACPI_TYPE_PROCESSOR:
+       case ACPI_TYPE_EVENT:
+       case ACPI_TYPE_REGION:
+
+               break;
+
+       default:
+               /*
+                * For all other types - the node was resolved to an actual
+                * operand object with a value, return the object
+                */
+               *element_ptr = (union acpi_operand_object *)resolved_node;
+               break;
+       }
+
+       return_VOID;
+}
index 9095d51f6b372cf479b6dd37b7606496bfc5b768..67b7370dcae520b52d8ea8ceea74fe1defc6da9f 100644 (file)
@@ -69,9 +69,14 @@ void acpi_ns_check_argument_types(struct acpi_evaluate_info *info)
        u8 user_arg_type;
        u32 i;
 
-       /* If not a predefined name, cannot typecheck args */
-
-       if (!info->predefined) {
+       /*
+        * If not a predefined name, cannot typecheck args, because
+        * we have no idea what argument types are expected.
+        * Also, ignore typecheck if warnings/errors if this method
+        * has already been evaluated at least once -- in order
+        * to suppress repetitive messages.
+        */
+       if (!info->predefined || (info->node->flags & ANOBJ_EVALUATED)) {
                return;
        }
 
@@ -93,6 +98,10 @@ void acpi_ns_check_argument_types(struct acpi_evaluate_info *info)
                                              acpi_ut_get_type_name
                                              (user_arg_type),
                                              acpi_ut_get_type_name(arg_type)));
+
+                       /* Prevent any additional typechecking for this method */
+
+                       info->node->flags |= ANOBJ_EVALUATED;
                }
        }
 }
@@ -121,7 +130,7 @@ acpi_ns_check_acpi_compliance(char *pathname,
        u32 aml_param_count;
        u32 required_param_count;
 
-       if (!predefined) {
+       if (!predefined || (node->flags & ANOBJ_EVALUATED)) {
                return;
        }
 
@@ -215,6 +224,10 @@ acpi_ns_check_argument_count(char *pathname,
        u32 aml_param_count;
        u32 required_param_count;
 
+       if (node->flags & ANOBJ_EVALUATED) {
+               return;
+       }
+
        if (!predefined) {
                /*
                 * Not a predefined name. Check the incoming user argument count
index ce33e7297ea7768ccd937c2cda2f46132d887a93..9c62979497122175d6338feb451246f4c93fcacd 100644 (file)
@@ -396,6 +396,20 @@ acpi_ns_init_one_object(acpi_handle obj_handle,
 
                info->package_init++;
                status = acpi_ds_get_package_arguments(obj_desc);
+               if (ACPI_FAILURE(status)) {
+                       break;
+               }
+
+               /*
+                * Resolve all named references in package objects (and all
+                * sub-packages). This action has been deferred until the entire
+                * namespace has been loaded, in order to support external and
+                * forward references from individual package elements (05/2017).
+                */
+               status = acpi_ut_walk_package_tree(obj_desc, NULL,
+                                                  acpi_ds_init_package_element,
+                                                  NULL);
+               obj_desc->package.flags |= AOPOBJ_DATA_VALID;
                break;
 
        default:
index aa16aeaa89379e7678cd6846e3c379d2fa256e67..a410760a03089f63c8a9a4b9b26c6e44bffe7583 100644 (file)
@@ -89,7 +89,14 @@ acpi_size acpi_ns_get_pathname_length(struct acpi_namespace_node *node)
 {
        acpi_size size;
 
-       ACPI_FUNCTION_ENTRY();
+       /* Validate the Node */
+
+       if (ACPI_GET_DESCRIPTOR_TYPE(node) != ACPI_DESC_TYPE_NAMED) {
+               ACPI_ERROR((AE_INFO,
+                           "Invalid/cached reference target node: %p, descriptor type %d",
+                           node, ACPI_GET_DESCRIPTOR_TYPE(node)));
+               return (0);
+       }
 
        size = acpi_ns_build_normalized_path(node, NULL, 0, FALSE);
        return (size);
index b4224005783c6d1c2fbf6f8bb970e6216ddd788a..bb04dec168ad72a9c7f634265131b64ec9b6194a 100644 (file)
@@ -164,6 +164,11 @@ acpi_ps_get_arguments(struct acpi_walk_state *walk_state,
                        INCREMENT_ARG_LIST(walk_state->arg_types);
                }
 
+               ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
+                                 "Final argument count: %u pass %u\n",
+                                 walk_state->arg_count,
+                                 walk_state->pass_number));
+
                /*
                 * Handle executable code at "module-level". This refers to
                 * executable opcodes that appear outside of any control method.
@@ -277,6 +282,11 @@ acpi_ps_get_arguments(struct acpi_walk_state *walk_state,
                             AML_NAME_OP)
                            && (walk_state->pass_number <=
                                ACPI_IMODE_LOAD_PASS2)) {
+                               ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
+                                                 "Setup Package/Buffer: Pass %u, AML Ptr: %p\n",
+                                                 walk_state->pass_number,
+                                                 aml_op_start));
+
                                /*
                                 * Skip parsing of Buffers and Packages because we don't have
                                 * enough info in the first pass to parse them correctly.
@@ -570,6 +580,10 @@ acpi_status acpi_ps_parse_loop(struct acpi_walk_state *walk_state)
 
                /* Check for arguments that need to be processed */
 
+               ACPI_DEBUG_PRINT((ACPI_DB_PARSE,
+                                 "Parseloop: argument count: %u\n",
+                                 walk_state->arg_count));
+
                if (walk_state->arg_count) {
                        /*
                         * There are arguments (complex ones), push Op and
index 443ffad01209b551786ac62af100c29f5fcdc015..4d9f3a2a4f8efaeb8fad7c813cd10002e5569669 100644 (file)
@@ -224,7 +224,7 @@ acpi_ut_create_update_state_and_push(union acpi_operand_object *object,
  *
  * RETURN:      Status
  *
- * DESCRIPTION: Walk through a package
+ * DESCRIPTION: Walk through a package, including subpackages
  *
  ******************************************************************************/
 
@@ -236,8 +236,8 @@ acpi_ut_walk_package_tree(union acpi_operand_object *source_object,
        acpi_status status = AE_OK;
        union acpi_generic_state *state_list = NULL;
        union acpi_generic_state *state;
-       u32 this_index;
        union acpi_operand_object *this_source_obj;
+       u32 this_index;
 
        ACPI_FUNCTION_TRACE(ut_walk_package_tree);
 
@@ -251,8 +251,10 @@ acpi_ut_walk_package_tree(union acpi_operand_object *source_object,
                /* Get one element of the package */
 
                this_index = state->pkg.index;
-               this_source_obj = (union acpi_operand_object *)
+               this_source_obj =
                    state->pkg.source_object->package.elements[this_index];
+               state->pkg.this_target_obj =
+                   &state->pkg.source_object->package.elements[this_index];
 
                /*
                 * Check for:
index 64e6641bfe824a5e6afd4bf7a4fa92766917d7e4..cb3db9fed50d1d753fcb1137518bdc096ae58356 100644 (file)
@@ -483,6 +483,11 @@ acpi_ut_get_simple_object_size(union acpi_operand_object *internal_object,
 
                /* A namespace node should never get here */
 
+               ACPI_ERROR((AE_INFO,
+                           "Received a namespace node [%4.4s] "
+                           "where an operand object is required",
+                           ACPI_CAST_PTR(struct acpi_namespace_node,
+                                         internal_object)->name.ascii));
                return_ACPI_STATUS(AE_AML_INTERNAL);
        }
 
index 64308c304ade735f795fe3a091997d9db1173080..eafabcd2fada3c4684d6a08a6f20c51ab8cc5951 100644 (file)
@@ -226,7 +226,7 @@ union acpi_generic_state *acpi_ut_create_update_state(union acpi_operand_object
 
 union acpi_generic_state *acpi_ut_create_pkg_state(void *internal_object,
                                                   void *external_object,
-                                                  u16 index)
+                                                  u32 index)
 {
        union acpi_generic_state *state;
 
index 9a07a42cae3495f96d4e2a00266a41a2d3fdd820..3c8de88ecbd5ff776f670a2b6b741e5d1a53eae6 100644 (file)
@@ -591,6 +591,10 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                return_VOID;
        }
 
+       if (!acpi_gbl_global_list) {
+               goto exit;
+       }
+
        element = acpi_gbl_global_list->list_head;
        while (element) {
                if ((element->component & component) &&
@@ -602,7 +606,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
 
                        if (element->size <
                            sizeof(struct acpi_common_descriptor)) {
-                               acpi_os_printf("%p Length 0x%04X %9.9s-%u "
+                               acpi_os_printf("%p Length 0x%04X %9.9s-%4.4u "
                                               "[Not a Descriptor - too small]\n",
                                               descriptor, element->size,
                                               element->module, element->line);
@@ -612,7 +616,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                                if (ACPI_GET_DESCRIPTOR_TYPE(descriptor) !=
                                    ACPI_DESC_TYPE_CACHED) {
                                        acpi_os_printf
-                                           ("%p Length 0x%04X %9.9s-%u [%s] ",
+                                           ("%p Length 0x%04X %9.9s-%4.4u [%s] ",
                                             descriptor, element->size,
                                             element->module, element->line,
                                             acpi_ut_get_descriptor_name
@@ -705,6 +709,7 @@ void acpi_ut_dump_allocations(u32 component, const char *module)
                element = element->next;
        }
 
+exit:
        (void)acpi_ut_release_mutex(ACPI_MTX_MEMORY);
 
        /* Print summary */