On 03/30/2016 04:49 PM, Ian Jackson wrote: > Joao Martins writes ("[PATCH v2 1/6] public/xen.h: add flags field to > vcpu_time_info"): >> This field has two possible flags (as of latest pvclock ABI >> shared with KVM). >> >> flags: bits in this field indicate extended capabilities >> coordinated between the guest and the hypervisor. Specifically >> on KVM, availability of specific flags has to be checked in >> 0x40000001 cpuid leaf. On Xen, we don't have that but we can >> still check some of the flags after registering the time info >> page since a force_update_vcpu_system_time is performed. > > Earlier in the thread there was a Reviewed-by from Andrew Cooper.
> Shouldn't that have been preserved here ? I dropped both of the Reviewed-by because I updated the patch with changes that weren't noted in earlier reviews - I had in mind that is the protocol. > Also, Andrew made some points about the linux.git MAINTAINERS file. Yes, the MAINTAINERS file will be updated with the linux v2 series and the point is to have xen-devel be notified whenever changes are made to the pvclock ABI (that is shared with KVM). Joao > Ideally I would like to commit this patch ASAP. > > Thanks, > Ian. > _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel