cpuidle: Consolidate disabled state checks
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Mon, 4 Nov 2019 11:16:17 +0000 (12:16 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Wed, 6 Nov 2019 12:19:56 +0000 (13:19 +0100)
commit99e98d3fb1008ef7416e16a1fd355cb73a253502
treed61215ef2626e545ccbbeabf8f9ac1c0ea4af2ed
parentfa583f71a99c85e52781ed877c82c8757437b680
cpuidle: Consolidate disabled state checks

There are two reasons why CPU idle states may be disabled: either
because the driver has disabled them or because they have been
disabled by user space via sysfs.

In the former case, the state's "disabled" flag is set once during
the initialization of the driver and it is never cleared later (it
is read-only effectively).  In the latter case, the "disable" field
of the given state's cpuidle_state_usage struct is set and it may be
changed via sysfs.  Thus checking whether or not an idle state has
been disabled involves reading these two flags every time.

In order to avoid the additional check of the state's "disabled" flag
(which is effectively read-only anyway), use the value of it at the
init time to set a (new) flag in the "disable" field of that state's
cpuidle_state_usage structure and use the sysfs interface to
manipulate another (new) flag in it.  This way the state is disabled
whenever the "disable" field of its cpuidle_state_usage structure is
nonzero, whatever the reason, and it is the only place to look into
to check whether or not the state has been disabled.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
Acked-by: Peter Zijlstra (Intel) <peterz@infradead.org>
drivers/cpuidle/cpuidle-powernv.c
drivers/cpuidle/cpuidle.c
drivers/cpuidle/governors/ladder.c
drivers/cpuidle/governors/menu.c
drivers/cpuidle/governors/teo.c
drivers/cpuidle/sysfs.c
include/linux/cpuidle.h