Received-SPF: pass (google.com: domain of linux-kernel+bounces-207843-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.199.223 as permitted sender) client-ip=147.75.199.223;
Message-ID: <dd446bf6-05ca-4861-b3b1-f113c3528f13@linaro.org>
Date: Mon, 10 Jun 2024 10:28:41 +0200
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 2/8] thermal/debugfs: Do not extend mitigation episodes
 beyond system resume
To: "Rafael J. Wysocki" <rjw@rjwysocki.net>,
 Linux PM <linux-pm@vger.kernel.org>
Cc: LKML <linux-kernel@vger.kernel.org>, "Rafael J. Wysocki"
 <rafael@kernel.org>, Lukasz Luba <lukasz.luba@arm.com>
References: <5794974.DvuYhMxLoT@kreacher> <2337425.ElGaqSPkdT@kreacher>
Content-Language: en-US
From: Daniel Lezcano <daniel.lezcano@linaro.org>
In-Reply-To: <2337425.ElGaqSPkdT@kreacher>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit

On 28/05/2024 16:53, Rafael J. Wysocki wrote:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> 
> Because thermal zone handling by the thermal core is started from
> scratch during resume from system-wide suspend, prevent the debug
> code from extending mitigation episodes beyond that point by ending
> the mitigation episode currently in progress, if any, for each thermal
> zone.

Why it is done at resume time and not at suspend time ?

> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
> ---
> 
> v1 -> v2: Rebase.
> 
> ---
>   drivers/thermal/thermal_core.c    |    1 +
>   drivers/thermal/thermal_debugfs.c |   36 ++++++++++++++++++++++++++++++++++++
>   drivers/thermal/thermal_debugfs.h |    2 ++
>   3 files changed, 39 insertions(+)
> 
> Index: linux-pm/drivers/thermal/thermal_core.c
> ===================================================================
> --- linux-pm.orig/drivers/thermal/thermal_core.c
> +++ linux-pm/drivers/thermal/thermal_core.c
> @@ -1641,6 +1641,7 @@ static void thermal_zone_device_resume(s
>   
>   	tz->suspended = false;
>   
> +	thermal_debug_tz_resume(tz);
>   	thermal_zone_device_init(tz);
>   	__thermal_zone_device_update(tz, THERMAL_EVENT_UNSPECIFIED);
>   
> Index: linux-pm/drivers/thermal/thermal_debugfs.c
> ===================================================================
> --- linux-pm.orig/drivers/thermal/thermal_debugfs.c
> +++ linux-pm/drivers/thermal/thermal_debugfs.c
> @@ -926,3 +926,39 @@ void thermal_debug_tz_remove(struct ther
>   	thermal_debugfs_remove_id(thermal_dbg);
>   	kfree(trips_crossed);
>   }
> +
> +void thermal_debug_tz_resume(struct thermal_zone_device *tz)
> +{
> +	struct thermal_debugfs *thermal_dbg = tz->debugfs;
> +	ktime_t now = ktime_get();
> +	struct tz_debugfs *tz_dbg;
> +	struct tz_episode *tze;
> +	int i;
> +
> +	if (!thermal_dbg)
> +		return;
> +
> +	mutex_lock(&thermal_dbg->lock);
> +
> +	tz_dbg = &thermal_dbg->tz_dbg;
> +
> +	if (!tz_dbg->nr_trips)
> +		goto out;
> +
> +	/*
> +	 * A mitigation episode was in progress before the preceding system
> +	 * suspend transition, so close it because the zone handling is starting
> +	 * over from scratch.
> +	 */
> +	tze = list_first_entry(&tz_dbg->tz_episodes, struct tz_episode, node);
> +
> +	for (i = 0; i < tz_dbg->nr_trips; i++)
> +		tz_episode_close_trip(tze, tz_dbg->trips_crossed[i], now);
> +
> +	tze->duration = ktime_sub(now, tze->timestamp);
> +
> +	tz_dbg->nr_trips = 0;
> +
> +out:
> +	mutex_unlock(&thermal_dbg->lock);
> +}
> Index: linux-pm/drivers/thermal/thermal_debugfs.h
> ===================================================================
> --- linux-pm.orig/drivers/thermal/thermal_debugfs.h
> +++ linux-pm/drivers/thermal/thermal_debugfs.h
> @@ -7,6 +7,7 @@ void thermal_debug_cdev_remove(struct th
>   void thermal_debug_cdev_state_update(const struct thermal_cooling_device *cdev, int state);
>   void thermal_debug_tz_add(struct thermal_zone_device *tz);
>   void thermal_debug_tz_remove(struct thermal_zone_device *tz);
> +void thermal_debug_tz_resume(struct thermal_zone_device *tz);
>   void thermal_debug_tz_trip_up(struct thermal_zone_device *tz,
>   			      const struct thermal_trip *trip);
>   void thermal_debug_tz_trip_down(struct thermal_zone_device *tz,
> @@ -20,6 +21,7 @@ static inline void thermal_debug_cdev_st
>   						   int state) {}
>   static inline void thermal_debug_tz_add(struct thermal_zone_device *tz) {}
>   static inline void thermal_debug_tz_remove(struct thermal_zone_device *tz) {}
> +static inline void thermal_debug_tz_resume(struct thermal_zone_device *tz) {}
>   static inline void thermal_debug_tz_trip_up(struct thermal_zone_device *tz,
>   					    const struct thermal_trip *trip) {};
>   static inline void thermal_debug_tz_trip_down(struct thermal_zone_device *tz,
> 
> 
> 

-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog