x86/cpuidle: switch to uniform meaning of "max_cstate="
authorJan Beulich <jbeulich@suse.com>
Mon, 22 Jul 2019 09:31:38 +0000 (11:31 +0200)
committerJan Beulich <jbeulich@suse.com>
Mon, 22 Jul 2019 09:31:38 +0000 (11:31 +0200)
commit71d571ddd75f445fc56d67e0620807cbce4c276a
tree105da32796b86b7b0eec36a3a9df538a239f3f43
parent93a8877544c3df39124eea6e1f12a319aa3ce54a
x86/cpuidle: switch to uniform meaning of "max_cstate="

While the MWAIT idle driver already takes it to mean an actual C state,
the ACPI idle driver so far used it as a list index. The list index,
however, is an implementation detail of Xen and affected by firmware
settings (i.e. not necessarily uniform for a particular system).

While touching this code also avoid invoking menu_get_trace_data()
when tracing is not active. For consistency do this also for the
MWAIT driver.

Note that I'm intentionally not adding any sorting logic to set_cx():
Before and after this patch we assume entries to arrive in order, so
this would be an orthogonal change.

Take the opportunity and add minimal documentation for the command line
option.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Roger Pau Monné <roger.pau@citrix.com>
Acked-by: Andrew Cooper <andrew.cooper3@citrix.com>
docs/misc/xen-command-line.pandoc
tools/misc/xenpm.c
xen/arch/x86/acpi/cpu_idle.c
xen/arch/x86/cpu/mwait-idle.c
xen/arch/x86/time.c
xen/drivers/acpi/pmstat.c
xen/include/public/sysctl.h