linux-kernel - [PATCH 2/2] thermal: power_allocator: update once cooling devices when temp is low

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-Id: <20210331163352.32416-3-lukasz.luba@arm.com>
Date:   Wed, 31 Mar 2021 17:33:52 +0100
From:   Lukasz Luba <lukasz.luba@....com>
To:     linux-kernel@...r.kernel.org, daniel.lezcano@...aro.org
Cc:     linux-pm@...r.kernel.org, amitk@...nel.org, rui.zhang@...el.com,
        lukasz.luba@....com
Subject: [PATCH 2/2] thermal: power_allocator: update once cooling devices when temp is low

The cooling device state change generates an event, also when there is no
need, because temperature is low and device is not throttled. Avoid to
unnecessary update the cooling device which means also not sending event.
The cooling device state has not changed because the temperature is still
below the first activation trip point value, so we can do this.
Add a tracking mechanism if the update was triggered and makes sure it
will be done only once when the temperature continue to stay below first
trip point.

Reported-by: Daniel Lezcano <daniel.lezcano@...aro.org>
Signed-off-by: Lukasz Luba <lukasz.luba@....com>
---
 drivers/thermal/gov_power_allocator.c | 21 +++++++++++++++++++--
 1 file changed, 19 insertions(+), 2 deletions(-)

diff --git a/drivers/thermal/gov_power_allocator.c b/drivers/thermal/gov_power_allocator.c
index 0cbd10cab193..6085b6661b8b 100644
--- a/drivers/thermal/gov_power_allocator.c
+++ b/drivers/thermal/gov_power_allocator.c
@@ -65,6 +65,8 @@ static inline s64 div_frac(s64 x, s64 y)
  *					controlling for.
  * @sustainable_power:	Sustainable power (heat) that this thermal zone can
  *			dissipate
+ * @updated:		Keeps track if the framework was updated when the
+ *			temperature went below the first trip point.
  */
 struct power_allocator_params {
 	bool allocated_tzp;
@@ -73,6 +75,7 @@ struct power_allocator_params {
 	int trip_switch_on;
 	int trip_max_desired_temperature;
 	u32 sustainable_power;
+	bool updated;
 };
 
 /**
@@ -395,6 +398,12 @@ static int allocate_power(struct thermal_zone_device *tz,
 
 	mutex_lock(&tz->lock);
 
+	/*
+	 * The temperature has crossed the first trip point, so next time
+	 * when it drops below, update cooling devices and framework.
+	 */
+	params->updated = false;
+
 	num_actors = 0;
 	total_weight = 0;
 	list_for_each_entry(instance, &tz->thermal_instances, tz_node) {
@@ -596,10 +605,18 @@ static void allow_maximum_power(struct thermal_zone_device *tz)
 		 */
 		cdev->ops->get_requested_power(cdev, &req_power);
 
-		instance->cdev->updated = false;
+		if (!params->updated)
+			instance->cdev->updated = false;
+
 		mutex_unlock(&instance->cdev->lock);
-		thermal_cdev_update(instance->cdev);
+
+		if (!params->updated)
+			thermal_cdev_update(instance->cdev);
 	}
+
+	/* Don't send updates next time when temperature is low. */
+	params->updated = true;
+
 	mutex_unlock(&tz->lock);
 }
 
-- 
2.17.1