Re: [PATCH] thermal: Schedule a backup thermal shutdown workqueue after a known period of time to tackle failed poweroff

[Keerthy]

Please ignore this patch. I will resend the right one.
Sorry about the noise.


On Monday 21 December 2015 11:14 AM, Keerthy wrote:

Signed-off-by: Keerthy <j-keer...@ti.com>
---
  drivers/thermal/Kconfig        |  9 +++++++++
  drivers/thermal/thermal_core.c | 26 ++++++++++++++++++++++++++
  2 files changed, 35 insertions(+)

diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig
index 8cc4ac6..25584ee 100644
--- a/drivers/thermal/Kconfig
+++ b/drivers/thermal/Kconfig
@@ -92,6 +92,15 @@ config THERMAL_DEFAULT_GOV_POWER_ALLOCATOR

  endchoice

+config THERMAL_BKUP_SHUTDOWN_DELAY_MS
+        int "Thermal shutdown  backup delay in milli-seconds"
+        depends on THERMAL
+        default "5000"
+        ---help---
+        The number of milliseconds to delay after orderly_poweroff call.
+
+        Default: 5000 (5 seconds)
+
  config THERMAL_GOV_FAIR_SHARE
        bool "Fair-share thermal governor"
        help
diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index d9e525c..b793857 100644
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -61,6 +61,12 @@ static DEFINE_MUTEX(thermal_governor_lock);

  static struct thermal_governor *def_governor;

+#ifdef CONFIG_THERMAL_BKUP_SHUTDOWN_DELAY_MS
+#define BKUP_SHUTDOWN_DELAY CONFIG_THERMAL_BKUP_SHUTDOWN_DELAY_MS
+#else
+#define BKUP_SHUTDOWN_DELAY 5000
+#endif
+
  static struct thermal_governor *__find_governor(const char *name)
  {
        struct thermal_governor *pos;
@@ -423,6 +429,18 @@ static void handle_non_critical_trips(struct 
thermal_zone_device *tz,
                       def_governor->throttle(tz, trip);
  }

+static void bkup_shutdown_func(struct work_struct *unused);
+static DECLARE_DELAYED_WORK(bkup_shutdown_work, bkup_shutdown_func);
+
+static void bkup_shutdown_func(struct work_struct *work)
+{
+       pr_warn("Orderly_poweroff has failed! Attempting kernel_power_off\n");
+       kernel_power_off();
+
+       pr_warn("kernel_power_off has failed! Attempting emergency_restart\n");
+       emergency_restart();
+}
+
  static void handle_critical_trips(struct thermal_zone_device *tz,
                                int trip, enum thermal_trip_type trip_type)
  {
@@ -443,6 +461,14 @@ static void handle_critical_trips(struct 
thermal_zone_device *tz,
                dev_emerg(&tz->device,
                          "critical temperature reached(%d C),shutting down\n",
                          tz->temperature / 1000);
+               /**
+                * This is a backup option to shutdown the
+                * system in case orderly_poweroff
+                * fails
+                */
+               schedule_delayed_work(&bkup_shutdown_work,
+                                     msecs_to_jiffies(BKUP_SHUTDOWN_DELAY));
+
                orderly_poweroff(true);
        }
  }

--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html