Merge tag 'pm+acpi-3.15-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm

Pull ACPI and power management updates from Rafael Wysocki:
 "The majority of this material spent some time in linux-next, some of
  it even several weeks.  There are a few relatively fresh commits in
  it, but they are mostly fixes and simple cleanups.

  ACPI took the lead this time, both in terms of the number of commits
  and the number of modified lines of code, cpufreq follows and there
  are a few changes in the PM core and in cpuidle too.

  A new feature that already got some LWN.net's attention is the device
  PM QoS extension allowing latency tolerance requirements to be
  propagated from leaf devices to their ancestors with hardware
  interfaces for specifying latency tolerance.  That should help systems
  with hardware-driven power management to avoid going too far with it
  in cases when there are latency tolerance constraints.

  There also are some significant changes in the ACPI core related to
  the way in which hotplug notifications are handled.  They affect PCI
  hotplug (ACPIPHP) and the ACPI dock station code too.  The bottom line
  is that all those notification now go through the root notify handler
  and are propagated to the interested subsystems by means of callbacks
  instead of having to install a notify handler for each device object
  that we can potentially get hotplug notifications for.

  In addition to that ACPICA will now advertise "Windows 2013"
  compatibility for _OSI, because some systems out there don't work
  correctly if that is not done (some of them don't even boot).

  On the system suspend side of things, all of the device suspend and
  resume callbacks, except for ->prepare() and ->complete(), are now
  going to be executed asynchronously as that turns out to speed up
  system suspend and resume on some platforms quite significantly and we
  have a few more optimizations in that area.

  Apart from that, there are some new device IDs and fixes and cleanups
  all over.  In particular, the system suspend and resume handling by
  cpufreq should be improved and the cpuidle menu governor should be a
  bit more robust now.

  Specifics:

   - Device PM QoS support for latency tolerance constraints on systems
     with hardware interfaces allowing such constraints to be specified.
     That is necessary to prevent hardware-driven power management from
     becoming overly aggressive on some systems and to prevent power
     management features leading to excessive latencies from being used
     in some cases.

   - Consolidation of the handling of ACPI hotplug notifications for
     device objects.  This causes all device hotplug notifications to go
     through the root notify handler (that was executed for all of them
     anyway before) that propagates them to individual subsystems, if
     necessary, by executing callbacks provided by those subsystems
     (those callbacks are associated with struct acpi_device objects
     during device enumeration).  As a result, the code in question
     becomes both smaller in size and more straightforward and all of
     those changes should not affect users.

   - ACPICA update, including fixes related to the handling of _PRT in
     cases when it is broken and the addition of "Windows 2013" to the
     list of supported "features" for _OSI (which is necessary to
     support systems that work incorrectly or don't even boot without
     it).  Changes from Bob Moore and Lv Zheng.

   - Consolidation of ACPI _OST handling from Jiang Liu.

   - ACPI battery and AC fixes allowing unusual system configurations to
     be handled by that code from Alexander Mezin.

   - New device IDs for the ACPI LPSS driver from Chiau Ee Chew.

   - ACPI fan and thermal optimizations related to system suspend and
     resume from Aaron Lu.

   - Cleanups related to ACPI video from Jean Delvare.

   - Assorted ACPI fixes and cleanups from Al Stone, Hanjun Guo, Lan
     Tianyu, Paul Bolle, Tomasz Nowicki.

   - Intel RAPL (Running Average Power Limits) driver cleanups from
     Jacob Pan.

   - intel_pstate fixes and cleanups from Dirk Brandewie.

   - cpufreq fixes related to system suspend/resume handling from Viresh
     Kumar.

   - cpufreq core fixes and cleanups from Viresh Kumar, Stratos
     Karafotis, Saravana Kannan, Rashika Kheria, Joe Perches.

   - cpufreq drivers updates from Viresh Kumar, Zhuoyu Zhang, Rob
     Herring.

   - cpuidle fixes related to the menu governor from Tuukka Tikkanen.

   - cpuidle fix related to coupled CPUs handling from Paul Burton.

   - Asynchronous execution of all device suspend and resume callbacks,
     except for ->prepare and ->complete, during system suspend and
     resume from Chuansheng Liu.

   - Delayed resuming of runtime-suspended devices during system suspend
     for the PCI bus type and ACPI PM domain.

   - New set of PM helper routines to allow device runtime PM callbacks
     to be used during system suspend and resume more easily from Ulf
     Hansson.

   - Assorted fixes and cleanups in the PM core from Geert Uytterhoeven,
     Prabhakar Lad, Philipp Zabel, Rashika Kheria, Sebastian Capella.

   - devfreq fix from Saravana Kannan"

* tag 'pm+acpi-3.15-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rafael/linux-pm: (162 commits)
  PM / devfreq: Rewrite devfreq_update_status() to fix multiple bugs
  PM / sleep: Correct whitespace errors in <linux/pm.h>
  intel_pstate: Set core to min P state during core offline
  cpufreq: Add stop CPU callback to cpufreq_driver interface
  cpufreq: Remove unnecessary braces
  cpufreq: Fix checkpatch errors and warnings
  cpufreq: powerpc: add cpufreq transition latency for FSL e500mc SoCs
  MAINTAINERS: Reorder maintainer addresses for PM and ACPI
  PM / Runtime: Update runtime_idle() documentation for return value meaning
  video / output: Drop display output class support
  fujitsu-laptop: Drop unneeded include
  acer-wmi: Stop selecting VIDEO_OUTPUT_CONTROL
  ACPI / gpu / drm: Stop selecting VIDEO_OUTPUT_CONTROL
  ACPI / video: fix ACPI_VIDEO dependencies
  cpufreq: remove unused notifier: CPUFREQ_{SUSPENDCHANGE|RESUMECHANGE}
  cpufreq: Do not allow ->setpolicy drivers to provide ->target
  cpufreq: arm_big_little: set 'physical_cluster' for each CPU
  cpufreq: arm_big_little: make vexpress driver depend on bL core driver
  ACPI / button: Add ACPI Button event via netlink routine
  ACPI: Remove duplicate definitions of PREFIX
  ...

1 files changed, 230 insertions, 50 deletions

diff --git a/drivers/base/power/main.c b/drivers/base/power/main.c
index 1b41fca3d65a..86d5e4fb5b98 100644
--- a/drivers/base/power/main.c
+++ b/drivers/base/power/main.c
@@ -29,6 +29,7 @@
 #include <linux/async.h>
 #include <linux/suspend.h>
 #include <trace/events/power.h>
+#include <linux/cpufreq.h>
 #include <linux/cpuidle.h>
 #include <linux/timer.h>
 
@@ -91,6 +92,8 @@ void device_pm_sleep_init(struct device *dev)
 {
         dev->power.is_prepared = false;
         dev->power.is_suspended = false;
+        dev->power.is_noirq_suspended = false;
+        dev->power.is_late_suspended = false;
         init_completion(&dev->power.completion);
         complete_all(&dev->power.completion);
         dev->power.wakeup = NULL;
@@ -467,7 +470,7 @@ static void dpm_watchdog_clear(struct dpm_watchdog *wd)
  * The driver of @dev will not receive interrupts while this function is being
  * executed.
  */
-static int device_resume_noirq(struct device *dev, pm_message_t state)
+static int device_resume_noirq(struct device *dev, pm_message_t state, bool async)
 {
         pm_callback_t callback = NULL;
         char *info = NULL;
@@ -479,6 +482,11 @@ static int device_resume_noirq(struct device *dev, pm_message_t state)
         if (dev->power.syscore)
                 goto Out;
 
+        if (!dev->power.is_noirq_suspended)
+                goto Out;
+
+        dpm_wait(dev->parent, async);
+
         if (dev->pm_domain) {
                 info = "noirq power domain ";
                 callback = pm_noirq_op(&dev->pm_domain->ops, state);
@@ -499,12 +507,32 @@ static int device_resume_noirq(struct device *dev, pm_message_t state)
         }
 
         error = dpm_run_callback(callback, dev, state, info);
+        dev->power.is_noirq_suspended = false;
 
  Out:
+        complete_all(&dev->power.completion);
         TRACE_RESUME(error);
         return error;
 }
 
+static bool is_async(struct device *dev)
+{
+        return dev->power.async_suspend && pm_async_enabled
+                && !pm_trace_is_enabled();
+}
+
+static void async_resume_noirq(void *data, async_cookie_t cookie)
+{
+        struct device *dev = (struct device *)data;
+        int error;
+
+        error = device_resume_noirq(dev, pm_transition, true);
+        if (error)
+                pm_dev_err(dev, pm_transition, " async", error);
+
+        put_device(dev);
+}
+
 /**
  * dpm_resume_noirq - Execute "noirq resume" callbacks for all devices.
  * @state: PM transition of the system being carried out.
@@ -514,29 +542,48 @@ static int device_resume_noirq(struct device *dev, pm_message_t state)
  */
 static void dpm_resume_noirq(pm_message_t state)
 {
+        struct device *dev;
         ktime_t starttime = ktime_get();
 
         mutex_lock(&dpm_list_mtx);
-        while (!list_empty(&dpm_noirq_list)) {
-                struct device *dev = to_device(dpm_noirq_list.next);
-                int error;
+        pm_transition = state;
 
+        /*
+         * Advanced the async threads upfront,
+         * in case the starting of async threads is
+         * delayed by non-async resuming devices.
+         */
+        list_for_each_entry(dev, &dpm_noirq_list, power.entry) {
+                reinit_completion(&dev->power.completion);
+                if (is_async(dev)) {
+                        get_device(dev);
+                        async_schedule(async_resume_noirq, dev);
+                }
+        }
+
+        while (!list_empty(&dpm_noirq_list)) {
+                dev = to_device(dpm_noirq_list.next);
                 get_device(dev);
                 list_move_tail(&dev->power.entry, &dpm_late_early_list);
                 mutex_unlock(&dpm_list_mtx);
 
-                error = device_resume_noirq(dev, state);
-                if (error) {
-                        suspend_stats.failed_resume_noirq++;
-                        dpm_save_failed_step(SUSPEND_RESUME_NOIRQ);
-                        dpm_save_failed_dev(dev_name(dev));
-                        pm_dev_err(dev, state, " noirq", error);
+                if (!is_async(dev)) {
+                        int error;
+
+                        error = device_resume_noirq(dev, state, false);
+                        if (error) {
+                                suspend_stats.failed_resume_noirq++;
+                                dpm_save_failed_step(SUSPEND_RESUME_NOIRQ);
+                                dpm_save_failed_dev(dev_name(dev));
+                                pm_dev_err(dev, state, " noirq", error);
+                        }
                 }
 
                 mutex_lock(&dpm_list_mtx);
                 put_device(dev);
         }
         mutex_unlock(&dpm_list_mtx);
+        async_synchronize_full();
         dpm_show_time(starttime, state, "noirq");
         resume_device_irqs();
         cpuidle_resume();
@@ -549,7 +596,7 @@ static void dpm_resume_noirq(pm_message_t state)
  *
  * Runtime PM is disabled for @dev while this function is being executed.
  */
-static int device_resume_early(struct device *dev, pm_message_t state)
+static int device_resume_early(struct device *dev, pm_message_t state, bool async)
 {
         pm_callback_t callback = NULL;
         char *info = NULL;
@@ -561,6 +608,11 @@ static int device_resume_early(struct device *dev, pm_message_t state)
         if (dev->power.syscore)
                 goto Out;
 
+        if (!dev->power.is_late_suspended)
+                goto Out;
+
+        dpm_wait(dev->parent, async);
+
         if (dev->pm_domain) {
                 info = "early power domain ";
                 callback = pm_late_early_op(&dev->pm_domain->ops, state);
@@ -581,43 +633,75 @@ static int device_resume_early(struct device *dev, pm_message_t state)
         }
 
         error = dpm_run_callback(callback, dev, state, info);
+        dev->power.is_late_suspended = false;
 
  Out:
         TRACE_RESUME(error);
 
         pm_runtime_enable(dev);
+        complete_all(&dev->power.completion);
         return error;
 }
 
+static void async_resume_early(void *data, async_cookie_t cookie)
+{
+        struct device *dev = (struct device *)data;
+        int error;
+
+        error = device_resume_early(dev, pm_transition, true);
+        if (error)
+                pm_dev_err(dev, pm_transition, " async", error);
+
+        put_device(dev);
+}
+
 /**
  * dpm_resume_early - Execute "early resume" callbacks for all devices.
  * @state: PM transition of the system being carried out.
  */
 static void dpm_resume_early(pm_message_t state)
 {
+        struct device *dev;
         ktime_t starttime = ktime_get();
 
         mutex_lock(&dpm_list_mtx);
-        while (!list_empty(&dpm_late_early_list)) {
-                struct device *dev = to_device(dpm_late_early_list.next);
-                int error;
+        pm_transition = state;
 
+        /*
+         * Advanced the async threads upfront,
+         * in case the starting of async threads is
+         * delayed by non-async resuming devices.
+         */
+        list_for_each_entry(dev, &dpm_late_early_list, power.entry) {
+                reinit_completion(&dev->power.completion);
+                if (is_async(dev)) {
+                        get_device(dev);
+                        async_schedule(async_resume_early, dev);
+                }
+        }
+
+        while (!list_empty(&dpm_late_early_list)) {
+                dev = to_device(dpm_late_early_list.next);
                 get_device(dev);
                 list_move_tail(&dev->power.entry, &dpm_suspended_list);
                 mutex_unlock(&dpm_list_mtx);
 
-                error = device_resume_early(dev, state);
-                if (error) {
-                        suspend_stats.failed_resume_early++;
-                        dpm_save_failed_step(SUSPEND_RESUME_EARLY);
-                        dpm_save_failed_dev(dev_name(dev));
-                        pm_dev_err(dev, state, " early", error);
-                }
+                if (!is_async(dev)) {
+                        int error;
 
+                        error = device_resume_early(dev, state, false);
+                        if (error) {
+                                suspend_stats.failed_resume_early++;
+                                dpm_save_failed_step(SUSPEND_RESUME_EARLY);
+                                dpm_save_failed_dev(dev_name(dev));
+                                pm_dev_err(dev, state, " early", error);
+                        }
+                }
                 mutex_lock(&dpm_list_mtx);
                 put_device(dev);
         }
         mutex_unlock(&dpm_list_mtx);
+        async_synchronize_full();
         dpm_show_time(starttime, state, "early");
 }
 
@@ -732,12 +816,6 @@ static void async_resume(void *data, async_cookie_t cookie)
         put_device(dev);
 }
 
-static bool is_async(struct device *dev)
-{
-        return dev->power.async_suspend && pm_async_enabled
-                && !pm_trace_is_enabled();
-}
-
 /**
  * dpm_resume - Execute "resume" callbacks for non-sysdev devices.
  * @state: PM transition of the system being carried out.
@@ -789,6 +867,8 @@ void dpm_resume(pm_message_t state)
         mutex_unlock(&dpm_list_mtx);
         async_synchronize_full();
         dpm_show_time(starttime, state, NULL);
+
+        cpufreq_resume();
 }
 
 /**
@@ -913,13 +993,24 @@ static pm_message_t resume_event(pm_message_t sleep_state)
  * The driver of @dev will not receive interrupts while this function is being
  * executed.
  */
-static int device_suspend_noirq(struct device *dev, pm_message_t state)
+static int __device_suspend_noirq(struct device *dev, pm_message_t state, bool async)
 {
         pm_callback_t callback = NULL;
         char *info = NULL;
+        int error = 0;
+
+        if (async_error)
+                goto Complete;
+
+        if (pm_wakeup_pending()) {
+                async_error = -EBUSY;
+                goto Complete;
+        }
 
         if (dev->power.syscore)
-                return 0;
+                goto Complete;
+
+        dpm_wait_for_children(dev, async);
 
         if (dev->pm_domain) {
                 info = "noirq power domain ";
@@ -940,7 +1031,41 @@ static int device_suspend_noirq(struct device *dev, pm_message_t state)
                 callback = pm_noirq_op(dev->driver->pm, state);
         }
 
-        return dpm_run_callback(callback, dev, state, info);
+        error = dpm_run_callback(callback, dev, state, info);
+        if (!error)
+                dev->power.is_noirq_suspended = true;
+        else
+                async_error = error;
+
+Complete:
+        complete_all(&dev->power.completion);
+        return error;
+}
+
+static void async_suspend_noirq(void *data, async_cookie_t cookie)
+{
+        struct device *dev = (struct device *)data;
+        int error;
+
+        error = __device_suspend_noirq(dev, pm_transition, true);
+        if (error) {
+                dpm_save_failed_dev(dev_name(dev));
+                pm_dev_err(dev, pm_transition, " async", error);
+        }
+
+        put_device(dev);
+}
+
+static int device_suspend_noirq(struct device *dev)
+{
+        reinit_completion(&dev->power.completion);
+
+        if (pm_async_enabled && dev->power.async_suspend) {
+                get_device(dev);
+                async_schedule(async_suspend_noirq, dev);
+                return 0;
+        }
+        return __device_suspend_noirq(dev, pm_transition, false);
 }
 
 /**
@@ -958,19 +1083,20 @@ static int dpm_suspend_noirq(pm_message_t state)
         cpuidle_pause();
         suspend_device_irqs();
         mutex_lock(&dpm_list_mtx);
+        pm_transition = state;
+        async_error = 0;
+
         while (!list_empty(&dpm_late_early_list)) {
                 struct device *dev = to_device(dpm_late_early_list.prev);
 
                 get_device(dev);
                 mutex_unlock(&dpm_list_mtx);
 
-                error = device_suspend_noirq(dev, state);
+                error = device_suspend_noirq(dev);
 
                 mutex_lock(&dpm_list_mtx);
                 if (error) {
                         pm_dev_err(dev, state, " noirq", error);
-                        suspend_stats.failed_suspend_noirq++;
-                        dpm_save_failed_step(SUSPEND_SUSPEND_NOIRQ);
                         dpm_save_failed_dev(dev_name(dev));
                         put_device(dev);
                         break;
@@ -979,16 +1105,21 @@ static int dpm_suspend_noirq(pm_message_t state)
                         list_move(&dev->power.entry, &dpm_noirq_list);
                 put_device(dev);
 
-                if (pm_wakeup_pending()) {
-                        error = -EBUSY;
+                if (async_error)
                         break;
-                }
         }
         mutex_unlock(&dpm_list_mtx);
-        if (error)
+        async_synchronize_full();
+        if (!error)
+                error = async_error;
+
+        if (error) {
+                suspend_stats.failed_suspend_noirq++;
+                dpm_save_failed_step(SUSPEND_SUSPEND_NOIRQ);
                 dpm_resume_noirq(resume_event(state));
-        else
+        } else {
                 dpm_show_time(starttime, state, "noirq");
+        }
         return error;
 }
 
@@ -999,15 +1130,26 @@ static int dpm_suspend_noirq(pm_message_t state)
  *
  * Runtime PM is disabled for @dev while this function is being executed.
  */
-static int device_suspend_late(struct device *dev, pm_message_t state)
+static int __device_suspend_late(struct device *dev, pm_message_t state, bool async)
 {
         pm_callback_t callback = NULL;
         char *info = NULL;
+        int error = 0;
 
         __pm_runtime_disable(dev, false);
 
+        if (async_error)
+                goto Complete;
+
+        if (pm_wakeup_pending()) {
+                async_error = -EBUSY;
+                goto Complete;
+        }
+
         if (dev->power.syscore)
-                return 0;
+                goto Complete;
+
+        dpm_wait_for_children(dev, async);
 
         if (dev->pm_domain) {
                 info = "late power domain ";
@@ -1028,7 +1170,41 @@ static int device_suspend_late(struct device *dev, pm_message_t state)
                 callback = pm_late_early_op(dev->driver->pm, state);
         }
 
-        return dpm_run_callback(callback, dev, state, info);
+        error = dpm_run_callback(callback, dev, state, info);
+        if (!error)
+                dev->power.is_late_suspended = true;
+        else
+                async_error = error;
+
+Complete:
+        complete_all(&dev->power.completion);
+        return error;
+}
+
+static void async_suspend_late(void *data, async_cookie_t cookie)
+{
+        struct device *dev = (struct device *)data;
+        int error;
+
+        error = __device_suspend_late(dev, pm_transition, true);
+        if (error) {
+                dpm_save_failed_dev(dev_name(dev));
+                pm_dev_err(dev, pm_transition, " async", error);
+        }
+        put_device(dev);
+}
+
+static int device_suspend_late(struct device *dev)
+{
+        reinit_completion(&dev->power.completion);
+
+        if (pm_async_enabled && dev->power.async_suspend) {
+                get_device(dev);
+                async_schedule(async_suspend_late, dev);
+                return 0;
+        }
+
+        return __device_suspend_late(dev, pm_transition, false);
 }
 
 /**
@@ -1041,19 +1217,20 @@ static int dpm_suspend_late(pm_message_t state)
         int error = 0;
 
         mutex_lock(&dpm_list_mtx);
+        pm_transition = state;
+        async_error = 0;
+
         while (!list_empty(&dpm_suspended_list)) {
                 struct device *dev = to_device(dpm_suspended_list.prev);
 
                 get_device(dev);
                 mutex_unlock(&dpm_list_mtx);
 
-                error = device_suspend_late(dev, state);
+                error = device_suspend_late(dev);
 
                 mutex_lock(&dpm_list_mtx);
                 if (error) {
                         pm_dev_err(dev, state, " late", error);
-                        suspend_stats.failed_suspend_late++;
-                        dpm_save_failed_step(SUSPEND_SUSPEND_LATE);
                         dpm_save_failed_dev(dev_name(dev));
                         put_device(dev);
                         break;
@@ -1062,17 +1239,18 @@ static int dpm_suspend_late(pm_message_t state)
                         list_move(&dev->power.entry, &dpm_late_early_list);
                 put_device(dev);
 
-                if (pm_wakeup_pending()) {
-                        error = -EBUSY;
+                if (async_error)
                         break;
-                }
         }
         mutex_unlock(&dpm_list_mtx);
-        if (error)
+        async_synchronize_full();
+        if (error) {
+                suspend_stats.failed_suspend_late++;
+                dpm_save_failed_step(SUSPEND_SUSPEND_LATE);
                 dpm_resume_early(resume_event(state));
-        else
+        } else {
                 dpm_show_time(starttime, state, "late");
-
+        }
         return error;
 }
 
@@ -1259,6 +1437,8 @@ int dpm_suspend(pm_message_t state)
 
         might_sleep();
 
+        cpufreq_suspend();
+
         mutex_lock(&dpm_list_mtx);
         pm_transition = state;
         async_error = 0;