Received: by 2002:a05:6a10:6d10:0:0:0:0 with SMTP id gq16csp2645172pxb; Mon, 18 Apr 2022 05:28:19 -0700 (PDT) X-Google-Smtp-Source: ABdhPJy1uA5R8ZnII/kRY4GqZNixmfAbAnz6jPTqv2AXB05KT4edEF+cmLQmMDl+aGvnaurw4s/Y X-Received: by 2002:a17:907:62a6:b0:6ef:8118:d3e2 with SMTP id nd38-20020a17090762a600b006ef8118d3e2mr7453777ejc.605.1650284899164; Mon, 18 Apr 2022 05:28:19 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1650284899; cv=none; d=google.com; s=arc-20160816; b=zZJ6t/t90hMQof/l374pNyZuBImUa4MSL2z3U5NYry01m6brxlt9E6Y/5/OyH+oN4h AjyF7mdWgwKo5x1e9rHd+eY8PkfniV4SgZL8EOyV6rP8euA87sHd/GVt3D439v70dhjK YqpyPLDOQr3dF3zt1C+nRj3T5bohWbk7PnUmHAogHZscWEkjspOyybeVq8LpYM8rLJSL tTjQmRrZ+Nf4hyETOaFt4SLsYgIkXuKYfSXvyt3GHIeWtE1MCZeJJGMJ8rj9zNh7E+jO VBcFPiqwYf4NViTdX87Cpi79CWJsoGrlgBdwFPr9ukWLQrCx2NNn9Od9+oX5XIcaN8lY PE7g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from :dkim-signature; bh=Bz3WXy9wp2iTizc3RUbJP1TFtfPRjIT22rWiujaCDg4=; b=dgisZzD7qk9XaweTa+4IHLaA7pQ0ddAoQifLITj7tNWVRn1k516+L9icw2bw5i+NQr Go0VSYiu/dG2qc4JytaXIhKyMleDf2LMKCidVCVdZGJ42JWpaOeZkEIPoQD+nIeCv9xb cuE0bCNVx8NLg4PjU2m83pizrjxcjkeZeIYIx0z68Bslf2QDeWT1qZawtpu5sQiN0yw/ kSTmHwspC8Z/Yf39UDKIeKmWuMTbyZ6NFY0Sl5oZ+u7g2l/nnS+pbFDVFVAWm8K3Msa1 XGmdOdKmlYAiex4t3gDzHCHO3CwnYv+XTHu0IAG9CGaOdhCucC0pTnTSTKevNkcCZb6G 1LOw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=ewftY+yE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id by9-20020a170906a2c900b006dfd28d349csi6107430ejb.119.2022.04.18.05.27.55; Mon, 18 Apr 2022 05:28:19 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20210112 header.b=ewftY+yE; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236988AbiDRHqF (ORCPT + 99 others); Mon, 18 Apr 2022 03:46:05 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:48006 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231948AbiDRHqC (ORCPT ); Mon, 18 Apr 2022 03:46:02 -0400 Received: from mail-pg1-x52e.google.com (mail-pg1-x52e.google.com [IPv6:2607:f8b0:4864:20::52e]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id AEE7012A8E; Mon, 18 Apr 2022 00:43:24 -0700 (PDT) Received: by mail-pg1-x52e.google.com with SMTP id q12so17441509pgj.13; Mon, 18 Apr 2022 00:43:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id; bh=Bz3WXy9wp2iTizc3RUbJP1TFtfPRjIT22rWiujaCDg4=; b=ewftY+yE8/uWZN1l3zx9HBD0GfZdbWrHPtcpH/DYz0i8BoRvxIk6lmvn5jbrtvd7NJ 2hHLmuGLuIh65EQy57k6QuJFZZcJW5WkJRLfxo6GTYy2bMAeVgIgv5iKykmlQ8RTH/U8 ynMsbHDuPygm/vPn/gP0pVcvPRU+D2R9+VK7NqPlbGYtYUu8VQPtyIBXkcMNsJHMoSe+ 74auquXJEripwV8sj1crTTYgkFcUYve7eAe1Md7X2jDFTWMCXYUhvn316mFQVvE+OgZB x2C+JjVRuPszvq30lCud6XRqpBcWrTQxDbhxRnceBc4QLkYjDStUT+fSfOaGbZvaJf9E hBxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=Bz3WXy9wp2iTizc3RUbJP1TFtfPRjIT22rWiujaCDg4=; b=HNEtJY4RjIUh27DsdvdcpT9aWvG1wyDydVUDVOSLbHBnxJs6GIlYx5OiE0wIOH1+kn 1nJz19mtZG4xmnj5oVWvHsZ3kZKKB5dGrKnDBNWiTKDGyJIz9cXHUpMkPt1wjUTWJjI2 E/Sl6qsMneerFHo9q9q+wDLZi7hjEWLMJdWWmUsSRGFNJkwD78NygLuL6q2gJAixliAI KgDcAw8SMPPVHvSG2/r7pxZgYHK4Dn4j8OfIF6+ZDLG2blXL2klm8MFCdSZ+C+ObMpNq at4uk2vp94QrQdQDoifwgn9k+oi5UNjal7C1S6gx/pZoqmH4oVBYwchpFwZ5vOCoJRyi 24Fw== X-Gm-Message-State: AOAM533b5qMD9d5EWTbx+gWLMoSYDzxPbMuLgvoqc9AYVCu79xA0T3LL ygmsMlzszxBNIsHc66keTQ6+0Ug6idA= X-Received: by 2002:a65:4b85:0:b0:399:8cd:5270 with SMTP id t5-20020a654b85000000b0039908cd5270mr8910175pgq.418.1650267804116; Mon, 18 Apr 2022 00:43:24 -0700 (PDT) Received: from localhost.localdomain ([203.205.141.114]) by smtp.googlemail.com with ESMTPSA id br10-20020a17090b0f0a00b001cd4fb89d38sm15595833pjb.9.2022.04.18.00.43.21 (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128); Mon, 18 Apr 2022 00:43:23 -0700 (PDT) From: Wanpeng Li X-Google-Original-From: Wanpeng Li To: linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: Paolo Bonzini , Sean Christopherson , Vitaly Kuznetsov , Wanpeng Li , Jim Mattson , Joerg Roedel , Marcelo Tosatti Subject: [PATCH] x86/kvm: Fix guest haltpoll after the guest suspend/resume Date: Mon, 18 Apr 2022 00:42:32 -0700 Message-Id: <1650267752-46796-1-git-send-email-wanpengli@tencent.com> X-Mailer: git-send-email 2.7.4 X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FREEMAIL_FROM, RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Wanpeng Li All AP cpus hot-unplugged during guest suspend and hot-plugged during guest resume, qemu makes host haltpoll as default during vCPU reset, it resets all the haltpoll to host haltpoll by KVM_SET_MSRS ioctl. All AP cpus switch to guest haltpoll again by haltpoll driver hotplug callbacks in the guest. However, BSP is not hotpluggable and keeps host haltpoll which means that the BSP switches from guest haltpoll to host haltpoll during the guest suspend and resume, this will have vmexit cost each time the guest enters idle. This patch fixes it by recording BSP's haltpoll state and resuming it during guest resume. Cc: Marcelo Tosatti Signed-off-by: Wanpeng Li --- arch/x86/kernel/kvm.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/arch/x86/kernel/kvm.c b/arch/x86/kernel/kvm.c index d0bb2b3fb305..5c65643cb005 100644 --- a/arch/x86/kernel/kvm.c +++ b/arch/x86/kernel/kvm.c @@ -69,6 +69,7 @@ static DEFINE_PER_CPU_DECRYPTED(struct kvm_vcpu_pv_apf_data, apf_reason) __align DEFINE_PER_CPU_DECRYPTED(struct kvm_steal_time, steal_time) __aligned(64) __visible; static int has_steal_clock = 0; +static int has_guest_poll = 0; /* * No need for any "IO delay" on KVM */ @@ -706,14 +707,26 @@ static int kvm_cpu_down_prepare(unsigned int cpu) static int kvm_suspend(void) { + u64 val = 0; + kvm_guest_cpu_offline(false); +#ifdef CONFIG_ARCH_CPUIDLE_HALTPOLL + if (kvm_para_has_feature(KVM_FEATURE_POLL_CONTROL)) + rdmsrl(MSR_KVM_POLL_CONTROL, val); + has_guest_poll = !(val & 1); +#endif return 0; } static void kvm_resume(void) { kvm_cpu_online(raw_smp_processor_id()); + +#ifdef CONFIG_ARCH_CPUIDLE_HALTPOLL + if (kvm_para_has_feature(KVM_FEATURE_POLL_CONTROL) && has_guest_poll) + wrmsrl(MSR_KVM_POLL_CONTROL, 0); +#endif } static struct syscore_ops kvm_syscore_ops = { -- 2.25.1