lkml.org 
[lkml]   [2022]   [Aug]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v2 08/19] KVM: x86: Move TSC fixup logic to KVM arch resume callback
    Date
    From: Isaku Yamahata <isaku.yamahata@intel.com>

    commit 0dd6a6edb012 ("KVM: Dont mark TSC unstable due to S4 suspend") made
    use of kvm_arch_hardware_enable() callback to detect that TSC goes backward
    due to S4 suspend. It has to check it only when resuming from S4. Not
    every time virtualization hardware ennoblement. Move the logic to
    kvm_arch_resume() callback.

    Suggested-by: Sean Christopherson <seanjc@google.com>
    Signed-off-by: Isaku Yamahata <isaku.yamahata@intel.com>
    ---
    arch/x86/kvm/x86.c | 27 ++++++++++++++++-----------
    1 file changed, 16 insertions(+), 11 deletions(-)

    diff --git a/arch/x86/kvm/x86.c b/arch/x86/kvm/x86.c
    index ca920b6b925d..0b112cd7de58 100644
    --- a/arch/x86/kvm/x86.c
    +++ b/arch/x86/kvm/x86.c
    @@ -11831,18 +11831,30 @@ void kvm_vcpu_deliver_sipi_vector(struct kvm_vcpu *vcpu, u8 vector)
    EXPORT_SYMBOL_GPL(kvm_vcpu_deliver_sipi_vector);

    int kvm_arch_hardware_enable(void)
    +{
    + return static_call(kvm_x86_hardware_enable)();
    +}
    +
    +void kvm_arch_hardware_disable(void)
    +{
    + static_call(kvm_x86_hardware_disable)();
    + drop_user_return_notifiers();
    +}
    +
    +void kvm_arch_resume(int usage_count)
    {
    struct kvm *kvm;
    struct kvm_vcpu *vcpu;
    unsigned long i;
    - int ret;
    u64 local_tsc;
    u64 max_tsc = 0;
    bool stable, backwards_tsc = false;

    - ret = static_call(kvm_x86_hardware_enable)();
    - if (ret != 0)
    - return ret;
    + if (!usage_count)
    + return;
    +
    + if (kvm_arch_hardware_enable())
    + return;

    local_tsc = rdtsc();
    stable = !kvm_check_tsc_unstable();
    @@ -11917,13 +11929,6 @@ int kvm_arch_hardware_enable(void)
    }

    }
    - return 0;
    -}
    -
    -void kvm_arch_hardware_disable(void)
    -{
    - static_call(kvm_x86_hardware_disable)();
    - drop_user_return_notifiers();
    }

    static inline void kvm_ops_update(struct kvm_x86_init_ops *ops)
    --
    2.25.1
    \
     
     \ /
      Last update: 2022-08-30 14:03    [W:7.976 / U:0.300 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site