From patchwork Mon Mar 18 16:14:22 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: David Woodhouse X-Patchwork-Id: 13595583 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BD7B1C54E58 for ; Mon, 18 Mar 2024 16:47:19 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:Message-ID:Date:Subject:Cc :To:From:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From: Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References: List-Owner; bh=9lL1h9Fcla6wEB5ySrNZlidM0wIaUKwLYqzIkktD1N0=; b=5Gr/lnkigV/ckO Ggh2/su3TdBamfcrEt12wFZ72ANyoSjA6fF3qw8u6ghGdCEtNWTqX6QkAjHZ+7R6OWy7z3SxBUnxh rHgxGjN+SXMZ+r8CHw+4qLcSdMCBYz44lwVA1hiCfQLq32St4+ULE9zIsHMjQeZOV0cth7CeSx4hI TMcJ9kUESMyO/uiPQcTa3AsuIqf5T1YN7slk3RYCx95OpiZ0Q4zIggdxCtmmZ0Yui2uFyQE45sO+F n9vQjoNgeRYXQpr04QH4V1AuxZKjdcWGy+YmvFhdYaGrmMqB8L+P0pkgRtl2GCPjkWG7cxYH99IGf LHbT4xL+Wr+JOCtVyW0Q==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmG8l-00000009I50-2RiL; Mon, 18 Mar 2024 16:47:11 +0000 Received: from desiato.infradead.org ([2001:8b0:10b:1:d65d:64ff:fe57:4e05]) by bombadil.infradead.org with esmtps (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmG8i-00000009I3t-2i6T for linux-arm-kernel@bombadil.infradead.org; Mon, 18 Mar 2024 16:47:08 +0000 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=infradead.org; s=desiato.20200630; h=Sender:Content-Transfer-Encoding: MIME-Version:Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-Type: Content-ID:Content-Description:In-Reply-To:References; bh=PepfWuDb0qWcSY62ic0M/q/leQ8EAPZZpQYANiCR9/0=; b=eLDnx06lB1n0d9bn85UZrkkWTI hE3cwCH64vBWVmVAyRNCovdc7I04JyPFUnR+MMxag37V6dpdmF02pjb1JzAulwnek4W1vfUQnJVZU lq6arc45Z3h/rVIxLnfp7GwgAHE73Uzc7YScPvMI9hTiQh/nSoQsjFn0cyKV86XzlRgO6i2d750VN S6znG4zO08OV+32EV1CNXFl16h/mn/Ttz2/2pVrRNUVThXNlcJ14AtSvKyzRnmJKkHlz454dfswdB EBiCgdPegSNpUlq2u/fYp3TK2zVpc4zFB1+8UJzBsUBXUli6ab8pWKsabG40DS7M0SQczFoy4D3xW Axl4yGkQ==; Received: from [2001:8b0:10b:1::ebe] (helo=i7.infradead.org) by desiato.infradead.org with esmtpsa (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmG8Z-0000000CjyA-1kHI; Mon, 18 Mar 2024 16:46:59 +0000 Received: from dwoodhou by i7.infradead.org with local (Exim 4.97.1 #2 (Red Hat Linux)) id 1rmG8W-00000004F1A-0SES; Mon, 18 Mar 2024 16:46:56 +0000 From: David Woodhouse To: linux-arm-kernel@lists.infradead.org, kvm@vger.kernel.org Cc: Paolo Bonzini , Jonathan Corbet , Marc Zyngier , Oliver Upton , James Morse , Suzuki K Poulose , Zenghui Yu , Catalin Marinas , Will Deacon , Mark Rutland , Lorenzo Pieralisi , "Rafael J. Wysocki" , Len Brown , Pavel Machek , David Woodhouse , Mostafa Saleh , Jean-Philippe Brucker , linux-doc@vger.kernel.org, linux-kernel@vger.kernel.org, kvmarm@lists.linux.dev, linux-pm@vger.kernel.org Subject: [RFC PATCH v2 0/4] arm64: Add PSCI v1.3 SYSTEM_OFF2 support for hibernation Date: Mon, 18 Mar 2024 16:14:22 +0000 Message-ID: <20240318164646.1010092-1-dwmw2@infradead.org> X-Mailer: git-send-email 2.44.0 MIME-Version: 1.0 X-SRS-Rewrite: SMTP reverse-path rewritten from by desiato.infradead.org. See http://www.infradead.org/rpr.html X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org The PSCI v1.3 spec (https://developer.arm.com/documentation/den0022, currently in Alpha state, hence 'RFC') adds support for a SYSTEM_OFF2 function enabling a HIBERNATE_OFF state which is analogous to ACPI S4. This will allow hosting environments to determine that a guest is hibernated rather than just powered off, and ensure that they preserve the virtual environment appropriately to allow the guest to resume safely (or bump the hardware_signature in the FACS to trigger a clean reboot instead). This adds support for it to KVM, exactly the same way as the existing support for SYSTEM_RESET2 as added in commits d43583b890e7 ("KVM: arm64: Expose PSCI SYSTEM_RESET2 call to the guest") and 34739fd95fab ("KVM: arm64: Indicate SYSTEM_RESET2 in kvm_run::system_event flags field"). Back then, KVM was unconditionally bumped to expose PSCI v1.1. This means that a kernel upgrade causes guest visible behaviour changes without any explicit opt-in from the VMM, which is... unconventional. In some cases, a PSCI update isn't just about new optional calls; PSCI v1.2 for example adds a new permitted error return from the existing CPU_ON function. There *is* a way for a VMM to opt *out* of newer PSCI versions... by setting a per-vCPU "special" register that actually ends up setting the PSCI version KVM-wide. Quite why this isn't just a simple KVM_CAP, I have no idea. There *is* a KVM_CAP_ARM_PSCI_0_2 but that's just for 0.1 vs. 0.2+, not the specific v0.2+ version that's exposed. Since the SYSTEM_OFF2 call is optional and discoverable through the PSCI_FEATURES call, I'm electing not to touch the PSCI versioning awfulness at all. Like the existing SYSTEM_RESET2, there's a KVM_CAP to enable it explicitly (as it's an optional call even in v1.3), and like the existing SYSTEM_RESET2 it doesn't depend on the advertised PSCI version. For the guest side, add a new SYS_OFF_MODE_POWER_OFF handler with higher priority than the EFI one, but which *only* triggers when there's a hibernation in progress. There are other ways to do this (see the commit message for more details) but this seemed like the simplest. Version 2 of the patch series splits out the psci.h definitions into a separate commit (a dependency for both the guest and KVM side), and adds definitions for the other new functions added in v1.3. It also moves the pKVM psci-relay support to a separate commit; although in arch/arm64/kvm that's actually about the *guest* side of SYSTEM_OFF2 (i.e. using it from the host kernel, relayed through nVHE). David Woodhouse (4): firmware/psci: Add definitions for PSCI v1.3 specification (ALPHA) KVM: arm64: Add PSCI SYSTEM_OFF2 function for hibernation KVM: arm64: nvhe: Pass through PSCI v1.3 SYSTEM_OFF2 call arm64: Use SYSTEM_OFF2 PSCI call to power off for hibernate Documentation/virt/kvm/api.rst | 11 +++++++++++ arch/arm64/include/asm/kvm_host.h | 2 ++ arch/arm64/include/uapi/asm/kvm.h | 6 ++++++ arch/arm64/kvm/arm.c | 5 +++++ arch/arm64/kvm/hyp/nvhe/psci-relay.c | 2 ++ arch/arm64/kvm/psci.c | 37 ++++++++++++++++++++++++++++++++++++ drivers/firmware/psci/psci.c | 35 ++++++++++++++++++++++++++++++++++ include/uapi/linux/kvm.h | 1 + include/uapi/linux/psci.h | 20 +++++++++++++++++++ kernel/power/hibernate.c | 5 ++++- 10 files changed, 123 insertions(+), 1 deletion(-)