]> nv-tegra.nvidia Code Review - linux-2.6.git/commitdiff
Merge branches 'release' and 'gpe-ack' into release
authorLen Brown <len.brown@intel.com>
Thu, 7 Feb 2008 08:11:47 +0000 (03:11 -0500)
committerLen Brown <len.brown@intel.com>
Thu, 7 Feb 2008 08:11:47 +0000 (03:11 -0500)
1  2 
drivers/acpi/events/evgpe.c
drivers/acpi/osl.c

index 056b78844829b51bcec0eecf7e1f62dbbddbdf8b,b4509f93ff8172ecea831b47f054fbb25d06b6f9..31cb8cd8cabeda504dacc71c90e39be334b82598
@@@ -270,18 -270,18 +270,18 @@@ acpi_status acpi_ev_disable_gpe(struct 
        case ACPI_GPE_TYPE_WAKE_RUN:
                ACPI_CLEAR_BIT(gpe_event_info->flags, ACPI_GPE_WAKE_ENABLED);
  
 -              /*lint -fallthrough */
 +              /* fallthrough */
  
        case ACPI_GPE_TYPE_RUNTIME:
  
                /* Disable the requested runtime GPE */
  
                ACPI_CLEAR_BIT(gpe_event_info->flags, ACPI_GPE_RUN_ENABLED);
 -              status = acpi_hw_write_gpe_enable_reg(gpe_event_info);
 -              break;
 +
 +              /* fallthrough */
  
        default:
 -              return_ACPI_STATUS(AE_BAD_PARAMETER);
 +              acpi_hw_write_gpe_enable_reg(gpe_event_info);
        }
  
        return_ACPI_STATUS(AE_OK);
@@@ -501,6 -501,7 +501,7 @@@ u32 acpi_ev_gpe_detect(struct acpi_gpe_
   *              an interrupt handler.
   *
   ******************************************************************************/
+ static void acpi_ev_asynch_enable_gpe(void *context);
  
  static void ACPI_SYSTEM_XFACE acpi_ev_asynch_execute_gpe_method(void *context)
  {
                                         method_node)));
                }
        }
+       /* Defer enabling of GPE until all notify handlers are done */
+       acpi_os_execute(OSL_NOTIFY_HANDLER, acpi_ev_asynch_enable_gpe,
+                               gpe_event_info);
+       return_VOID;
+ }
  
-       if ((local_gpe_event_info.flags & ACPI_GPE_XRUPT_TYPE_MASK) ==
+ static void acpi_ev_asynch_enable_gpe(void *context)
+ {
+       struct acpi_gpe_event_info *gpe_event_info = context;
+       acpi_status status;
+       if ((gpe_event_info->flags & ACPI_GPE_XRUPT_TYPE_MASK) ==
            ACPI_GPE_LEVEL_TRIGGERED) {
                /*
                 * GPE is level-triggered, we clear the GPE status bit after
                 * handling the event.
                 */
-               status = acpi_hw_clear_gpe(&local_gpe_event_info);
+               status = acpi_hw_clear_gpe(gpe_event_info);
                if (ACPI_FAILURE(status)) {
                        return_VOID;
                }
        }
  
        /* Enable this GPE */
-       (void)acpi_hw_write_gpe_enable_reg(&local_gpe_event_info);
+       (void)acpi_hw_write_gpe_enable_reg(gpe_event_info);
        return_VOID;
  }
  
diff --combined drivers/acpi/osl.c
index 9772a48f36f1e1938404479312472368a397cd59,21d34595dbae4a7f5e5d5991d55e52a1f54cc95f..07c18dbde698bbba86c17d7894a6ff5360abdcba
@@@ -77,55 -77,11 +77,55 @@@ static struct workqueue_struct *kacpi_n
  #define       OSI_STRING_LENGTH_MAX 64        /* arbitrary */
  static char osi_additional_string[OSI_STRING_LENGTH_MAX];
  
 -static int osi_linux;         /* disable _OSI(Linux) by default */
 +/*
 + * "Ode to _OSI(Linux)"
 + *
 + * osi_linux -- Control response to BIOS _OSI(Linux) query.
 + *
 + * As Linux evolves, the features that it supports change.
 + * So an OSI string such as "Linux" is not specific enough
 + * to be useful across multiple versions of Linux.  It
 + * doesn't identify any particular feature, interface,
 + * or even any particular version of Linux...
 + *
 + * Unfortunately, Linux-2.6.22 and earlier responded "yes"
 + * to a BIOS _OSI(Linux) query.  When
 + * a reference mobile BIOS started using it, its use
 + * started to spread to many vendor platforms.
 + * As it is not supportable, we need to halt that spread.
 + *
 + * Today, most BIOS references to _OSI(Linux) are noise --
 + * they have no functional effect and are just dead code
 + * carried over from the reference BIOS.
 + *
 + * The next most common case is that _OSI(Linux) harms Linux,
 + * usually by causing the BIOS to follow paths that are
 + * not tested during Windows validation.
 + *
 + * Finally, there is a short list of platforms
 + * where OSI(Linux) benefits Linux.
 + *
 + * In Linux-2.6.23, OSI(Linux) is first disabled by default.
 + * DMI is used to disable the dmesg warning about OSI(Linux)
 + * on platforms where it is known to have no effect.
 + * But a dmesg warning remains for systems where
 + * we do not know if OSI(Linux) is good or bad for the system.
 + * DMI is also used to enable OSI(Linux) for the machines
 + * that are known to need it.
 + *
 + * BIOS writers should NOT query _OSI(Linux) on future systems.
 + * It will be ignored by default, and to get Linux to
 + * not ignore it will require a kernel source update to
 + * add a DMI entry, or a boot-time "acpi_osi=Linux" invocation.
 + */
 +#define OSI_LINUX_ENABLE 0
  
 -#ifdef CONFIG_DMI
 -static struct __initdata dmi_system_id acpi_osl_dmi_table[];
 -#endif
 +static struct osi_linux {
 +      unsigned int    enable:1;
 +      unsigned int    dmi:1;
 +      unsigned int    cmdline:1;
 +      unsigned int    known:1;
 +} osi_linux = { OSI_LINUX_ENABLE, 0, 0, 0};
  
  static void __init acpi_request_region (struct acpi_generic_address *addr,
        unsigned int length, char *desc)
@@@ -177,6 -133,7 +177,6 @@@ device_initcall(acpi_reserve_resources)
  
  acpi_status __init acpi_os_initialize(void)
  {
 -      dmi_check_system(acpi_osl_dmi_table);
        return AE_OK;
  }
  
@@@ -250,12 -207,8 +250,12 @@@ acpi_physical_address __init acpi_os_ge
                               "System description tables not found\n");
                        return 0;
                }
 -      } else
 -              return acpi_find_rsdp();
 +      } else {
 +              acpi_physical_address pa = 0;
 +
 +              acpi_find_root_pointer(&pa);
 +              return pa;
 +      }
  }
  
  void __iomem *acpi_os_map_memory(acpi_physical_address phys, acpi_size size)
@@@ -665,25 -618,6 +665,6 @@@ static void acpi_os_execute_deferred(st
        dpc->function(dpc->context);
        kfree(dpc);
  
-       /* Yield cpu to notify thread */
-       cond_resched();
-       return;
- }
- static void acpi_os_execute_notify(struct work_struct *work)
- {
-       struct acpi_os_dpc *dpc = container_of(work, struct acpi_os_dpc, work);
-       if (!dpc) {
-               printk(KERN_ERR PREFIX "Invalid (NULL) context\n");
-               return;
-       }
-       dpc->function(dpc->context);
-       kfree(dpc);
        return;
  }
  
@@@ -707,7 -641,7 +688,7 @@@ acpi_status acpi_os_execute(acpi_execut
  {
        acpi_status status = AE_OK;
        struct acpi_os_dpc *dpc;
+       struct workqueue_struct *queue;
        ACPI_DEBUG_PRINT((ACPI_DB_EXEC,
                          "Scheduling function [%p(%p)] for deferred execution.\n",
                          function, context));
        dpc->function = function;
        dpc->context = context;
  
-       if (type == OSL_NOTIFY_HANDLER) {
-               INIT_WORK(&dpc->work, acpi_os_execute_notify);
-               if (!queue_work(kacpi_notify_wq, &dpc->work)) {
-                       status = AE_ERROR;
-                       kfree(dpc);
-               }
-       } else {
-               INIT_WORK(&dpc->work, acpi_os_execute_deferred);
-               if (!queue_work(kacpid_wq, &dpc->work)) {
-                       ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
-                                 "Call to queue_work() failed.\n"));
-                       status = AE_ERROR;
-                       kfree(dpc);
-               }
+       INIT_WORK(&dpc->work, acpi_os_execute_deferred);
+       queue = (type == OSL_NOTIFY_HANDLER) ? kacpi_notify_wq : kacpid_wq;
+       if (!queue_work(queue, &dpc->work)) {
+               ACPI_DEBUG_PRINT((ACPI_DB_ERROR,
+                         "Call to queue_work() failed.\n"));
+               status = AE_ERROR;
+               kfree(dpc);
        }
        return_ACPI_STATUS(status);
  }
@@@ -1011,37 -938,13 +985,37 @@@ static int __init acpi_os_name_setup(ch
  
  __setup("acpi_os_name=", acpi_os_name_setup);
  
 -static void enable_osi_linux(int enable) {
 +static void __init set_osi_linux(unsigned int enable)
 +{
 +      if (osi_linux.enable != enable) {
 +              osi_linux.enable = enable;
 +              printk(KERN_NOTICE PREFIX "%sed _OSI(Linux)\n",
 +                      enable ? "Add": "Delet");
 +      }
 +      return;
 +}
 +
 +static void __init acpi_cmdline_osi_linux(unsigned int enable)
 +{
 +      osi_linux.cmdline = 1;  /* cmdline set the default */
 +      set_osi_linux(enable);
 +
 +      return;
 +}
 +
 +void __init acpi_dmi_osi_linux(int enable, const struct dmi_system_id *d)
 +{
 +      osi_linux.dmi = 1;      /* DMI knows that this box asks OSI(Linux) */
 +
 +      printk(KERN_NOTICE PREFIX "DMI detected: %s\n", d->ident);
 +
 +      if (enable == -1)
 +              return;
 +
 +      osi_linux.known = 1;    /* DMI knows which OSI(Linux) default needed */
  
 -      if (osi_linux != enable)
 -              printk(KERN_INFO PREFIX "%sabled _OSI(Linux)\n",
 -                      enable ? "En": "Dis");
 +      set_osi_linux(enable);
  
 -      osi_linux = enable;
        return;
  }
  
@@@ -1058,12 -961,12 +1032,12 @@@ static int __init acpi_osi_setup(char *
                printk(KERN_INFO PREFIX "_OSI method disabled\n");
                acpi_gbl_create_osi_method = FALSE;
        } else if (!strcmp("!Linux", str)) {
 -              enable_osi_linux(0);
 +              acpi_cmdline_osi_linux(0);      /* !enable */
        } else if (*str == '!') {
                if (acpi_osi_invalidate(++str) == AE_OK)
                        printk(KERN_INFO PREFIX "Deleted _OSI(%s)\n", str);
        } else if (!strcmp("Linux", str)) {
 -              enable_osi_linux(1);
 +              acpi_cmdline_osi_linux(1);      /* enable */
        } else if (*osi_additional_string == '\0') {
                strncpy(osi_additional_string, str, OSI_STRING_LENGTH_MAX);
                printk(KERN_INFO PREFIX "Added _OSI(%s)\n", str);
@@@ -1212,34 -1115,6 +1186,34 @@@ acpi_status acpi_os_release_object(acpi
        return (AE_OK);
  }
  
 +/**
 + *    acpi_dmi_dump - dump DMI slots needed for blacklist entry
 + *
 + *    Returns 0 on success
 + */
 +static int acpi_dmi_dump(void)
 +{
 +
 +      if (!dmi_available)
 +              return -1;
 +
 +      printk(KERN_NOTICE PREFIX "DMI System Vendor: %s\n",
 +              dmi_get_system_info(DMI_SYS_VENDOR));
 +      printk(KERN_NOTICE PREFIX "DMI Product Name: %s\n",
 +              dmi_get_system_info(DMI_PRODUCT_NAME));
 +      printk(KERN_NOTICE PREFIX "DMI Product Version: %s\n",
 +              dmi_get_system_info(DMI_PRODUCT_VERSION));
 +      printk(KERN_NOTICE PREFIX "DMI Board Name: %s\n",
 +              dmi_get_system_info(DMI_BOARD_NAME));
 +      printk(KERN_NOTICE PREFIX "DMI BIOS Vendor: %s\n",
 +              dmi_get_system_info(DMI_BIOS_VENDOR));
 +      printk(KERN_NOTICE PREFIX "DMI BIOS Date: %s\n",
 +              dmi_get_system_info(DMI_BIOS_DATE));
 +
 +      return 0;
 +}
 +
 +
  /******************************************************************************
   *
   * FUNCTION:    acpi_os_validate_interface
@@@ -1259,29 -1134,13 +1233,29 @@@ acpi_os_validate_interface (char *inter
        if (!strncmp(osi_additional_string, interface, OSI_STRING_LENGTH_MAX))
                return AE_OK;
        if (!strcmp("Linux", interface)) {
 -              printk(KERN_WARNING PREFIX
 -                      "System BIOS is requesting _OSI(Linux)\n");
 -              printk(KERN_WARNING PREFIX
 -                      "If \"acpi_osi=Linux\" works better,\n"
 -                      "Please send dmidecode "
 -                      "to linux-acpi@vger.kernel.org\n");
 -              if(osi_linux)
 +
 +              printk(KERN_NOTICE PREFIX
 +                      "BIOS _OSI(Linux) query %s%s\n",
 +                      osi_linux.enable ? "honored" : "ignored",
 +                      osi_linux.cmdline ? " via cmdline" :
 +                      osi_linux.dmi ? " via DMI" : "");
 +
 +              if (!osi_linux.dmi) {
 +                      if (acpi_dmi_dump())
 +                              printk(KERN_NOTICE PREFIX
 +                                      "[please extract dmidecode output]\n");
 +                      printk(KERN_NOTICE PREFIX
 +                              "Please send DMI info above to "
 +                              "linux-acpi@vger.kernel.org\n");
 +              }
 +              if (!osi_linux.known && !osi_linux.cmdline) {
 +                      printk(KERN_NOTICE PREFIX
 +                              "If \"acpi_osi=%sLinux\" works better, "
 +                              "please notify linux-acpi@vger.kernel.org\n",
 +                              osi_linux.enable ? "!" : "");
 +              }
 +
 +              if (osi_linux.enable)
                        return AE_OK;
        }
        return AE_SUPPORT;
@@@ -1313,4 -1172,28 +1287,4 @@@ acpi_os_validate_address 
      return AE_OK;
  }
  
 -#ifdef CONFIG_DMI
 -static int dmi_osi_linux(const struct dmi_system_id *d)
 -{
 -      printk(KERN_NOTICE "%s detected: enabling _OSI(Linux)\n", d->ident);
 -      enable_osi_linux(1);
 -      return 0;
 -}
 -
 -static struct dmi_system_id acpi_osl_dmi_table[] __initdata = {
 -      /*
 -       * Boxes that need _OSI(Linux)
 -       */
 -      {
 -       .callback = dmi_osi_linux,
 -       .ident = "Intel Napa CRB",
 -       .matches = {
 -                   DMI_MATCH(DMI_BOARD_VENDOR, "Intel Corporation"),
 -                   DMI_MATCH(DMI_BOARD_NAME, "MPAD-MSAE Customer Reference Boards"),
 -                   },
 -       },
 -      {}
 -};
 -#endif /* CONFIG_DMI */
 -
  #endif