cpufreq: stats: Mark few conditionals with unlikely()
authorViresh Kumar <viresh.kumar@linaro.org>
Mon, 5 Oct 2020 07:56:03 +0000 (13:26 +0530)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 5 Oct 2020 13:13:43 +0000 (15:13 +0200)
Since this will be part of the scheduler's hotpath in some cases, use
unlikely() for few of the obvious conditionals.

Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/cpufreq/cpufreq_stats.c

index 45a0678..bba04da 100644 (file)
@@ -260,7 +260,7 @@ void cpufreq_stats_record_transition(struct cpufreq_policy *policy,
        struct cpufreq_stats *stats = policy->stats;
        int old_index, new_index;
 
-       if (!stats)
+       if (unlikely(!stats))
                return;
 
        if (unlikely(READ_ONCE(stats->reset_pending)))
@@ -270,7 +270,7 @@ void cpufreq_stats_record_transition(struct cpufreq_policy *policy,
        new_index = freq_table_get_index(stats, new_freq);
 
        /* We can't do stats->time_in_state[-1]= .. */
-       if (old_index == -1 || new_index == -1 || old_index == new_index)
+       if (unlikely(old_index == -1 || new_index == -1 || old_index == new_index))
                return;
 
        cpufreq_stats_update(stats, stats->last_time);