From mboxrd@z Thu Jan 1 00:00:00 1970 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 19E72C83F34 for ; Sat, 19 Jul 2025 02:29:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:Reply-To:List-Subscribe: List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id:Cc:To: Content-Transfer-Encoding:Content-Type:MIME-Version:Message-Id:Date:Subject: From:Content-ID:Content-Description:Resent-Date:Resent-From:Resent-Sender: Resent-To:Resent-Cc:Resent-Message-ID:In-Reply-To:References:List-Owner; bh=o0906EP5K0r2F2xgUoUqkJgm+2hZT0Ft0/SR/1igcPA=; b=iIDKkBbEgLmHXtPqFdzT8+HMX7 g9KasxZWSwHAfZf89MGLNyQ8BVD4Z1kRdTOvQwnkQoyOXQwyoUREw/Cwra0m3ajOnVhg0g7yQS4Fn pkGbDYCrV7ePwrJAY8O//qni8UoD6lhxoi92AOgzU82kC++L/j3NPcWmjr/ku8X+S2AIK2gQiTrGX vwwL6i1dax3FLb4hAG+tSCgJ+TN+xfV/4pOnm/7rFCQWYWjk7EsWtEuWE3ihnkbq9GJ2xin7zNmU9 Nk3hWK4lCEbNpOn/gIOC1PZQ2oeE4LPFG6/PkimI0dKgRh+1WBdKRI89PUXdgNxNjPXWxOo96szgb D87s83uw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98.2 #2 (Red Hat Linux)) id 1ucxKJ-0000000Dka8-2rAZ; Sat, 19 Jul 2025 02:29:27 +0000 Received: from sea.source.kernel.org ([172.234.252.31]) by bombadil.infradead.org with esmtps (Exim 4.98.2 #2 (Red Hat Linux)) id 1ucx3L-0000000DjJO-0Nkv for linux-arm-kernel@lists.infradead.org; Sat, 19 Jul 2025 02:11:57 +0000 Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id D24DA451ED; Sat, 19 Jul 2025 02:11:53 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPS id 9A1A0C4CEEB; Sat, 19 Jul 2025 02:11:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1752891113; bh=S9tg4OxSGoLcSw6DTtpLwBLOnhyECCWkSW4GScn6akg=; h=From:Subject:Date:To:Cc:Reply-To:From; b=hDlWLWwJb1gHocb2dUgSue5I1WOtEZKhzUPzJbvm2VVvb2gyjxUGre1bAIR/1/csy Tg1bVr4K7fn8IMaqsVDraqgJwzsksfmymSt3kVGek/wDVYY5cEVrYnkIAjkASI4x1x /lesEfd4TEl/I1fblfvBwmHgVxdjpzeCfpXzdlKyYApL43dxz8l59eddUWHd4BTTrl tthFZp8cdLASdsGsutEBNOGB0GpMO+uuueipl9c2Qx25UmyAPk8Sws2+AOlBejqY9k 7hii2uiIqGuIEvx4TBP9mxMLZYntMYGok9Z85bU2A0l50TjEwcS7B9FMpqtUl6WcIs elvH71/5rVzng== Received: from aws-us-west-2-korg-lkml-1.web.codeaurora.org (localhost.localdomain [127.0.0.1]) by smtp.lore.kernel.org (Postfix) with ESMTP id 86F96C83F1A; Sat, 19 Jul 2025 02:11:53 +0000 (UTC) From: Per Larsen via B4 Relay Subject: [PATCH v8 0/7] KVM: arm64: Support FF-A 1.2 and SEND_DIRECT2 ABI Date: Sat, 19 Jul 2025 02:11:22 +0000 Message-Id: <20250719-virtio-msg-ffa-v8-0-03e8e8dbe856@google.com> MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-B4-Tracking: v=1; b=H4sIAMr+emgC/2XQTWrDMBCG4asEr6swGv26q9yjdKHKI0cQW0F2T Uvw3SsbStxqOYvn5WMezUQ50tS8nh5NpiVOMY3lsC+nxl/d2BOLXbkbBFSgQLMl5jkmNkw9C8E xRBcM+ha5gqage6YQv/bg23u5Q04Dm6+Z3DGDUADYs0CUQnLG2Z3y7RKH4XN043z2adhi1zjNK X/v4xbckr8B+3/HggwYdVYGC0q0Wl36lPob7altyCIOnovKi+I7rb3RIfAP9JWXR1/9YZHFKwt EBqjs45VXT695W3lVvOTlk1Ypa8FUXh88msrr4q0HDF0rqXP1fvP0BnjlTfFtq1zwohNW0R+/r usP5dMV/ioCAAA= X-Change-ID: 20250506-virtio-msg-ffa-22af72c92150 To: Marc Zyngier , Oliver Upton , Joey Gouly , Suzuki K Poulose , Zenghui Yu , Catalin Marinas , Will Deacon , Sudeep Holla Cc: linux-arm-kernel@lists.infradead.org, kvmarm@lists.linux.dev, linux-kernel@vger.kernel.org, ahomescu@google.com, armellel@google.com, arve@android.com, ayrton@google.com, qperret@google.com, sebastianene@google.com, qwandor@google.com, Per Larsen X-Mailer: b4 0.14.2 X-Developer-Signature: v=1; a=ed25519-sha256; t=1752891112; l=5422; i=perlarsen@google.com; s=20250508; h=from:subject:message-id; bh=S9tg4OxSGoLcSw6DTtpLwBLOnhyECCWkSW4GScn6akg=; b=xBzVi6qON6PxUBnnfhGd3EnytKzJFu/s+0dfKWqFXLULoHYjU3fG+OrFj8H1ZqJ7+Ju8kAFZW GlWvwAGM2R3ApTbbtM4IIfHWYNU4LMgQX/bTn2YaQaPUqGUDIx47ckS X-Developer-Key: i=perlarsen@google.com; a=ed25519; pk=jjc/Ta4VmrLRmMoahP6d1mBcKzvWU+nsmdtYe2oS2kQ= X-Endpoint-Received: by B4 Relay for perlarsen@google.com/20250508 with auth_id=402 X-Original-From: Per Larsen X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250718_191155_171394_696031C6 X-CRM114-Status: GOOD ( 15.69 ) 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: , Reply-To: perlarsen@google.com Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hi, The FF-A 1.2 specification introduces a new SEND_DIRECT2 ABI which allows registers x4-x17 to be used for the message payload. This patch set prevents the host from using a lower FF-A version than what has already been negotiated with the hypervisor. This is necessary because the hypervisor does not have the necessary compatibility paths to translate from the hypervisor FF-A version to a previous version. Support for FF-A 1.2 in the hypervisor is added as a precursor to the addition of the FFA_MSG_SEND_DIRECT_REQ2 messaging interface. Notably, this patch updates all smc calls to use SMCCC 1.2 as it makes it simpler to support interfaces that need to accept more than 8 arguments and/or return more than 4 values. The list of optional/unsupported interfaces was updated to reflect additions in FF-A 1.2. Tested by booting Android under QEMU and loading Trusty as the guest VM and observing the SEND_DIRECT2 ABI being used successfully during guest boot. Changes in v8: - 2/5 -> 2/7: Use func_id argument to distinguish SMC32/SMC64. Update rationale. - 4/5 -> 4/7: Split up 4/5: Add optional FF-A 1.2 interfaces - 4/5 -> 5/7: Split up 4/5: Mask FFA_RXTX_MAP minium buffer size in a2. Don't check MBZ bits. - 4/5 -> 6/7: Remainder of 4/5: Bump supported version to FF-A 1.2 - 5/5 -> 7/7: Don't check src id in do_ffa_direct_msg2 - Link to v7: https://lore.kernel.org/r/20250701-virtio-msg-ffa-v7-0-995afc3d385e@google.com Changes in v7: - 0/6: Revised cover letter slightly - 4/6: ffa_call_supported: Clarify optional interfaces hyp_ffa_post_init: Sanitize MBZ bits in x2 and x3 address nit: use GENMASK - Link to v6: https://lore.kernel.org/r/20250627-virtio-msg-ffa-v6-0-8c02fd94edac@google.com Changes since v5: - 3/6 -> 2/5: Merged with parent patch as they were closely related Don't rename variables from "res" to "regs" to avoid churn Update and clarify reasoning for condition added to ffa_set_retval Adopt style suggested by Marc when passing regs to arm_smccc_1_2_smc Fix inadvertent duplicate statements in ffa_{map,unmap}_hyp_buffers - Link to v5: https://lore.kernel.org/r/20250619-virtio-msg-ffa-v5-0-412c98558807@google.com Changes since v4: - 1/5 -> 1/6: Added Acked-by: Will Deacon Reworded patch subject and description - 2/5: Dropped patch: zero x4-x7 in ffa_set_retval - 2/6: New: use SMCCC 1.2 during hyp init - 3/6: New: use SMCCC 1.2 in host FF-A handler - 3/5 -> 4/6: No change; can revisit denylist design in later patchset Added Acked-by: Will Deacon - 4/5 -> 5/6: No change - 5/5 -> 6/6: No change - Link to v4: https://lore.kernel.org/r/20250516-virtio-msg-ffa-v4-0-580ee70e5081@google.com Changes since v3: - all: Remove Signed-off-by: Per Larsen - 2/5: Split out from 2/3, zero x4-x7 in set_ffa_retval - 3/5: Split out from 2/3, Mark FFA_NOTIFICATION_* calls as unsupported - 2/3 -> 4/5: Drop ffa_get_hypervisor_version, access hyp_ffa_version directly when version negotiation is complete. - 3/3 -> 5/5: Call ffa_to_smccc_1_2_error directly in do_do_ffa_direct_msg2 Push call to ffa_call_needs_smccc_1_2 lower down by adding ffa_get_func_id and ffa_set_retval_smccc_1_x. Drop ffa_to_smccc_1_2_regs and ffa_to_smccc_1_2_regs_prop as they are no longer used. - Link to v3: https://lore.kernel.org/r/20250513-virtio-msg-ffa-v3-0-d66c76ff1b2c@google.com Changes since v2: - 2/3: Removed files added by mistake. Add and use ffa_get_hypervisor_version to access hyp_ffa_version - 3/3: Use ffa_get_hypervisor_version to access hyp_ffa_version safely - Link to v2: https://lore.kernel.org/r/20250508-virtio-msg-ffa-v2-0-ed84f8053965@google.com Changes since v1: - 1/3: Simplify commit message; drop long comment in do_ffa_version - 2/3: Correct use of Co-developed-by: footer s/arm_smccc_1_2_smc_fallback/arm_smccc_1_x_smc/ Always access hyp_ffa_version w/lock held Remove superfluous comments in ffa_call_supported Add and use FFA_FEAT_RXTX_MIN_SZ_MASK instead of constant Add FFA_PARTITION_INFO_GET_REGS to calls that require SMCCC 1.2 - 3/3: Always access hyp_ffa_version w/lock held Correct formatting Thanks, Per Larsen -- 2.49.0 --- Per Larsen (7): KVM: arm64: Correct return value on host version downgrade attempt KVM: arm64: Use SMCCC 1.2 for FF-A initialization and in host handler KVM: arm64: Mark FFA_NOTIFICATION_* calls as unsupported KVM: arm64: Mark optional FF-A 1.2 interfaces as unsupported KVM: arm64: Mask response to FFA_FEATURE call KVM: arm64: Bump the supported version of FF-A to 1.2 KVM: arm64: Support FFA_MSG_SEND_DIRECT_REQ2 in host handler arch/arm64/kvm/hyp/nvhe/Makefile | 1 + arch/arm64/kvm/hyp/nvhe/ffa.c | 227 ++++++++++++++++++++++++++------------- include/linux/arm_ffa.h | 1 + 3 files changed, 156 insertions(+), 73 deletions(-) --- base-commit: 414aaef1537ac7f90dcb54e618864680aca2e197 change-id: 20250506-virtio-msg-ffa-22af72c92150 Best regards, -- Per Larsen