| Message ID | c88502e8-3342-3bae-a670-2780c56b54c6@suse.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ACPI/cpuidle: bump maximum number of power states we support | expand |
On 25.09.19 11:12, Jan Beulich wrote: > Commit 4c6cd64519 ("mwait_idle: Skylake Client Support") added a table > with 8 entries, which - together with C0 - rendered the current limit > too low. It should have been accompanied by an increase of the constant; > do this now. Don't bump by too much though, as there are a number of on- > stack arrays which are dimensioned by this constant. > > Signed-off-by: Jan Beulich <jbeulich@suse.com> Release-acked-by: Juergen Gross <jgross@suse.com> Juergen
On Wed, Sep 25, 2019 at 11:12:59AM +0200, Jan Beulich wrote: > Commit 4c6cd64519 ("mwait_idle: Skylake Client Support") added a table > with 8 entries, which - together with C0 - rendered the current limit > too low. It should have been accompanied by an increase of the constant; > do this now. Don't bump by too much though, as there are a number of on- > stack arrays which are dimensioned by this constant. > > Signed-off-by: Jan Beulich <jbeulich@suse.com> Reviewed-by: Wei Liu <wl@xen.org>
--- a/xen/include/xen/cpuidle.h +++ b/xen/include/xen/cpuidle.h @@ -29,7 +29,7 @@ #include <xen/cpumask.h> #include <xen/spinlock.h> -#define ACPI_PROCESSOR_MAX_POWER 8 +#define ACPI_PROCESSOR_MAX_POWER 12 #define CPUIDLE_NAME_LEN 16 #define ACPI_CSTATE_EM_NONE 0
Commit 4c6cd64519 ("mwait_idle: Skylake Client Support") added a table with 8 entries, which - together with C0 - rendered the current limit too low. It should have been accompanied by an increase of the constant; do this now. Don't bump by too much though, as there are a number of on- stack arrays which are dimensioned by this constant. Signed-off-by: Jan Beulich <jbeulich@suse.com>