Gerd Hoffmann wrote:
> +cycle_t pvclock_clocksource_read(struct kvm_vcpu_time_info *src)
> +{
> +     struct pvclock_shadow_time *shadow;
> +     cycle_t ret;
> +     unsigned version;
> +
> +     shadow = &get_cpu_var(shadow_time);
> +     do {
> +             version = pvclock_get_time_values(shadow, src);
> +             barrier();
> +             ret = shadow->system_timestamp + 
> pvclock_get_nsec_offset(shadow);
> +             barrier();
>   

Is barrier() strong enough?  Does kvm guarantee that the per-cpu time 
parameters are only ever updated by that cpu?  I'm pretty sure Xen does, 
so that's OK.

    J

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
kvm-devel mailing list
kvm-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/kvm-devel

Reply via email to