Fedora kernel-2.6.17-1.2142_FC4 patched with stable patch-2.6.17.4-vs2.0.2-rc26.diff
[linux-2.6.git] / drivers / acpi / executer / exconfig.c
index ac3c061..a29782f 100644 (file)
@@ -5,7 +5,7 @@
  *****************************************************************************/
 
 /*
- * Copyright (C) 2000 - 2005, R. Byron Moore
+ * Copyright (C) 2000 - 2006, R. Byron Moore
  * All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
@@ -41,7 +41,6 @@
  * POSSIBILITY OF SUCH DAMAGES.
  */
 
-
 #include <acpi/acpi.h>
 #include <acpi/acinterp.h>
 #include <acpi/amlcode.h>
 #include <acpi/actables.h>
 #include <acpi/acdispat.h>
 
-
 #define _COMPONENT          ACPI_EXECUTER
-        ACPI_MODULE_NAME    ("exconfig")
+ACPI_MODULE_NAME("exconfig")
 
+/* Local prototypes */
+static acpi_status
+acpi_ex_add_table(struct acpi_table_header *table,
+                 struct acpi_namespace_node *parent_node,
+                 union acpi_operand_object **ddb_handle);
 
 /*******************************************************************************
  *
  *
  ******************************************************************************/
 
-acpi_status
-acpi_ex_add_table (
-       struct acpi_table_header        *table,
-       struct acpi_namespace_node      *parent_node,
-       union acpi_operand_object       **ddb_handle)
+static acpi_status
+acpi_ex_add_table(struct acpi_table_header *table,
+                 struct acpi_namespace_node *parent_node,
+                 union acpi_operand_object **ddb_handle)
 {
-       acpi_status                     status;
-       struct acpi_table_desc          table_info;
-       union acpi_operand_object       *obj_desc;
-
-
-       ACPI_FUNCTION_TRACE ("ex_add_table");
+       acpi_status status;
+       struct acpi_table_desc table_info;
+       union acpi_operand_object *obj_desc;
 
+       ACPI_FUNCTION_TRACE("ex_add_table");
 
        /* Create an object to be the table handle */
 
-       obj_desc = acpi_ut_create_internal_object (ACPI_TYPE_LOCAL_REFERENCE);
+       obj_desc = acpi_ut_create_internal_object(ACPI_TYPE_LOCAL_REFERENCE);
        if (!obj_desc) {
-               return_ACPI_STATUS (AE_NO_MEMORY);
+               return_ACPI_STATUS(AE_NO_MEMORY);
        }
 
+       /* Init the table handle */
+
+       obj_desc->reference.opcode = AML_LOAD_OP;
+       *ddb_handle = obj_desc;
+
        /* Install the new table into the local data structures */
 
-       ACPI_MEMSET (&table_info, 0, sizeof (struct acpi_table_desc));
+       ACPI_MEMSET(&table_info, 0, sizeof(struct acpi_table_desc));
+
+       table_info.type = ACPI_TABLE_SSDT;
+       table_info.pointer = table;
+       table_info.length = (acpi_size) table->length;
+       table_info.allocation = ACPI_MEM_ALLOCATED;
 
-       table_info.type        = ACPI_TABLE_SSDT;
-       table_info.pointer     = table;
-       table_info.length      = (acpi_size) table->length;
-       table_info.allocation  = ACPI_MEM_ALLOCATED;
+       status = acpi_tb_install_table(&table_info);
+       obj_desc->reference.object = table_info.installed_desc;
 
-       status = acpi_tb_install_table (&table_info);
-       if (ACPI_FAILURE (status)) {
+       if (ACPI_FAILURE(status)) {
+               if (status == AE_ALREADY_EXISTS) {
+                       /* Table already exists, just return the handle */
+
+                       return_ACPI_STATUS(AE_OK);
+               }
                goto cleanup;
        }
 
        /* Add the table to the namespace */
 
-       status = acpi_ns_load_table (table_info.installed_desc, parent_node);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_ns_load_table(table_info.installed_desc, parent_node);
+       if (ACPI_FAILURE(status)) {
                /* Uninstall table on error */
 
-               (void) acpi_tb_uninstall_table (table_info.installed_desc);
+               (void)acpi_tb_uninstall_table(table_info.installed_desc);
                goto cleanup;
        }
 
-       /* Init the table handle */
+       return_ACPI_STATUS(AE_OK);
 
-       obj_desc->reference.opcode = AML_LOAD_OP;
-       obj_desc->reference.object = table_info.installed_desc;
-       *ddb_handle = obj_desc;
-       return_ACPI_STATUS (AE_OK);
-
-
-cleanup:
-       acpi_ut_remove_reference (obj_desc);
-       return_ACPI_STATUS (status);
+      cleanup:
+       acpi_ut_remove_reference(obj_desc);
+       *ddb_handle = NULL;
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ex_load_table_op
@@ -143,56 +149,53 @@ cleanup:
  ******************************************************************************/
 
 acpi_status
-acpi_ex_load_table_op (
-       struct acpi_walk_state          *walk_state,
-       union acpi_operand_object       **return_desc)
+acpi_ex_load_table_op(struct acpi_walk_state *walk_state,
+                     union acpi_operand_object **return_desc)
 {
-       acpi_status                     status;
-       union acpi_operand_object       **operand = &walk_state->operands[0];
-       struct acpi_table_header        *table;
-       struct acpi_namespace_node      *parent_node;
-       struct acpi_namespace_node      *start_node;
-       struct acpi_namespace_node      *parameter_node = NULL;
-       union acpi_operand_object       *ddb_handle;
-
-
-       ACPI_FUNCTION_TRACE ("ex_load_table_op");
+       acpi_status status;
+       union acpi_operand_object **operand = &walk_state->operands[0];
+       struct acpi_table_header *table;
+       struct acpi_namespace_node *parent_node;
+       struct acpi_namespace_node *start_node;
+       struct acpi_namespace_node *parameter_node = NULL;
+       union acpi_operand_object *ddb_handle;
 
+       ACPI_FUNCTION_TRACE("ex_load_table_op");
 
 #if 0
        /*
         * Make sure that the signature does not match one of the tables that
         * is already loaded.
         */
-       status = acpi_tb_match_signature (operand[0]->string.pointer, NULL);
+       status = acpi_tb_match_signature(operand[0]->string.pointer, NULL);
        if (status == AE_OK) {
                /* Signature matched -- don't allow override */
 
-               return_ACPI_STATUS (AE_ALREADY_EXISTS);
+               return_ACPI_STATUS(AE_ALREADY_EXISTS);
        }
 #endif
 
        /* Find the ACPI table */
 
-       status = acpi_tb_find_table (operand[0]->string.pointer,
-                          operand[1]->string.pointer,
-                          operand[2]->string.pointer, &table);
-       if (ACPI_FAILURE (status)) {
+       status = acpi_tb_find_table(operand[0]->string.pointer,
+                                   operand[1]->string.pointer,
+                                   operand[2]->string.pointer, &table);
+       if (ACPI_FAILURE(status)) {
                if (status != AE_NOT_FOUND) {
-                       return_ACPI_STATUS (status);
+                       return_ACPI_STATUS(status);
                }
 
                /* Table not found, return an Integer=0 and AE_OK */
 
-               ddb_handle = acpi_ut_create_internal_object (ACPI_TYPE_INTEGER);
+               ddb_handle = acpi_ut_create_internal_object(ACPI_TYPE_INTEGER);
                if (!ddb_handle) {
-                       return_ACPI_STATUS (AE_NO_MEMORY);
+                       return_ACPI_STATUS(AE_NO_MEMORY);
                }
 
                ddb_handle->integer.value = 0;
                *return_desc = ddb_handle;
 
-               return_ACPI_STATUS (AE_OK);
+               return_ACPI_STATUS(AE_OK);
        }
 
        /* Default nodes */
@@ -207,10 +210,12 @@ acpi_ex_load_table_op (
                 * Find the node referenced by the root_path_string. This is the
                 * location within the namespace where the table will be loaded.
                 */
-               status = acpi_ns_get_node_by_path (operand[3]->string.pointer, start_node,
-                                  ACPI_NS_SEARCH_PARENT, &parent_node);
-               if (ACPI_FAILURE (status)) {
-                       return_ACPI_STATUS (status);
+               status =
+                   acpi_ns_get_node_by_path(operand[3]->string.pointer,
+                                            start_node, ACPI_NS_SEARCH_PARENT,
+                                            &parent_node);
+               if (ACPI_FAILURE(status)) {
+                       return_ACPI_STATUS(status);
                }
        }
 
@@ -218,7 +223,7 @@ acpi_ex_load_table_op (
 
        if (operand[4]->string.length > 0) {
                if ((operand[4]->string.pointer[0] != '\\') &&
-                       (operand[4]->string.pointer[0] != '^')) {
+                   (operand[4]->string.pointer[0] != '^')) {
                        /*
                         * Path is not absolute, so it will be relative to the node
                         * referenced by the root_path_string (or the NS root if omitted)
@@ -226,21 +231,22 @@ acpi_ex_load_table_op (
                        start_node = parent_node;
                }
 
-               /*
-                * Find the node referenced by the parameter_path_string
-                */
-               status = acpi_ns_get_node_by_path (operand[4]->string.pointer, start_node,
-                                  ACPI_NS_SEARCH_PARENT, &parameter_node);
-               if (ACPI_FAILURE (status)) {
-                       return_ACPI_STATUS (status);
+               /* Find the node referenced by the parameter_path_string */
+
+               status =
+                   acpi_ns_get_node_by_path(operand[4]->string.pointer,
+                                            start_node, ACPI_NS_SEARCH_PARENT,
+                                            &parameter_node);
+               if (ACPI_FAILURE(status)) {
+                       return_ACPI_STATUS(status);
                }
        }
 
        /* Load the table into the namespace */
 
-       status = acpi_ex_add_table (table, parent_node, &ddb_handle);
-       if (ACPI_FAILURE (status)) {
-               return_ACPI_STATUS (status);
+       status = acpi_ex_add_table(table, parent_node, &ddb_handle);
+       if (ACPI_FAILURE(status)) {
+               return_ACPI_STATUS(status);
        }
 
        /* Parameter Data (optional) */
@@ -248,19 +254,20 @@ acpi_ex_load_table_op (
        if (parameter_node) {
                /* Store the parameter data into the optional parameter object */
 
-               status = acpi_ex_store (operand[5], ACPI_CAST_PTR (union acpi_operand_object, parameter_node),
-                                walk_state);
-               if (ACPI_FAILURE (status)) {
-                       (void) acpi_ex_unload_table (ddb_handle);
-                       return_ACPI_STATUS (status);
+               status = acpi_ex_store(operand[5],
+                                      ACPI_CAST_PTR(union acpi_operand_object,
+                                                    parameter_node),
+                                      walk_state);
+               if (ACPI_FAILURE(status)) {
+                       (void)acpi_ex_unload_table(ddb_handle);
+                       return_ACPI_STATUS(status);
                }
        }
 
        *return_desc = ddb_handle;
-       return_ACPI_STATUS  (status);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ex_load_op
@@ -277,38 +284,37 @@ acpi_ex_load_table_op (
  ******************************************************************************/
 
 acpi_status
-acpi_ex_load_op (
-       union acpi_operand_object       *obj_desc,
-       union acpi_operand_object       *target,
-       struct acpi_walk_state          *walk_state)
+acpi_ex_load_op(union acpi_operand_object *obj_desc,
+               union acpi_operand_object *target,
+               struct acpi_walk_state *walk_state)
 {
-       acpi_status                     status;
-       union acpi_operand_object       *ddb_handle;
-       union acpi_operand_object       *buffer_desc = NULL;
-       struct acpi_table_header        *table_ptr = NULL;
-       acpi_physical_address           address;
-       struct acpi_table_header        table_header;
-       u32                             i;
-
-       ACPI_FUNCTION_TRACE ("ex_load_op");
+       acpi_status status;
+       union acpi_operand_object *ddb_handle;
+       union acpi_operand_object *buffer_desc = NULL;
+       struct acpi_table_header *table_ptr = NULL;
+       acpi_physical_address address;
+       struct acpi_table_header table_header;
+       u32 i;
 
+       ACPI_FUNCTION_TRACE("ex_load_op");
 
        /* Object can be either an op_region or a Field */
 
-       switch (ACPI_GET_OBJECT_TYPE (obj_desc)) {
+       switch (ACPI_GET_OBJECT_TYPE(obj_desc)) {
        case ACPI_TYPE_REGION:
 
-               ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, "Load from Region %p %s\n",
-                       obj_desc, acpi_ut_get_object_type_name (obj_desc)));
+               ACPI_DEBUG_PRINT((ACPI_DB_EXEC, "Load from Region %p %s\n",
+                                 obj_desc,
+                                 acpi_ut_get_object_type_name(obj_desc)));
 
                /*
                 * If the Region Address and Length have not been previously evaluated,
                 * evaluate them now and save the results.
                 */
                if (!(obj_desc->common.flags & AOPOBJ_DATA_VALID)) {
-                       status = acpi_ds_get_region_arguments (obj_desc);
-                       if (ACPI_FAILURE (status)) {
-                               return_ACPI_STATUS (status);
+                       status = acpi_ds_get_region_arguments(obj_desc);
+                       if (ACPI_FAILURE(status)) {
+                               return_ACPI_STATUS(status);
                        }
                }
 
@@ -320,115 +326,127 @@ acpi_ex_load_op (
 
                table_header.length = 0;
                for (i = 0; i < 8; i++) {
-                       status = acpi_ev_address_space_dispatch (obj_desc, ACPI_READ,
-                                          (acpi_physical_address) (i + address), 8,
-                                          ((u8 *) &table_header) + i);
-                       if (ACPI_FAILURE (status)) {
-                               return_ACPI_STATUS (status);
+                       status =
+                           acpi_ev_address_space_dispatch(obj_desc, ACPI_READ,
+                                                          (acpi_physical_address)
+                                                          (i + address), 8,
+                                                          ((u8 *) &
+                                                           table_header) + i);
+                       if (ACPI_FAILURE(status)) {
+                               return_ACPI_STATUS(status);
                        }
                }
 
                /* Sanity check the table length */
 
-               if (table_header.length < sizeof (struct acpi_table_header)) {
-                       return_ACPI_STATUS (AE_BAD_HEADER);
+               if (table_header.length < sizeof(struct acpi_table_header)) {
+                       return_ACPI_STATUS(AE_BAD_HEADER);
                }
 
                /* Allocate a buffer for the entire table */
 
-               table_ptr = ACPI_MEM_ALLOCATE (table_header.length);
+               table_ptr = ACPI_MEM_ALLOCATE(table_header.length);
                if (!table_ptr) {
-                       return_ACPI_STATUS (AE_NO_MEMORY);
+                       return_ACPI_STATUS(AE_NO_MEMORY);
                }
 
                /* Get the entire table from the op region */
 
                for (i = 0; i < table_header.length; i++) {
-                       status = acpi_ev_address_space_dispatch (obj_desc, ACPI_READ,
-                                          (acpi_physical_address) (i + address), 8,
-                                          ((u8 *) table_ptr + i));
-                       if (ACPI_FAILURE (status)) {
+                       status =
+                           acpi_ev_address_space_dispatch(obj_desc, ACPI_READ,
+                                                          (acpi_physical_address)
+                                                          (i + address), 8,
+                                                          ((u8 *) table_ptr +
+                                                           i));
+                       if (ACPI_FAILURE(status)) {
                                goto cleanup;
                        }
                }
                break;
 
-
        case ACPI_TYPE_LOCAL_REGION_FIELD:
        case ACPI_TYPE_LOCAL_BANK_FIELD:
        case ACPI_TYPE_LOCAL_INDEX_FIELD:
 
-               ACPI_DEBUG_PRINT ((ACPI_DB_EXEC, "Load from Field %p %s\n",
-                       obj_desc, acpi_ut_get_object_type_name (obj_desc)));
+               ACPI_DEBUG_PRINT((ACPI_DB_EXEC, "Load from Field %p %s\n",
+                                 obj_desc,
+                                 acpi_ut_get_object_type_name(obj_desc)));
 
                /*
                 * The length of the field must be at least as large as the table.
                 * Read the entire field and thus the entire table.  Buffer is
                 * allocated during the read.
                 */
-               status = acpi_ex_read_data_from_field (walk_state, obj_desc, &buffer_desc);
-               if (ACPI_FAILURE (status)) {
-                       goto cleanup;
+               status =
+                   acpi_ex_read_data_from_field(walk_state, obj_desc,
+                                                &buffer_desc);
+               if (ACPI_FAILURE(status)) {
+                       return_ACPI_STATUS(status);
                }
 
-               table_ptr = ACPI_CAST_PTR (struct acpi_table_header, buffer_desc->buffer.pointer);
+               table_ptr = ACPI_CAST_PTR(struct acpi_table_header,
+                                         buffer_desc->buffer.pointer);
+
+               /* All done with the buffer_desc, delete it */
+
+               buffer_desc->buffer.pointer = NULL;
+               acpi_ut_remove_reference(buffer_desc);
 
-                /* Sanity check the table length */
+               /* Sanity check the table length */
 
-               if (table_ptr->length < sizeof (struct acpi_table_header)) {
-                       return_ACPI_STATUS (AE_BAD_HEADER);
+               if (table_ptr->length < sizeof(struct acpi_table_header)) {
+                       status = AE_BAD_HEADER;
+                       goto cleanup;
                }
                break;
 
-
        default:
-               return_ACPI_STATUS (AE_AML_OPERAND_TYPE);
+               return_ACPI_STATUS(AE_AML_OPERAND_TYPE);
        }
 
        /* The table must be either an SSDT or a PSDT */
 
-       if ((!ACPI_STRNCMP (table_ptr->signature,
-                         acpi_gbl_table_data[ACPI_TABLE_PSDT].signature,
-                         acpi_gbl_table_data[ACPI_TABLE_PSDT].sig_length)) &&
-               (!ACPI_STRNCMP (table_ptr->signature,
-                                acpi_gbl_table_data[ACPI_TABLE_SSDT].signature,
-                                acpi_gbl_table_data[ACPI_TABLE_SSDT].sig_length))) {
-               ACPI_DEBUG_PRINT ((ACPI_DB_ERROR,
-                       "Table has invalid signature [%4.4s], must be SSDT or PSDT\n",
-                       table_ptr->signature));
+       if ((!ACPI_STRNCMP(table_ptr->signature,
+                          acpi_gbl_table_data[ACPI_TABLE_PSDT].signature,
+                          acpi_gbl_table_data[ACPI_TABLE_PSDT].sig_length)) &&
+           (!ACPI_STRNCMP(table_ptr->signature,
+                          acpi_gbl_table_data[ACPI_TABLE_SSDT].signature,
+                          acpi_gbl_table_data[ACPI_TABLE_SSDT].sig_length))) {
+               ACPI_ERROR((AE_INFO,
+                           "Table has invalid signature [%4.4s], must be SSDT or PSDT",
+                           table_ptr->signature));
                status = AE_BAD_SIGNATURE;
                goto cleanup;
        }
 
        /* Install the new table into the local data structures */
 
-       status = acpi_ex_add_table (table_ptr, acpi_gbl_root_node, &ddb_handle);
-       if (ACPI_FAILURE (status)) {
-               goto cleanup;
-       }
-
-       /* Store the ddb_handle into the Target operand */
+       status = acpi_ex_add_table(table_ptr, acpi_gbl_root_node, &ddb_handle);
+       if (ACPI_FAILURE(status)) {
+               /* On error, table_ptr was deallocated above */
 
-       status = acpi_ex_store (ddb_handle, target, walk_state);
-       if (ACPI_FAILURE (status)) {
-               (void) acpi_ex_unload_table (ddb_handle);
+               return_ACPI_STATUS(status);
        }
 
-       return_ACPI_STATUS (status);
+       /* Store the ddb_handle into the Target operand */
 
+       status = acpi_ex_store(ddb_handle, target, walk_state);
+       if (ACPI_FAILURE(status)) {
+               (void)acpi_ex_unload_table(ddb_handle);
 
-cleanup:
+               /* table_ptr was deallocated above */
 
-       if (buffer_desc) {
-               acpi_ut_remove_reference (buffer_desc);
+               return_ACPI_STATUS(status);
        }
-       else {
-               ACPI_MEM_FREE (table_ptr);
+
+      cleanup:
+       if (ACPI_FAILURE(status)) {
+               ACPI_MEM_FREE(table_ptr);
        }
-       return_ACPI_STATUS (status);
+       return_ACPI_STATUS(status);
 }
 
-
 /*******************************************************************************
  *
  * FUNCTION:    acpi_ex_unload_table
@@ -441,17 +459,13 @@ cleanup:
  *
  ******************************************************************************/
 
-acpi_status
-acpi_ex_unload_table (
-       union acpi_operand_object       *ddb_handle)
+acpi_status acpi_ex_unload_table(union acpi_operand_object *ddb_handle)
 {
-       acpi_status                     status = AE_OK;
-       union acpi_operand_object       *table_desc = ddb_handle;
-       struct acpi_table_desc          *table_info;
-
-
-       ACPI_FUNCTION_TRACE ("ex_unload_table");
+       acpi_status status = AE_OK;
+       union acpi_operand_object *table_desc = ddb_handle;
+       struct acpi_table_desc *table_info;
 
+       ACPI_FUNCTION_TRACE("ex_unload_table");
 
        /*
         * Validate the handle
@@ -460,28 +474,28 @@ acpi_ex_unload_table (
         * validated here.
         */
        if ((!ddb_handle) ||
-               (ACPI_GET_DESCRIPTOR_TYPE (ddb_handle) != ACPI_DESC_TYPE_OPERAND) ||
-               (ACPI_GET_OBJECT_TYPE (ddb_handle) != ACPI_TYPE_LOCAL_REFERENCE)) {
-               return_ACPI_STATUS (AE_BAD_PARAMETER);
+           (ACPI_GET_DESCRIPTOR_TYPE(ddb_handle) != ACPI_DESC_TYPE_OPERAND) ||
+           (ACPI_GET_OBJECT_TYPE(ddb_handle) != ACPI_TYPE_LOCAL_REFERENCE)) {
+               return_ACPI_STATUS(AE_BAD_PARAMETER);
        }
 
        /* Get the actual table descriptor from the ddb_handle */
 
-       table_info = (struct acpi_table_desc *) table_desc->reference.object;
+       table_info = (struct acpi_table_desc *)table_desc->reference.object;
 
        /*
         * Delete the entire namespace under this table Node
         * (Offset contains the table_id)
         */
-       acpi_ns_delete_namespace_by_owner (table_info->table_id);
+       acpi_ns_delete_namespace_by_owner(table_info->owner_id);
+       acpi_ut_release_owner_id(&table_info->owner_id);
 
        /* Delete the table itself */
 
-       (void) acpi_tb_uninstall_table (table_info->installed_desc);
+       (void)acpi_tb_uninstall_table(table_info->installed_desc);
 
        /* Delete the table descriptor (ddb_handle) */
 
-       acpi_ut_remove_reference (table_desc);
-       return_ACPI_STATUS (status);
+       acpi_ut_remove_reference(table_desc);
+       return_ACPI_STATUS(status);
 }
-