>>> On 21.03.16 at 13:24, <jgr...@suse.com> wrote:
> @@ -758,9 +759,14 @@ struct smp_sync_call_struct {
>  static void smp_call_sync_callback(struct work_struct *work)
>  {
>       struct smp_sync_call_struct *sscs;
> +     unsigned int cpu = smp_processor_id();

So this obtains the vCPU number, yet ...

>       sscs = container_of(work, struct smp_sync_call_struct, work);
> +     preempt_disable();
> +     hypervisor_pin_vcpu(cpu);

... here you're supposed to pass a pCPU number.

Also don't you need to call smp_processor_id() after preempt_disable()?

Jan

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org
https://lists.linuxfoundation.org/mailman/listinfo/virtualization

Reply via email to