From: Marcelo Tosatti <mtosatti@redhat.com>
Date: Fri, 6 Dec 2019 15:07:41 +0000 (-0200)
Subject: cpuidle: use first valid target residency as poll time
X-Git-Tag: v5.5-rc2~20^2^2~2
X-Git-Url: https://www.infradead.org/git/?a=commitdiff_plain;h=36fcb4292473cb9c9ce7706d038bcf0eda5cabeb;p=users%2Fwilly%2Flinux.git

cpuidle: use first valid target residency as poll time

Commit 259231a04561 ("cpuidle: add poll_limit_ns to cpuidle_device
structure") changed, by mistake, the target residency from the first
available sleep state to the last available sleep state (which should
be longer).

This might cause excessive polling.

Fixes: 259231a04561 ("cpuidle: add poll_limit_ns to cpuidle_device structure")
Signed-off-by: Marcelo Tosatti <mtosatti@redhat.com>
Cc: 5.4+ <stable@vger.kernel.org> # 5.4+
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
---

diff --git a/drivers/cpuidle/cpuidle.c b/drivers/cpuidle/cpuidle.c
index 0005be5ea2b4..15877b431143 100644
--- a/drivers/cpuidle/cpuidle.c
+++ b/drivers/cpuidle/cpuidle.c
@@ -382,6 +382,7 @@ u64 cpuidle_poll_time(struct cpuidle_driver *drv,
 			continue;
 
 		limit_ns = (u64)drv->states[i].target_residency_ns;
+		break;
 	}
 
 	dev->poll_limit_ns = limit_ns;