xen/acpi: upload _PSD info for non Dom0 CPUs too
authorJoao Martins <joao.m.martins@oracle.com>
Thu, 15 Mar 2018 14:22:05 +0000 (14:22 +0000)
committerBoris Ostrovsky <boris.ostrovsky@oracle.com>
Wed, 21 Mar 2018 12:29:13 +0000 (08:29 -0400)
commit4d0f1ce6955913c490263359eadd392574cf9fe3
tree4fc49d915e85b9c66764232442d93eba4aa7f4ec
parent36104cb9012a82e73c32a3b709257766b16bcd1d
xen/acpi: upload _PSD info for non Dom0 CPUs too

All uploaded PM data from non-dom0 CPUs takes the info from vCPU 0 and
changing only the acpi_id. For processors which P-state coordination type
is HW_ALL (0xFD) it is OK to upload bogus P-state dependency information
(_PSD), because Xen will ignore any cpufreq domains created for past CPUs.

Albeit for platforms which expose coordination types as SW_ANY or SW_ALL,
this will have some unintended side effects. Effectively, it will look at
the P-state domain existence and *if it already exists* it will skip the
acpi-cpufreq initialization and thus inherit the policy from the first CPU
in the cpufreq domain. This will finally lead to the original cpu not
changing target freq to P0 other than the first in the domain. Which will
make turbo boost not getting enabled (e.g. for 'performance' governor) for
all cpus.

This patch fixes that, by also evaluating _PSD when we enumerate all ACPI
processors and thus always uploading the correct info to Xen. We export
acpi_processor_get_psd() for that this purpose, but change signature
to not assume an existent of acpi_processor given that ACPI isn't creating
an acpi_processor for non-dom0 CPUs.

Signed-off-by: Joao Martins <joao.m.martins@oracle.com>
Reviewed-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
Acked-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Boris Ostrovsky <boris.ostrovsky@oracle.com>
drivers/acpi/processor_perflib.c
drivers/xen/xen-acpi-processor.c
include/acpi/processor.h