On 01/18/2016 08:39 AM, 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?
Yes, that's fine, my tag still stands.
-boris
* 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.
Jan
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel
