On 01/18/16 06:39, Jan Beulich wrote: > >>> On 17.01.16 at 22:58, <haozhong.zh...@intel.com> wrote: > > The existing hvm_[set|get]_guest_tsc_fixed() calculate the guest TSC by > > adding the TSC offset to the host TSC. When the TSC scaling is enabled, > > the host TSC should be scaled first. This patch adds the scaling logic > > to those two functions. > > > > Reviewed-by: Boris Ostrovsky <boris.ostrov...@oracle.com> > > Signed-off-by: Haozhong Zhang <haozhong.zh...@intel.com> > > --- > > Changes in v4: > > (addressing Jan Beulich's comments) > > * Remove redundant check of cpu_has_tsc_ratio in svm_scale_tsc(). > > This would better have caused Boris' R-b tag to get dropped, or > at least I don't recall him having offered it to stay despite the > change. Boris - can you confirm one way or the other? > > > * Move check of d->arch.vtsc in svm_scale_tsc() to callers. > > No d->arch.vtsc instances appear anywhere in this patch, so the > wording is confusing. Indeed the two callers already check this (or > really sit on suitable "else" paths), so at least the patch is correct. >
Sorry for the confusion. It should be "Remove check of d->arch.vtsc in svm_scale_tsc() as it has been checked by callers." Haozhong _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel