>--- a/arch/i386/xen/time.c
>+++ b/arch/i386/xen/time.c
>@@ -105,17 +105,15 @@ static void get_runstate_snapshot(struct
>       preempt_enable();
> }
> 
>-static void setup_runstate_info(void)
>+static void setup_runstate_info(int cpu)
> {
>       struct vcpu_register_runstate_memory_area area;
> 
>-      area.addr.v = &__get_cpu_var(runstate);
>+      area.addr.v = &per_cpu(runstate, cpu);
> 
>       if (HYPERVISOR_vcpu_op(VCPUOP_register_runstate_memory_area,
>                              smp_processor_id(), &area))

Shouldn't this be 'cpu' rather than 'smp_processor_id()'?

>               BUG();
>-
>-      get_runstate_snapshot(&__get_cpu_var(runstate_snapshot));
> }
> 
> static void do_stolen_accounting(void)

Jan


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

Reply via email to