[v3,07/19] KVM: arm64: Rename the KVM_REQ_SLEEP handler

Message ID	20220223041844.3984439-8-oupton@google.com
State	Not Applicable
Headers	show Return-Path: <kvm-riscv-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org> Date: Wed, 23 Feb 2022 04:18:32 +0000 In-Reply-To: <20220223041844.3984439-1-oupton@google.com> Message-Id: <20220223041844.3984439-8-oupton@google.com> Mime-Version: 1.0 References: <20220223041844.3984439-1-oupton@google.com> Subject: [PATCH v3 07/19] KVM: arm64: Rename the KVM_REQ_SLEEP handler From: Oliver Upton <oupton@google.com> To: kvmarm@lists.cs.columbia.edu Cc: Paolo Bonzini <pbonzini@redhat.com>, Marc Zyngier <maz@kernel.org>, James Morse <james.morse@arm.com>, Alexandru Elisei <alexandru.elisei@arm.com>, Suzuki K Poulose <suzuki.poulose@arm.com>, Anup Patel <anup@brainfault.org>, Atish Patra <atishp@atishpatra.org>, Sean Christopherson <seanjc@google.com>, Vitaly Kuznetsov <vkuznets@redhat.com>, Wanpeng Li <wanpengli@tencent.com>, Jim Mattson <jmattson@google.com>, Joerg Roedel <joro@8bytes.org>, kvm@vger.kernel.org, kvm-riscv@lists.infradead.org, Peter Shier <pshier@google.com>, Reiji Watanabe <reijiw@google.com>, Ricardo Koller <ricarkol@google.com>, Raghavendra Rao Ananta <rananta@google.com>, Jing Zhang <jingzhangos@google.com>, Oliver Upton <oupton@google.com>, Andrew Jones <drjones@redhat.com> preview: The naming of the kvm_req_sleep function is confusing: the function itself sleeps the vCPU, it does not request such an event. Rename the function to make its purpose more clear. No functional change intended. Content analysis details: (-7.7 points, 5.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at https://www.dnswl.org/, no trust [2607:f8b0:4864:20:0:0:0:14a listed in] [list.dnswl.org] -0.0 SPF_PASS SPF: sender matches SPF record 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -7.5 USER_IN_DEF_DKIM_WL From: address is in the default DKIM white-list -0.1 DKIM_VALID_AU Message has a valid DKIM or DK signature from author's domain -0.1 DKIM_VALID Message has at least one valid DKIM or DK signature 0.1 DKIM_SIGNED Message has a DKIM or DK signature, not necessarily valid -0.1 DKIM_VALID_EF Message has a valid DKIM or DK signature from envelope-from domain -0.0 DKIMWL_WL_MED DKIMwl.org - Medium trust sender Precedence: list Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "kvm-riscv" <kvm-riscv-bounces@lists.infradead.org> Errors-To: kvm-riscv-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org
Series	KVM: arm64: Implement PSCI SYSTEM_SUSPEND \| expand [v3,00/19] KVM: arm64: Implement PSCI SYSTEM_SUSPEND [v3,01/19] KVM: arm64: Drop unused param from kvm_psci_version() [v3,02/19] KVM: arm64: Create a helper to check if IPA is valid [v3,03/19] KVM: arm64: Reject invalid addresses for CPU_ON PSCI call [v3,04/19] KVM: arm64: Clean up SMC64 PSCI filtering for AArch32 guests [v3,05/19] KVM: arm64: Dedupe vCPU power off helpers [v3,06/19] KVM: arm64: Track vCPU power state using MP state values [v3,07/19] KVM: arm64: Rename the KVM_REQ_SLEEP handler [v3,08/19] KVM: arm64: Add reset helper that accepts caller-provided reset state [v3,09/19] KVM: arm64: Implement PSCI SYSTEM_SUSPEND [v3,10/19] KVM: Create helper for setting a system event exit [v3,11/19] KVM: arm64: Return a value from check_vcpu_requests() [v3,12/19] KVM: arm64: Add support for userspace to suspend a vCPU [v3,13/19] KVM: arm64: Add support KVM_SYSTEM_EVENT_SUSPEND to PSCI SYSTEM_SUSPEND [v3,14/19] KVM: arm64: Raise default PSCI version to v1.1 [v3,15/19] selftests: KVM: Rename psci_cpu_on_test to psci_test [v3,16/19] selftests: KVM: Create helper for making SMCCC calls [v3,17/19] selftests: KVM: Use KVM_SET_MP_STATE to power off vCPU in psci_test [v3,18/19] selftests: KVM: Refactor psci_test to make it amenable to new tests [v3,19/19] selftests: KVM: Test SYSTEM_SUSPEND PSCI call

Message ID

20220223041844.3984439-8-oupton@google.com

State

Not Applicable

Headers

Date: Wed, 23 Feb 2022 04:18:32 +0000
In-Reply-To: <20220223041844.3984439-1-oupton@google.com>
Message-Id: <20220223041844.3984439-8-oupton@google.com>
Mime-Version: 1.0
References: <20220223041844.3984439-1-oupton@google.com>
Subject: [PATCH v3 07/19] KVM: arm64: Rename the KVM_REQ_SLEEP handler
From: Oliver Upton <oupton@google.com>
To: kvmarm@lists.cs.columbia.edu
Cc: Paolo Bonzini <pbonzini@redhat.com>, Marc Zyngier <maz@kernel.org>,
 James Morse <james.morse@arm.com>,
 Alexandru Elisei <alexandru.elisei@arm.com>,
 Suzuki K Poulose <suzuki.poulose@arm.com>, Anup Patel <anup@brainfault.org>,
 Atish Patra <atishp@atishpatra.org>, Sean Christopherson <seanjc@google.com>,
 Vitaly Kuznetsov <vkuznets@redhat.com>, Wanpeng Li <wanpengli@tencent.com>,
 Jim Mattson <jmattson@google.com>, Joerg Roedel <joro@8bytes.org>,
 kvm@vger.kernel.org,
 kvm-riscv@lists.infradead.org, Peter Shier <pshier@google.com>,
 Reiji Watanabe <reijiw@google.com>, Ricardo Koller <ricarkol@google.com>,
 Raghavendra Rao Ananta <rananta@google.com>,
 Jing Zhang <jingzhangos@google.com>,
 Oliver Upton <oupton@google.com>, Andrew Jones <drjones@redhat.com>
Precedence: list
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Sender: "kvm-riscv" <kvm-riscv-bounces@lists.infradead.org>
Errors-To: kvm-riscv-bounces+incoming=patchwork.ozlabs.org@lists.infradead.org

Series

KVM: arm64: Implement PSCI SYSTEM_SUSPEND | expand

Commit Message

Oliver Upton Feb. 23, 2022, 4:18 a.m. UTC

The naming of the kvm_req_sleep function is confusing: the function
itself sleeps the vCPU, it does not request such an event. Rename the
function to make its purpose more clear.

No functional change intended.

Signed-off-by: Oliver Upton <oupton@google.com>
Reviewed-by: Andrew Jones <drjones@redhat.com>
---
 arch/arm64/kvm/arm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm64/kvm/arm.c b/arch/arm64/kvm/arm.c
index b4987b891f38..6af680675810 100644
--- a/arch/arm64/kvm/arm.c
+++ b/arch/arm64/kvm/arm.c
@@ -665,7 +665,7 @@  void kvm_arm_resume_guest(struct kvm *kvm)
 	}
 }
 
-static void vcpu_req_sleep(struct kvm_vcpu *vcpu)
+static void kvm_vcpu_sleep(struct kvm_vcpu *vcpu)
 {
 	struct rcuwait *wait = kvm_arch_vcpu_get_wait(vcpu);
 
@@ -723,7 +723,7 @@  static void check_vcpu_requests(struct kvm_vcpu *vcpu)
 {
 	if (kvm_request_pending(vcpu)) {
 		if (kvm_check_request(KVM_REQ_SLEEP, vcpu))
-			vcpu_req_sleep(vcpu);
+			kvm_vcpu_sleep(vcpu);
 
 		if (kvm_check_request(KVM_REQ_VCPU_RESET, vcpu))
 			kvm_reset_vcpu(vcpu);

[v3,07/19] KVM: arm64: Rename the KVM_REQ_SLEEP handler

Commit Message

Patch