From nobody Mon Apr 6 10:44:15 2026 Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id E65ED3D9DA0; Mon, 30 Mar 2026 14:24:26 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=217.140.110.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774880668; cv=none; b=nYTcpIljllj34n5f75oWS+MoKSWpp9q2w01FIU1OYHv2SkfAoqaci8h5qeQh/IIZJzuQFk4dv/ZUWS+ilHgOVqlM8IGGBPaEI+hjjJ5xlfZ4S2yQ/XRSeDDz63HjYyulAD4tdq3ONe3TgBwAn+HJkLkvA/3+1SFEz39tkKAzvqM= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774880668; c=relaxed/simple; bh=qGLvB00ZHRhBLyIPTlP+4u3c9g38wfqMl89bfux33vo=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=gqeEewOzObu9y7B+FxhkthM499m2tqxBqc/5HTX+w04odin8XF11c59ImdZZKVkCxmkno7IFSM7VLBdQ9blKF6+2bNBBiOzzqF11gn8H1lVJJOmMGoiBWvNIlumXOTrZCPeoQ4/XJiiLob6QaNYzkmce2R/bZnxMoqniK0hZSwE= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com; spf=pass smtp.mailfrom=arm.com; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b=bHFQ7URq; arc=none smtp.client-ip=217.140.110.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=arm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=arm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=arm.com header.i=@arm.com header.b="bHFQ7URq" Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 86C971E7D; Mon, 30 Mar 2026 07:24:20 -0700 (PDT) Received: from ewhatever.cambridge.arm.com (ewhatever.cambridge.arm.com [10.1.197.1]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id C521A3F915; Mon, 30 Mar 2026 07:24:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=arm.com; s=foss; t=1774880666; bh=qGLvB00ZHRhBLyIPTlP+4u3c9g38wfqMl89bfux33vo=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=bHFQ7URqfWWfaTZMCwV/J++ssPsLmXIonQD1zZFZAjrgQb8YRudd0ZOWvfL1dw7oH VXcOffFdVyZM6QgXqRcd936ZGDnTRDtbHAVgmQiehRHpSdyPsJA+A1MyWpBgoYhECF RorYs1QimIy3YF9qTogzQF88YEQCOytWl+EK5VD4= From: Suzuki K Poulose To: kvmarm@lists.linux.dev, will@kernel.org Cc: kvm@vger.kernel.org, alexandru.elisei@arm.com, oupton@kernel.org, maz@kernel.org, tabba@google.com, aneesh.kumar@kernel.org, linux-kernel@vger.kernel.org, Oliver Upton , Suzuki K Poulose Subject: [PATCH v7 08/17] arm64: Use KVM_SET_MP_STATE ioctl to power off non-boot vCPUs Date: Mon, 30 Mar 2026 15:23:25 +0100 Message-ID: <20260330142334.3309961-9-suzuki.poulose@arm.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20260330142334.3309961-1-suzuki.poulose@arm.com> References: <20260330142334.3309961-1-suzuki.poulose@arm.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Oliver Upton Using the POWER_OFF flag in kvm_vcpu_init gets in the way of resetting a vCPU in response to a PSCI CPU_ON call, for obvious reasons. Drop the flag in favor of using the KVM_SET_MP_STATE call for non-boot vCPUs. Reviewed-by: Marc Zyngier Signed-off-by: Oliver Upton Signed-off-by: Suzuki K Poulose --- arm64/kvm-cpu.c | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/arm64/kvm-cpu.c b/arm64/kvm-cpu.c index 3d914112..c7286484 100644 --- a/arm64/kvm-cpu.c +++ b/arm64/kvm-cpu.c @@ -143,10 +143,6 @@ struct kvm_cpu *kvm_cpu__arch_init(struct kvm *kvm, un= signed long cpu_id) if (vcpu->kvm_run =3D=3D MAP_FAILED) die("unable to mmap vcpu fd"); =20 - /* VCPU 0 is the boot CPU, the others start in a poweroff state. */ - if (cpu_id > 0) - vcpu_init.features[0] |=3D (1UL << KVM_ARM_VCPU_POWER_OFF); - /* Set KVM_ARM_VCPU_PSCI_0_2 if available */ if (kvm__supports_extension(kvm, KVM_CAP_ARM_PSCI_0_2)) { vcpu_init.features[0] |=3D (1UL << KVM_ARM_VCPU_PSCI_0_2); @@ -201,6 +197,16 @@ struct kvm_cpu *kvm_cpu__arch_init(struct kvm *kvm, un= signed long cpu_id) if (err || target->init(vcpu)) die("Unable to initialise vcpu"); =20 + /* VCPU 0 is the boot CPU, the others start in a poweroff state. */ + if (cpu_id > 0) { + struct kvm_mp_state mp_state =3D { + .mp_state =3D KVM_MP_STATE_STOPPED, + }; + + if (ioctl(vcpu->vcpu_fd, KVM_SET_MP_STATE, &mp_state)) + die_perror("KVM_SET_MP_STATE failed"); + } + coalesced_offset =3D ioctl(kvm->sys_fd, KVM_CHECK_EXTENSION, KVM_CAP_COALESCED_MMIO); if (coalesced_offset) --=20 2.43.0