Re: [Xen-devel] [PATCH v3 12/16] x86/VPMU: Handle PMU interrupts for PV guests

[Boris Ostrovsky <boris.ostrovsky@xxxxxxxxxx>]

On 01/16/2014 04:15 AM, Jan Beulich wrote:
> > @@ -82,7 +87,23 @@ int vpmu_do_wrmsr(unsigned int msr, uint64_t msr_content)
> > ...
> > +        {
> > +            vpmu_set(vpmu, VPMU_CONTEXT_SAVE);
> > +            (void)vpmu->arch_vpmu_ops->arch_vpmu_save(current);
> What's this cast good for?
These routines return an int that indicates whether this was full 
context save or just that the counters have been stopped. Depending on 
where the routines are called from we may ignore the return value 
(having VPMU_CONTEXT_SAVE forces "full save").
I did it so that compiler won't complain about ignoring return value (I 
don't think it does now but I imagine some version still may).
> > +            vpmu_reset(vpmu, VPMU_CONTEXT_SAVE | VPMU_CONTEXT_LOADED);
> > +        }
> > +        return val;
> > +    }
> >       return 0;
> >   }
> >   
...
> > +    /* dom0 will handle this interrupt */
> > +    if ( v->domain->domain_id >= DOMID_FIRST_RESERVED )
> > +    {
> > +        if ( smp_processor_id() >= dom0->max_vcpus )
> > +            return 0;
> > +        v = dom0->vcpu[smp_processor_id()];
> > +    }
> Ugly new uses of "dom0". And the correlation between
> smp_processor_id() and dom0->max_vcpus doesn't look sane
> either.
Yes, this needs to be fixed. Maybe

    v = dom0->vcpu[smp_processor_id() % dom0->max_vcpus];

And drop the check. (And the same change in pmu_softnmi() in a later patch).

Not sure what to do about dom0. What else can I use instead? I need to 
choose dom0's vcpu.

Thanks.
-boris

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