On 1/8/2025 1:07 AM, Borislav Petkov wrote:
> On Mon, Jan 06, 2025 at 06:16:32PM +0530, Nikunj A Dadhania wrote:
>> Although the kernel switches over to stable TSC clocksource instead of
>> PV clocksource, the scheduler still keeps on using PV clocks as the
>> sched clock source. This is because KVM, Xen and VMWare, switch the
>> paravirt sched clock handler in their init routines. HyperV is the
>> only PV clock source that checks if the platform provides an invariant
>> TSC and does not switch to PV sched clock.
>
> So this below looks like some desperate hackery. Are you doing this because
> kvm_sched_clock_init() does
>
> paravirt_set_sched_clock(kvm_sched_clock_read);
>
> ?
All guests(including STSC) running on KVM or other hypervisors like Xen/VMWare
uses the regular TSC when the guest is booted with TscInvariant bit set. But it
doesn't switch the sched clock to use TSC which it should have, pointed here[1]
by Sean:
No, I'm saying that the guest should prefer the raw TSC over kvmclock if the
TSC is stable, irrespective of SNP or TDX. This is effectively already done
for the timekeeping base (see commit 7539b174aef4 ("x86: kvmguest: use TSC
clocksource if invariant TSC is exposed")), but the scheduler still uses
kvmclock thanks to the kvm_sched_clock_init() code.
> If so, why don't you simply prevent that on STSC guests?
This is for all guests running on different hypervisor with stable TSC. And I
did try to limit this to KVM here [2], here is Sean's comment to that
implementation:
> Although the kernel switches over to stable TSC clocksource instead of
> kvmclock, the scheduler still keeps on using kvmclock as the sched clock.
> This is due to kvm_sched_clock_init() updating the pv_sched_clock()
> unconditionally.
All PV clocks are affected by this, no? This seems like something that
should be handled in common code, which is the point I was trying to make in
v11.
Regards
Nikunj
1. https://lore.kernel.org/kvm/ZurCbP7MesWXQbqZ@xxxxxxxxxx/
2. https://lore.kernel.org/kvm/20241009092850.197575-17-nikunj@xxxxxxx/
