Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 410a1fce authored by Saravana Kannan's avatar Saravana Kannan Committed by Jonathan Avila
Browse files

cpufreq: schedutil: Add tracepoints for debugging 



Add trace points when load gets updated and when frequency is chosen
to facilitate better debugging.

Change-Id: Ib19911a03fc941bc7db76ab07394e152d0572871
Signed-off-by: default avatarSaravana Kannan <skannan@codeaurora.org>
Signed-off-by: default avatarRohit Gupta <rohgup@codeaurora.org>
[avilaj@codeaurora.org: Fix some conflicts introduced by reordering this patch]
Signed-off-by: default avatarJonathan Avila <avilaj@codeaurora.org>
parent fc876615

include/trace/events/power.h

+54 −0
Original line number Diff line number Diff line
@@ -571,6 +571,60 @@ DEFINE_EVENT(dev_pm_qos_request, dev_pm_qos_remove_request, 


	TP_ARGS(name, type, new_value)

);



TRACE_EVENT(sugov_util_update,

	    TP_PROTO(int cpu,

		     unsigned long util, unsigned long avg_cap,

		     unsigned long max_cap, unsigned long nl, unsigned long pl,

		     unsigned int flags),

	    TP_ARGS(cpu, util, avg_cap, max_cap, nl, pl, flags),

	    TP_STRUCT__entry(

		    __field(int, cpu)

		    __field(unsigned long, util)

		    __field(unsigned long, avg_cap)

		    __field(unsigned long, max_cap)

		    __field(unsigned long, nl)

		    __field(unsigned long, pl)

		    __field(unsigned int, flags)

	    ),

	    TP_fast_assign(

		    __entry->cpu = cpu;

		    __entry->util = util;

		    __entry->avg_cap = avg_cap;

		    __entry->max_cap = max_cap;

		    __entry->nl = nl;

		    __entry->pl = pl;

		    __entry->flags = flags;

	    ),

	    TP_printk("cpu=%d util=%lu avg_cap=%lu max_cap=%lu nl=%lu pl=%lu flags=0x%x",

		      __entry->cpu, __entry->util, __entry->avg_cap,

		      __entry->max_cap, __entry->nl,

		      __entry->pl, __entry->flags)

);



TRACE_EVENT(sugov_next_freq,

	    TP_PROTO(unsigned int cpu, unsigned long util, unsigned long max,

		     unsigned int freq),

	    TP_ARGS(cpu, util, max, freq),

	    TP_STRUCT__entry(

		    __field(unsigned int, cpu)

		    __field(unsigned long, util)

		    __field(unsigned long, max)

		    __field(unsigned int, freq)

	    ),

	    TP_fast_assign(

		    __entry->cpu = cpu;

		    __entry->util = util;

		    __entry->max = max;

		    __entry->freq = freq;

	    ),

	    TP_printk("cpu=%u util=%lu max=%lu freq=%u",

		      __entry->cpu,

		      __entry->util,

		      __entry->max,

		      __entry->freq)

);



#endif /* _TRACE_POWER_H */



/* This part must be outside protection */

kernel/sched/cpufreq_schedutil.c

+9 −0
Original line number Diff line number Diff line
@@ -212,6 +212,7 @@ static unsigned int get_next_freq(struct sugov_policy *sg_policy, 
				policy->cpuinfo.max_freq : policy->cur;



	freq = map_util_freq(util, freq, max);

	trace_sugov_next_freq(policy->cpu, util, max, freq);



	if (freq == sg_policy->cached_raw_freq && !sg_policy->need_freq_update)

		return sg_policy->next_freq;
@@ -602,6 +603,10 @@ static void sugov_update_single(struct update_util_data *hook, u64 time, 
		sg_policy->hispeed_util = hs_util;

	}



	trace_sugov_util_update(sg_cpu->cpu, sg_cpu->util,

				sg_policy->avg_cap, max, sg_cpu->walt_load.nl,

				sg_cpu->walt_load.pl, flags);



	sugov_iowait_apply(sg_cpu, time, &util, &max);

	sugov_calc_avg_cap(sg_policy, sg_cpu->walt_load.ws,

			   sg_policy->policy->cur);
@@ -708,6 +713,10 @@ sugov_update_shared(struct update_util_data *hook, u64 time, unsigned int flags) 
			   sg_policy->policy->cur);

	ignore_dl_rate_limit(sg_cpu, sg_policy);



	trace_sugov_util_update(sg_cpu->cpu, sg_cpu->util, sg_policy->avg_cap,

				sg_cpu->max, sg_cpu->walt_load.nl,

				sg_cpu->walt_load.pl, flags);



	if (sugov_should_update_freq(sg_policy, time)) {

		next_f = sugov_next_freq_shared(sg_cpu, time);