9 files changed, 201 insertions, 146 deletions

diff --git a/drivers/acpi/bus.c b/drivers/acpi/bus.c
index e0596954290b..d59175efc428 100644
--- a/drivers/acpi/bus.c
+++ b/drivers/acpi/bus.c
@@ -994,8 +994,6 @@ static int __init acpi_bus_init(void)
         status = acpi_ec_ecdt_probe();
         /* Ignore result. Not having an ECDT is not fatal. */
 
-        acpi_bus_osc_support();
-
         status = acpi_initialize_objects(ACPI_FULL_INITIALIZATION);
         if (ACPI_FAILURE(status)) {
                 printk(KERN_ERR PREFIX "Unable to initialize ACPI objects\n");
@@ -1003,6 +1001,12 @@ static int __init acpi_bus_init(void)
         }
 
         /*
+         * _OSC method may exist in module level code,
+         * so it must be run after ACPI_FULL_INITIALIZATION
+         */
+        acpi_bus_osc_support();
+
+        /*
          * _PDC control method may load dynamic SSDT tables,
          * and we need to install the table handler before that.
          */
diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c
index 314a3b84bbc7..f0d936b65e37 100644
--- a/drivers/acpi/button.c
+++ b/drivers/acpi/button.c
@@ -450,15 +450,4 @@ static int acpi_button_remove(struct acpi_device *device, int type)
         return 0;
 }
 
-static int __init acpi_button_init(void)
-{
-        return acpi_bus_register_driver(&acpi_button_driver);
-}
-
-static void __exit acpi_button_exit(void)
-{
-        acpi_bus_unregister_driver(&acpi_button_driver);
-}
-
-module_init(acpi_button_init);
-module_exit(acpi_button_exit);
+module_acpi_driver(acpi_button_driver);
diff --git a/drivers/acpi/fan.c b/drivers/acpi/fan.c
index bc36a476f1ab..3bd6a54702d6 100644
--- a/drivers/acpi/fan.c
+++ b/drivers/acpi/fan.c
@@ -212,24 +212,4 @@ static int acpi_fan_resume(struct device *dev)
 }
 #endif
 
-static int __init acpi_fan_init(void)
-{
-        int result = 0;
-
-        result = acpi_bus_register_driver(&acpi_fan_driver);
-        if (result < 0)
-                return -ENODEV;
-
-        return 0;
-}
-
-static void __exit acpi_fan_exit(void)
-{
-
-        acpi_bus_unregister_driver(&acpi_fan_driver);
-
-        return;
-}
-
-module_init(acpi_fan_init);
-module_exit(acpi_fan_exit);
+module_acpi_driver(acpi_fan_driver);
diff --git a/drivers/acpi/glue.c b/drivers/acpi/glue.c
index 243ee85e4d2e..d1a2d74033e9 100644
--- a/drivers/acpi/glue.c
+++ b/drivers/acpi/glue.c
@@ -25,6 +25,8 @@
 static LIST_HEAD(bus_type_list);
 static DECLARE_RWSEM(bus_type_sem);
 
+#define PHYSICAL_NODE_STRING "physical_node"
+
 int register_acpi_bus_type(struct acpi_bus_type *type)
 {
         if (acpi_disabled)
@@ -124,84 +126,119 @@ acpi_handle acpi_get_child(acpi_handle parent, u64 address)
 
 EXPORT_SYMBOL(acpi_get_child);
 
-/* Link ACPI devices with physical devices */
-static void acpi_glue_data_handler(acpi_handle handle,
-                                   void *context)
-{
-        /* we provide an empty handler */
-}
-
-/* Note: a success call will increase reference count by one */
-struct device *acpi_get_physical_device(acpi_handle handle)
-{
-        acpi_status status;
-        struct device *dev;
-
-        status = acpi_get_data(handle, acpi_glue_data_handler, (void **)&dev);
-        if (ACPI_SUCCESS(status))
-                return get_device(dev);
-        return NULL;
-}
-
-EXPORT_SYMBOL(acpi_get_physical_device);
-
 static int acpi_bind_one(struct device *dev, acpi_handle handle)
 {
         struct acpi_device *acpi_dev;
         acpi_status status;
+        struct acpi_device_physical_node *physical_node;
+        char physical_node_name[sizeof(PHYSICAL_NODE_STRING) + 2];
+        int retval = -EINVAL;
 
         if (dev->archdata.acpi_handle) {
                 dev_warn(dev, "Drivers changed 'acpi_handle'\n");
                 return -EINVAL;
         }
+
         get_device(dev);
-        status = acpi_attach_data(handle, acpi_glue_data_handler, dev);
-        if (ACPI_FAILURE(status)) {
-                put_device(dev);
-                return -EINVAL;
+        status = acpi_bus_get_device(handle, &acpi_dev);
+        if (ACPI_FAILURE(status))
+                goto err;
+
+        physical_node = kzalloc(sizeof(struct acpi_device_physical_node),
+                GFP_KERNEL);
+        if (!physical_node) {
+                retval = -ENOMEM;
+                goto err;
         }
-        dev->archdata.acpi_handle = handle;
 
-        status = acpi_bus_get_device(handle, &acpi_dev);
-        if (!ACPI_FAILURE(status)) {
-                int ret;
-
-                ret = sysfs_create_link(&dev->kobj, &acpi_dev->dev.kobj,
-                                "firmware_node");
-                ret = sysfs_create_link(&acpi_dev->dev.kobj, &dev->kobj,
-                                "physical_node");
-                if (acpi_dev->wakeup.flags.valid)
-                        device_set_wakeup_capable(dev, true);
+        mutex_lock(&acpi_dev->physical_node_lock);
+        /* allocate physical node id according to physical_node_id_bitmap */
+        physical_node->node_id =
+                find_first_zero_bit(acpi_dev->physical_node_id_bitmap,
+                ACPI_MAX_PHYSICAL_NODE);
+        if (physical_node->node_id >= ACPI_MAX_PHYSICAL_NODE) {
+                retval = -ENOSPC;
+                mutex_unlock(&acpi_dev->physical_node_lock);
+                goto err;
         }
 
+        set_bit(physical_node->node_id, acpi_dev->physical_node_id_bitmap);
+        physical_node->dev = dev;
+        list_add_tail(&physical_node->node, &acpi_dev->physical_node_list);
+        acpi_dev->physical_node_count++;
+        mutex_unlock(&acpi_dev->physical_node_lock);
+
+        dev->archdata.acpi_handle = handle;
+
+        if (!physical_node->node_id)
+                strcpy(physical_node_name, PHYSICAL_NODE_STRING);
+        else
+                sprintf(physical_node_name,
+                        "physical_node%d", physical_node->node_id);
+        retval = sysfs_create_link(&acpi_dev->dev.kobj, &dev->kobj,
+                        physical_node_name);
+        retval = sysfs_create_link(&dev->kobj, &acpi_dev->dev.kobj,
+                "firmware_node");
+
+        if (acpi_dev->wakeup.flags.valid)
+                device_set_wakeup_capable(dev, true);
+
         return 0;
+
+ err:
+        put_device(dev);
+        return retval;
 }
 
 static int acpi_unbind_one(struct device *dev)
 {
+        struct acpi_device_physical_node *entry;
+        struct acpi_device *acpi_dev;
+        acpi_status status;
+        struct list_head *node, *next;
+
         if (!dev->archdata.acpi_handle)
                 return 0;
-        if (dev == acpi_get_physical_device(dev->archdata.acpi_handle)) {
-                struct acpi_device *acpi_dev;
 
-                /* acpi_get_physical_device increase refcnt by one */
-                put_device(dev);
+        status = acpi_bus_get_device(dev->archdata.acpi_handle,
+                &acpi_dev);
+        if (ACPI_FAILURE(status))
+                goto err;
 
-                if (!acpi_bus_get_device(dev->archdata.acpi_handle,
-                                        &acpi_dev)) {
-                        sysfs_remove_link(&dev->kobj, "firmware_node");
-                        sysfs_remove_link(&acpi_dev->dev.kobj, "physical_node");
-                }
+        mutex_lock(&acpi_dev->physical_node_lock);
+        list_for_each_safe(node, next, &acpi_dev->physical_node_list) {
+                char physical_node_name[sizeof(PHYSICAL_NODE_STRING) + 2];
+
+                entry = list_entry(node, struct acpi_device_physical_node,
+                        node);
+                if (entry->dev != dev)
+                        continue;
+
+                list_del(node);
+                clear_bit(entry->node_id, acpi_dev->physical_node_id_bitmap);
 
-                acpi_detach_data(dev->archdata.acpi_handle,
-                                 acpi_glue_data_handler);
+                acpi_dev->physical_node_count--;
+
+                if (!entry->node_id)
+                        strcpy(physical_node_name, PHYSICAL_NODE_STRING);
+                else
+                        sprintf(physical_node_name,
+                                "physical_node%d", entry->node_id);
+
+                sysfs_remove_link(&acpi_dev->dev.kobj, physical_node_name);
+                sysfs_remove_link(&dev->kobj, "firmware_node");
                 dev->archdata.acpi_handle = NULL;
                 /* acpi_bind_one increase refcnt by one */
                 put_device(dev);
-        } else {
-                dev_err(dev, "Oops, 'acpi_handle' corrupt\n");
+                kfree(entry);
         }
+        mutex_unlock(&acpi_dev->physical_node_lock);
+
         return 0;
+
+err:
+        dev_err(dev, "Oops, 'acpi_handle' corrupt\n");
+        return -EINVAL;
 }
 
 static int acpi_platform_notify(struct device *dev)
diff --git a/drivers/acpi/hed.c b/drivers/acpi/hed.c
index d0c1967f7597..20a0f2c3ca3b 100644
--- a/drivers/acpi/hed.c
+++ b/drivers/acpi/hed.c
@@ -86,25 +86,7 @@ static struct acpi_driver acpi_hed_driver = {
                 .notify = acpi_hed_notify,
         },
 };
-
-static int __init acpi_hed_init(void)
-{
-        if (acpi_disabled)
-                return -ENODEV;
-
-        if (acpi_bus_register_driver(&acpi_hed_driver) < 0)
-                return -ENODEV;
-
-        return 0;
-}
-
-static void __exit acpi_hed_exit(void)
-{
-        acpi_bus_unregister_driver(&acpi_hed_driver);
-}
-
-module_init(acpi_hed_init);
-module_exit(acpi_hed_exit);
+module_acpi_driver(acpi_hed_driver);
 
 ACPI_MODULE_NAME("hed");
 MODULE_AUTHOR("Huang Ying");
diff --git a/drivers/acpi/proc.c b/drivers/acpi/proc.c
index 251c7b6273a9..27adb090bb30 100644
--- a/drivers/acpi/proc.c
+++ b/drivers/acpi/proc.c
@@ -302,26 +302,41 @@ acpi_system_wakeup_device_seq_show(struct seq_file *seq, void *offset)
         list_for_each_safe(node, next, &acpi_wakeup_device_list) {
                 struct acpi_device *dev =
                     container_of(node, struct acpi_device, wakeup_list);
-                struct device *ldev;
+                struct acpi_device_physical_node *entry;
 
                 if (!dev->wakeup.flags.valid)
                         continue;
 
-                ldev = acpi_get_physical_device(dev->handle);
-                seq_printf(seq, "%s\t  S%d\t%c%-8s  ",
+                seq_printf(seq, "%s\t  S%d\t",
                            dev->pnp.bus_id,
-                           (u32) dev->wakeup.sleep_state,
-                           dev->wakeup.flags.run_wake ? '*' : ' ',
-                           (device_may_wakeup(&dev->dev)
-                             || (ldev && device_may_wakeup(ldev))) ?
-                               "enabled" : "disabled");
-                if (ldev)
-                        seq_printf(seq, "%s:%s",
-                                   ldev->bus ? ldev->bus->name : "no-bus",
-                                   dev_name(ldev));
-                seq_printf(seq, "\n");
-                put_device(ldev);
-
+                           (u32) dev->wakeup.sleep_state);
+
+                if (!dev->physical_node_count)
+                        seq_printf(seq, "%c%-8s\n",
+                                dev->wakeup.flags.run_wake ?
+                                '*' : ' ', "disabled");
+                else {
+                        struct device *ldev;
+                        list_for_each_entry(entry, &dev->physical_node_list,
+                                        node) {
+                                ldev = get_device(entry->dev);
+                                if (!ldev)
+                                        continue;
+
+                                if (&entry->node !=
+                                                dev->physical_node_list.next)
+                                        seq_printf(seq, "\t\t");
+
+                                seq_printf(seq, "%c%-8s  %s:%s\n",
+                                        dev->wakeup.flags.run_wake ? '*' : ' ',
+                                        (device_may_wakeup(&dev->dev) ||
+                                        (ldev && device_may_wakeup(ldev))) ?
+                                        "enabled" : "disabled",
+                                        ldev->bus ? ldev->bus->name :
+                                        "no-bus", dev_name(ldev));
+                                put_device(ldev);
+                        }
+                }
         }
         mutex_unlock(&acpi_device_lock);
         return 0;
@@ -329,12 +344,14 @@ acpi_system_wakeup_device_seq_show(struct seq_file *seq, void *offset)
 
 static void physical_device_enable_wakeup(struct acpi_device *adev)
 {
-        struct device *dev = acpi_get_physical_device(adev->handle);
+        struct acpi_device_physical_node *entry;
 
-        if (dev && device_can_wakeup(dev)) {
-                bool enable = !device_may_wakeup(dev);
-                device_set_wakeup_enable(dev, enable);
-        }
+        list_for_each_entry(entry,
+                &adev->physical_node_list, node)
+                if (entry->dev && device_can_wakeup(entry->dev)) {
+                        bool enable = !device_may_wakeup(entry->dev);
+                        device_set_wakeup_enable(entry->dev, enable);
+                }
 }
 
 static ssize_t
diff --git a/drivers/acpi/sbshc.c b/drivers/acpi/sbshc.c
index f8d2a472795c..cf6129a8af7c 100644
--- a/drivers/acpi/sbshc.c
+++ b/drivers/acpi/sbshc.c
@@ -310,23 +310,7 @@ static int acpi_smbus_hc_remove(struct acpi_device *device, int type)
         return 0;
 }
 
-static int __init acpi_smb_hc_init(void)
-{
-        int result;
-
-        result = acpi_bus_register_driver(&acpi_smb_hc_driver);
-        if (result < 0)
-                return -ENODEV;
-        return 0;
-}
-
-static void __exit acpi_smb_hc_exit(void)
-{
-        acpi_bus_unregister_driver(&acpi_smb_hc_driver);
-}
-
-module_init(acpi_smb_hc_init);
-module_exit(acpi_smb_hc_exit);
+module_acpi_driver(acpi_smb_hc_driver);
 
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Alexey Starikovskiy");
diff --git a/drivers/acpi/scan.c b/drivers/acpi/scan.c
index d1ecca2b641a..1fcb8678665c 100644
--- a/drivers/acpi/scan.c
+++ b/drivers/acpi/scan.c
@@ -10,6 +10,7 @@
 #include <linux/signal.h>
 #include <linux/kthread.h>
 #include <linux/dmi.h>
+#include <linux/nls.h>
 
 #include <acpi/acpi_drivers.h>
 
@@ -232,8 +233,35 @@ end:
 }
 static DEVICE_ATTR(path, 0444, acpi_device_path_show, NULL);
 
+/* sysfs file that shows description text from the ACPI _STR method */
+static ssize_t description_show(struct device *dev,
+                                struct device_attribute *attr,
+                                char *buf) {
+        struct acpi_device *acpi_dev = to_acpi_device(dev);
+        int result;
+
+        if (acpi_dev->pnp.str_obj == NULL)
+                return 0;
+
+        /*
+         * The _STR object contains a Unicode identifier for a device.
+         * We need to convert to utf-8 so it can be displayed.
+         */
+        result = utf16s_to_utf8s(
+                (wchar_t *)acpi_dev->pnp.str_obj->buffer.pointer,
+                acpi_dev->pnp.str_obj->buffer.length,
+                UTF16_LITTLE_ENDIAN, buf,
+                PAGE_SIZE);
+
+        buf[result++] = '\n';
+
+        return result;
+}
+static DEVICE_ATTR(description, 0444, description_show, NULL);
+
 static int acpi_device_setup_files(struct acpi_device *dev)
 {
+        struct acpi_buffer buffer = {ACPI_ALLOCATE_BUFFER, NULL};
         acpi_status status;
         acpi_handle temp;
         int result = 0;
@@ -257,6 +285,21 @@ static int acpi_device_setup_files(struct acpi_device *dev)
                         goto end;
         }
 
+        /*
+         * If device has _STR, 'description' file is created
+         */
+        status = acpi_get_handle(dev->handle, "_STR", &temp);
+        if (ACPI_SUCCESS(status)) {
+                status = acpi_evaluate_object(dev->handle, "_STR",
+                                        NULL, &buffer);
+                if (ACPI_FAILURE(status))
+                        buffer.pointer = NULL;
+                dev->pnp.str_obj = buffer.pointer;
+                result = device_create_file(&dev->dev, &dev_attr_description);
+                if (result)
+                        goto end;
+        }
+
         /*
          * If device has _EJ0, 'eject' file is created that is used to trigger
          * hot-removal function from userland.
@@ -274,8 +317,15 @@ static void acpi_device_remove_files(struct acpi_device *dev)
         acpi_handle temp;
 
         /*
-         * If device has _EJ0, 'eject' file is created that is used to trigger
-         * hot-removal function from userland.
+         * If device has _STR, remove 'description' file
+         */
+        status = acpi_get_handle(dev->handle, "_STR", &temp);
+        if (ACPI_SUCCESS(status)) {
+                kfree(dev->pnp.str_obj);
+                device_remove_file(&dev->dev, &dev_attr_description);
+        }
+        /*
+         * If device has _EJ0, remove 'eject' file.
          */
         status = acpi_get_handle(dev->handle, "_EJ0", &temp);
         if (ACPI_SUCCESS(status))
@@ -481,6 +531,8 @@ static int acpi_device_register(struct acpi_device *device)
         INIT_LIST_HEAD(&device->children);
         INIT_LIST_HEAD(&device->node);
         INIT_LIST_HEAD(&device->wakeup_list);
+        INIT_LIST_HEAD(&device->physical_node_list);
+        mutex_init(&device->physical_node_lock);
 
         new_bus_id = kzalloc(sizeof(struct acpi_device_bus_id), GFP_KERNEL);
         if (!new_bus_id) {
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
index f336bca7c450..2572d9715bda 100644
--- a/drivers/acpi/tables.c
+++ b/drivers/acpi/tables.c
@@ -240,10 +240,17 @@ acpi_table_parse_entries(char *id,
                table_end) {
                 if (entry->type == entry_id
                     && (!max_entries || count++ < max_entries))
-                        if (handler(entry, table_end)) {
-                                early_acpi_os_unmap_memory((char *)table_header, tbl_size);
-                                return -EINVAL;
-                        }
+                        if (handler(entry, table_end))
+                                goto err;
+
+                /*
+                 * If entry->length is 0, break from this loop to avoid
+                 * infinite loop.
+                 */
+                if (entry->length == 0) {
+                        pr_err(PREFIX "[%4.4s:0x%02x] Invalid zero length\n", id, entry_id);
+                        goto err;
+                }
 
                 entry = (struct acpi_subtable_header *)
                     ((unsigned long)entry + entry->length);
@@ -255,6 +262,9 @@ acpi_table_parse_entries(char *id,
 
         early_acpi_os_unmap_memory((char *)table_header, tbl_size);
         return count;
+err:
+        early_acpi_os_unmap_memory((char *)table_header, tbl_size);
+        return -EINVAL;
 }
 
 int __init