(Sorry for the delay, I had to spend some days sick at home :-( ) On Fri, Feb 11, 2011 at 05:12:29PM -0500, Zachary Amsden wrote: > On 02/09/2011 12:29 PM, Joerg Roedel wrote: > So I've gone over this series and the only issue I see so far is with > this patch, and it doesn't have to do with upstream code, rather with > code I was about to send. > > Logically, the compensation done by adjust_tsc_offset should also be > scaled; currently, this happens only for reasons, both of which are > meant to deal with unstable TSCs; since TSC scaling won't happen on > those processors with unstable TSCs, we don't need to worry about it there. The tsc_offset is applied after the TSC is scaled so there is no good way to scale the offset with the TSC value itself. What we can do is to use guest-tsc values only when we calculate an adjustment. So any tsc-offset adjustment made with adjust_tsc_offset() needs to be a function of guest-tsc values. One call-place of the function already does this and the other one can be converted easily. I'll do that in the next version of this patch-set. >From what I understand of your upcoming patch the accumulation of tsc-offsets could also be calculated from guest-tsc values instead of native_read_tsc() values, no? Regards, Joerg -- AMD Operating System Research Center Advanced Micro Devices GmbH Einsteinring 24 85609 Dornach General Managers: Alberto Bozzo, Andrew Bowd Registration: Dornach, Landkr. Muenchen; Registerger. Muenchen, HRB Nr. 43632 -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
