Re: [Xen-devel] [PATCH v16 12/23] x86/VPMU: Replace vcpu with vpmu as argument to some routines

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 17.12.14 at 16:38, <boris.ostrovsky@xxxxxxxxxx> wrote:
> @@ -146,36 +145,34 @@ static void vpmu_save_force(void *arg)
>      vpmu_set(vpmu, VPMU_CONTEXT_SAVE);
>  
>      if ( vpmu->arch_vpmu_ops )
> -        (void)vpmu->arch_vpmu_ops->arch_vpmu_save(v);
> +        (void)vpmu->arch_vpmu_ops->arch_vpmu_save(vpmu);

Pointless cast.

> @@ -193,7 +190,7 @@ void vpmu_load(struct vcpu *v)
>          if ( vpmu_is_set(vpmu, VPMU_CONTEXT_LOADED) )
>          {
>              on_selected_cpus(cpumask_of(vpmu->last_pcpu),
> -                             vpmu_save_force, (void *)v, 1);
> +                             vpmu_save_force, (void *)vpmu, 1);

Again.

Apart from that
Acked-by: Jan Beulich <jbeulich@xxxxxxxx>


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel