>>> On 17.12.14 at 16:38, <boris.ostrov...@oracle.com> 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 <jbeul...@suse.com> _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel