Thread (4 messages) 4 messages, 3 authors, 2021-02-19

Re: [PATCH] cpufreq: schedutil: Remove comment to update_lock of struct sugov_policy

From: Viresh Kumar <viresh.kumar@linaro.org>
Date: 2021-02-18 10:50:52

Always mention version number and version history in a patch. Don't
resend it now anyway.

On 18-02-21, 17:37, Yue Hu wrote:
quoted hunk ↗ jump to hunk
From: Yue Hu <redacted>

Currently, update_lock is also used in sugov_update_single_freq(). The
comment is not helpful anymore.

Signed-off-by: Yue Hu <redacted>
---
 kernel/sched/cpufreq_schedutil.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/kernel/sched/cpufreq_schedutil.c b/kernel/sched/cpufreq_schedutil.c
index add8081..fd43d59 100644
--- a/kernel/sched/cpufreq_schedutil.c
+++ b/kernel/sched/cpufreq_schedutil.c
@@ -26,7 +26,7 @@ struct sugov_policy {
 	struct sugov_tunables	*tunables;
 	struct list_head	tunables_hook;
 
-	raw_spinlock_t		update_lock;	/* For shared policies */
+	raw_spinlock_t		update_lock;
 	u64			last_freq_update_time;
 	s64			freq_update_delay_ns;
 	unsigned int		next_freq;
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>

-- 
viresh
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help