Re: [PATCH v3 1/4] thermal: send notifications for cdev state change

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks good to me.

The only platforms I've ever seen with substantial numbers of cooling devices whose states change relatively frequently are NVIDIA's. So I don't think this will be a performance concern.

-Matt Longnecker

On 03/29/2016 05:41 PM, Srikar Srimath Tirumala wrote:
Send notifications using sysfs_notify to indicate that a cooling
device cur state has changed. This can alert the listeners that
hardware clocks are about to be throttled.

A listening user space application can use this notification to
perform workload throttling.

Change-Id: I71cd6d11bff87f181fa3b7db36bf6c953d1d77eb
Signed-off-by: Srikar Srimath Tirumala <srikars@xxxxxxxxxx>
---
  drivers/thermal/thermal_core.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/drivers/thermal/thermal_core.c b/drivers/thermal/thermal_core.c
index d4b5465..b09fff1 100644
--- a/drivers/thermal/thermal_core.c
+++ b/drivers/thermal/thermal_core.c
@@ -1642,6 +1642,7 @@ void thermal_cdev_update(struct thermal_cooling_device *cdev)
  	cdev->updated = true;
  	trace_cdev_update(cdev, target);
  	dev_dbg(&cdev->device, "set to state %lu\n", target);
+	sysfs_notify(&cdev->device.kobj, NULL, "cur_state");
  }
  EXPORT_SYMBOL(thermal_cdev_update);

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



[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux