Commit b7e7fffd authored by Rafael J. Wysocki's avatar Rafael J. Wysocki
Browse files

cpuidle: teo: Get rid of redundant check in teo_update() 



Notice that setting measured_us to UINT_MAX in teo_update() earlier
doesn't change the behavior of the following code, so do that and
eliminate a redundant check used for setting measured_us to UINT_MAX.

This change is not expected to alter functionality.

Signed-off-by: default avatarRafael J. Wysocki <rafael.j.wysocki@intel.com>
parent cab09f3d

drivers/cpuidle/governors/teo.c

+4 −12
Original line number Diff line number Diff line
@@ -123,10 +123,11 @@ static void teo_update(struct cpuidle_driver *drv, struct cpuidle_device *dev) 


	if (cpu_data->time_span_ns >= cpu_data->sleep_length_ns) {

		/*

		 * One of the safety nets has triggered or this was a timer

		 * wakeup (or equivalent).

		 * One of the safety nets has triggered or the wakeup was close

		 * enough to the closest timer event expected at the idle state

		 * selection time to be discarded.

		 */

		measured_us = sleep_length_us;

		measured_us = UINT_MAX;

	} else {

		unsigned int lat;
@@ -188,15 +189,6 @@ static void teo_update(struct cpuidle_driver *drv, struct cpuidle_device *dev) 
		cpu_data->states[idx_timer].hits = hits;

	}



	/*

	 * If the total time span between idle state selection and the "reflect"

	 * callback is greater than or equal to the sleep length determined at

	 * the idle state selection time, the wakeup is likely to be due to a

	 * timer event.

	 */

	if (cpu_data->time_span_ns >= cpu_data->sleep_length_ns)

		measured_us = UINT_MAX;



	/*

	 * Save idle duration values corresponding to non-timer wakeups for

	 * pattern detection.

CRA Git | Maintained and supported by SUSTech CRA and CCSE