On 23.07.2019 11:20, Juergen Gross wrote:
> --- a/xen/common/schedule.c
> +++ b/xen/common/schedule.c
> @@ -708,6 +708,8 @@ void restore_vcpu_affinity(struct domain *d)
>            * set v->processor of each of their vCPUs to something that will
>            * make sense for the scheduler of the cpupool in which they are in.
>            */
> +        lock = vcpu_schedule_lock_irq(v);
> +
>           cpumask_and(cpumask_scratch_cpu(cpu), v->cpu_hard_affinity,
>                       cpupool_domain_cpumask(d));
>           if ( cpumask_empty(cpumask_scratch_cpu(cpu)) )
> @@ -731,6 +733,9 @@ void restore_vcpu_affinity(struct domain *d)
>   
>           v->processor = cpumask_any(cpumask_scratch_cpu(cpu));
>   
> +        spin_unlock_irq(lock);
> +
> +     /* v->processor might have changed, so reacquire the lock. */

Whoever commits this (perhaps me, later today) will want to replace
the hard tab here.

Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel

Reply via email to