[PATCH] acpiphp: fix acpi_path_name
authorMUNEDA Takahiro <muneda.takahiro@jp.fujitsu.com>
Fri, 17 Mar 2006 00:18:39 +0000 (09:18 +0900)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 23 Mar 2006 22:35:17 +0000 (14:35 -0800)
I encountered the problem that the insmod of the acpiphp
fails because of the mis-freeing of the memory.

I tested this patch on my tiger4 box.

Signed-off-by: MUNEDA Takahiro <muneda.takahiro@jp.fujitsu.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/pci/hotplug/acpi_pcihp.c
drivers/pci/hotplug/pci_hotplug.h
drivers/pci/hotplug/pciehp_hpc.c

index 0f713531754279dc09921aeffbf51017476557b3..39af9c325f3532e37013d53a6fbf0412bc482d63 100644 (file)
 #define        METHOD_NAME__HPP        "_HPP"
 #define        METHOD_NAME_OSHP        "OSHP"
 
-/* acpi_path_name
- *
- * @handle - the acpi_handle of the object who's name you want.
- *
- * Caller must free buffer.
- */
-u8 * acpi_path_name(acpi_handle handle)
-{
-       acpi_status status;
-       struct acpi_buffer ret_buf = {ACPI_ALLOCATE_BUFFER, NULL};
-       union acpi_object *obj;
-
-       status = acpi_get_name(handle, ACPI_FULL_PATHNAME, &ret_buf);
-       if (ACPI_FAILURE(status)) {
-               return NULL;
-       }
-       obj = ret_buf.pointer;
-       return obj->string.pointer;
-}
-EXPORT_SYMBOL_GPL(acpi_path_name);
-
-
 
 static acpi_status
 acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
@@ -66,10 +44,12 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
        acpi_status             status;
        u8                      nui[4];
        struct acpi_buffer      ret_buf = { 0, NULL};
+       struct acpi_buffer      string = { ACPI_ALLOCATE_BUFFER, NULL };
        union acpi_object       *ext_obj, *package;
-       u8                      *path_name = acpi_path_name(handle);
        int                     i, len = 0;
 
+       acpi_get_name(handle, ACPI_FULL_PATHNAME, &string);
+
        /* get _hpp */
        status = acpi_evaluate_object(handle, METHOD_NAME__HPP, NULL, &ret_buf);
        switch (status) {
@@ -77,8 +57,8 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
                ret_buf.pointer = kmalloc (ret_buf.length, GFP_KERNEL);
                if (!ret_buf.pointer) {
                        printk(KERN_ERR "%s:%s alloc for _HPP fail\n",
-                               __FUNCTION__, path_name);
-                       acpi_os_free(path_name);
+                               __FUNCTION__, (char *)string.pointer);
+                       acpi_os_free(string.pointer);
                        return AE_NO_MEMORY;
                }
                status = acpi_evaluate_object(handle, METHOD_NAME__HPP,
@@ -88,8 +68,8 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
        default:
                if (ACPI_FAILURE(status)) {
                        pr_debug("%s:%s _HPP fail=0x%x\n", __FUNCTION__,
-                                       path_name, status);
-                       acpi_os_free(path_name);
+                               (char *)string.pointer, status);
+                       acpi_os_free(string.pointer);
                        return status;
                }
        }
@@ -97,7 +77,7 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
        ext_obj = (union acpi_object *) ret_buf.pointer;
        if (ext_obj->type != ACPI_TYPE_PACKAGE) {
                printk(KERN_ERR "%s:%s _HPP obj not a package\n", __FUNCTION__,
-                               path_name);
+                               (char *)string.pointer);
                status = AE_ERROR;
                goto free_and_return;
        }
@@ -112,7 +92,7 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
                        break;
                default:
                        printk(KERN_ERR "%s:%s _HPP obj type incorrect\n",
-                               __FUNCTION__, path_name);
+                               __FUNCTION__, (char *)string.pointer);
                        status = AE_ERROR;
                        goto free_and_return;
                }
@@ -129,8 +109,8 @@ acpi_run_hpp(acpi_handle handle, struct hotplug_params *hpp)
        pr_debug("  _HPP: enable PERR    =0x%x\n", hpp->enable_perr);
 
 free_and_return:
-       acpi_os_free(path_name);
-       kfree(ret_buf.pointer);
+       acpi_os_free(string.pointer);
+       acpi_os_free(ret_buf.pointer);
        return status;
 }
 
@@ -143,16 +123,20 @@ free_and_return:
 acpi_status acpi_run_oshp(acpi_handle handle)
 {
        acpi_status             status;
-       u8                      *path_name = acpi_path_name(handle);
+       struct acpi_buffer      string = { ACPI_ALLOCATE_BUFFER, NULL };
+
+       acpi_get_name(handle, ACPI_FULL_PATHNAME, &string);
 
        /* run OSHP */
        status = acpi_evaluate_object(handle, METHOD_NAME_OSHP, NULL, NULL);
        if (ACPI_FAILURE(status))
                printk(KERN_ERR "%s:%s OSHP fails=0x%x\n", __FUNCTION__,
-                       path_name, status);
+                       (char *)string.pointer, status);
        else
-               pr_debug("%s:%s OSHP passes\n", __FUNCTION__, path_name);
-       acpi_os_free(path_name);
+               pr_debug("%s:%s OSHP passes\n", __FUNCTION__,
+                       (char *)string.pointer);
+
+       acpi_os_free(string.pointer);
        return status;
 }
 EXPORT_SYMBOL_GPL(acpi_run_oshp);
index e476ed0333847ee3b0bfb65a1ed6e9a34f9570d5..eb0d01d47236700ef4b3fb8dc60fcfdf6422f8e2 100644 (file)
@@ -190,7 +190,6 @@ struct hotplug_params {
 extern acpi_status acpi_run_oshp(acpi_handle handle);
 extern acpi_status acpi_get_hp_params_from_firmware(struct pci_dev *dev,
                                struct hotplug_params *hpp);
-extern u8 * acpi_path_name(acpi_handle handle);
 int acpi_root_bridge(acpi_handle handle);
 #endif
 #endif
index 22dcd12e4c1c28955f8e5cd8211c4c57e2f510bb..6c14d9e46b2e1f1a911191e3e7e40df61aa9d841 100644 (file)
@@ -1246,7 +1246,7 @@ int pciehp_acpi_get_hp_hw_control_from_firmware(struct pci_dev *dev)
        acpi_handle chandle, handle = DEVICE_ACPI_HANDLE(&(dev->dev));
        struct pci_dev *pdev = dev;
        struct pci_bus *parent;
-       u8 *path_name = NULL;
+       struct acpi_buffer string = { ACPI_ALLOCATE_BUFFER, NULL };
 
        /*
         * Per PCI firmware specification, we should run the ACPI _OSC
@@ -1278,16 +1278,17 @@ int pciehp_acpi_get_hp_hw_control_from_firmware(struct pci_dev *dev)
        }
 
        while (handle) {
-               path_name = acpi_path_name(handle);
-               dbg("Trying to get hotplug control for %s \n", path_name);
+               acpi_get_name(handle, ACPI_FULL_PATHNAME, &string);
+               dbg("Trying to get hotplug control for %s \n",
+                       (char *)string.pointer);
                status = pci_osc_control_set(handle,
                                OSC_PCI_EXPRESS_NATIVE_HP_CONTROL);
                if (status == AE_NOT_FOUND)
                        status = acpi_run_oshp(handle);
                if (ACPI_SUCCESS(status)) {
                        dbg("Gained control for hotplug HW for pci %s (%s)\n",
-                               pci_name(dev), path_name);
-                       acpi_os_free(path_name);
+                               pci_name(dev), (char *)string.pointer);
+                       acpi_os_free(string.pointer);
                        return 0;
                }
                if (acpi_root_bridge(handle))
@@ -1300,8 +1301,8 @@ int pciehp_acpi_get_hp_hw_control_from_firmware(struct pci_dev *dev)
 
        err("Cannot get control of hotplug hardware for pci %s\n",
                        pci_name(dev));
-       if (path_name)
-               acpi_os_free(path_name);
+
+       acpi_os_free(string.pointer);
        return -1;
 }
 #endif