On 16.07.2019 16:48, Roger Pau Monné wrote:
> On Wed, Jul 03, 2019 at 01:03:02PM +0000, Jan Beulich wrote:
>> @@ -592,7 +608,13 @@ static void acpi_processor_idle(void)
>>
>> do {
>> cx = &power->states[next_state];
>> - } while ( cx->type > max_state && --next_state );
>> + } while ( (cx->type > max_state ||
>> + cx->entry_method == ACPI_CSTATE_EM_NONE ||
>> + (cx->entry_method == ACPI_CSTATE_EM_FFH &&
>> + cx->type == max_cstate &&
>> + (cx->address & MWAIT_SUBSTATE_MASK) > max_csubstate)) &&
>> + --next_state );
>> + cx = &power->states[next_state];
>
> Is the line above a stray addition? It is at least not properly
> aligned AFAICT.
Oh, yes, that's a re-basing mistake. Thanks for spotting.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
