On 05.03.2025 10:11, Mykola Kvach wrote:
> --- a/xen/common/percpu.c
> +++ b/xen/common/percpu.c
> @@ -74,7 +74,8 @@ static int cf_check cpu_percpu_callback(
> switch ( action )
> {
> case CPU_UP_PREPARE:
> - rc = init_percpu_area(cpu);
> + if ( system_state != SYS_STATE_resume )
> + rc = init_percpu_area(cpu);
> break;
>
> case CPU_UP_CANCELED:
Right now I can't see why we wouldn't need such an adjustment also for S3 on
AMD x86 hardware. However, please let's not further split how things are
being checked for. I.e. can we please keep the park_offline_cpus and the
system_state checks together, either here or in init_percpu_area()? Just
like CPU_DEAD etc handling has it.
Jan
