mbox series

[0/6] cpuidle: Make cpuidle governor switchable to be the default behaviour

Message ID 1589869525-29893-1-git-send-email-guohanjun@huawei.com (mailing list archive)
Headers show
Series cpuidle: Make cpuidle governor switchable to be the default behaviour | expand

Message

Hanjun Guo May 19, 2020, 6:25 a.m. UTC
This is the formal patch set after the RFC v2 [0].

For now cpuidle governor can be switched via sysfs only when the
boot option "cpuidle_sysfs_switch" is passed, but it's useful
to switch the governor to adapt to different workloads, especially
after TEO and haltpoll governors were introduced.

Make cpuidle governor switchable to be the default behaviour by
removing the sysfs_switch and switch attributes, also update the
document as well.

Patch 1/6 and 2/6 are bugfix patch which can be triggered if the
governor name is 15 characters, it is not a 'real' bug for now as we
don't have such usecases, so we can merge them together via this
patchset.

Patch 3/6 and 4/6 are the functional update to make cpuidle governor
switchable to be the default behaviour.

Patch 5/6 and 6/6 are the document update.

Changes since RFC v2:
 - Remove sizeof(char) in patch 1/6;
 - Fix some minor typos;
 - Adding Review/Test/ACK tags.

[0]:https://patchwork.kernel.org/cover/11519511/

Hanjun Guo (6):
  cpuidle: sysfs: Fix the overlap for showing available governors
  cpuidle: sysfs: Accept governor name with 15 characters
  cpuidle: Make cpuidle governor switchable to be the default behaviour
  cpuidle: sysfs: Remove sysfs_switch and switch attributes
  Documentation: cpuidle: update the document
  Documentation: ABI: make current_governer_ro as a candidate for
    removal

 Documentation/ABI/obsolete/sysfs-cpuidle           |  9 ++++
 Documentation/ABI/testing/sysfs-devices-system-cpu | 24 ++++-----
 Documentation/admin-guide/pm/cpuidle.rst           | 20 ++++----
 Documentation/driver-api/pm/cpuidle.rst            |  5 +-
 drivers/cpuidle/sysfs.c                            | 58 ++++++----------------
 5 files changed, 45 insertions(+), 71 deletions(-)
 create mode 100644 Documentation/ABI/obsolete/sysfs-cpuidle

Comments

Hanjun Guo May 19, 2020, 6:53 a.m. UTC | #1
On 2020/5/19 14:25, Hanjun Guo wrote:
> This is the formal patch set after the RFC v2 [0].
> 
> For now cpuidle governor can be switched via sysfs only when the
> boot option "cpuidle_sysfs_switch" is passed, but it's useful
> to switch the governor to adapt to different workloads, especially
> after TEO and haltpoll governors were introduced.
> 
> Make cpuidle governor switchable to be the default behaviour by
> removing the sysfs_switch and switch attributes, also update the
> document as well.
> 
> Patch 1/6 and 2/6 are bugfix patch which can be triggered if the
> governor name is 15 characters, it is not a 'real' bug for now as we
> don't have such usecases, so we can merge them together via this
> patchset.
> 
> Patch 3/6 and 4/6 are the functional update to make cpuidle governor
> switchable to be the default behaviour.
> 
> Patch 5/6 and 6/6 are the document update.
> 
> Changes since RFC v2:
>   - Remove sizeof(char) in patch 1/6;
>   - Fix some minor typos;
>   - Adding Review/Test/ACK tags.

Forgot to mention that this patch set is rebased on linux-next
branch of linux-pm.git

Thanks
Hanjun