From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 6F64E3F58E2 for ; Thu, 4 Jun 2026 08:59:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780563547; cv=none; b=djeFoqWv1twz3RTEPjZctECqb0v79M1xcDLXb4tTp/509XvEhjBJAQfgEQAiyDhhAtqcNt4631W+ZeeiPe006ah4RtZcJJmtc1d9hMdRlSoWb+8XkSMHIhZs2mcq0y7+dm6iYzFF3Yge2vIKaPY2oVRFaP3nr6BeXHJOHQAbY2c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1780563547; c=relaxed/simple; bh=x4+PC/8GNgN2U/9j72ZDlusahJYEOfeeDsm7yFTBcPM=; h=From:To:Cc:Subject:In-Reply-To:References:Date:Message-ID: MIME-Version:Content-Type; b=uKIrgHXJoUGExJLraswRP3Agd+vXinCRLjftOOF/nr/nhLTiqpYgg4q8gsv2c1mA696NzSNBdVLCIQhWVaUFBYIFxKbkzNwgRsmL7NBBxj7xUtiePAscNmig2JS6CR8yMCreVG4SENG9X5KFFN47TrXE7MvpCQSgPSAFxsbLN/k= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=ICi+m2HI; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="ICi+m2HI" Received: by smtp.kernel.org (Postfix) with ESMTPSA id AA7571F00893; Thu, 4 Jun 2026 08:59:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1780563546; bh=e0tO+AXwvzwsR7hiiJIrE45BPYOVlZEsl/62oMi/RRQ=; h=From:To:Cc:Subject:In-Reply-To:References:Date; b=ICi+m2HIEnZNl1BehmK/+66vRKvHjYUFk4TJyUZbA+YfBqGaVSkUrtx/ZLMAiiOS9 D+AUbtvnCtzpBLZP93wmeJ5NMSs90OWjozSrl/6hMCQFn6bmXL1l9jXVKTJDvBZW/C z4D43pW43uFItUBz++xzjw3Obqq+CWRPQAyIHfwJI+7s13e+aOFVzAKssmm+sR8nr4 F8v35knExCwFFQWmewEiJfvZGTeqJRK2QjRElDrcemlF7GmQe87n9kfm4VFZMA+ul3 RUcPrPNa29EXnF3sSR/c53jr9yJ7mzvtHlXakmcyMiY6b8LlIJyPBzMOk+6/l0hxpc X9zDcb6HrQvoA== X-Mailer: emacs 30.2 (via feedmail 11-beta-1 I) From: Aneesh Kumar K.V To: Mostafa Saleh , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Cc: akpm@linux-foundation.org, catalin.marinas@arm.com, will@kernel.org, rppt@kernel.org, maz@kernel.org, Mostafa Saleh Subject: Re: [PATCH 3/3] arm64/coco: Add pKVM as a CC platform In-Reply-To: <20260603110522.3331819-4-smostafa@google.com> References: <20260603110522.3331819-1-smostafa@google.com> <20260603110522.3331819-4-smostafa@google.com> Date: Thu, 04 Jun 2026 14:29:00 +0530 Message-ID: Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain Mostafa Saleh writes: > pKVM does support memory encryption, expose that to the rest of > the kernel through cc_platform_has() > > At the moment, all devices inside the guest are emulated which > requires its memory to be shared back to the host (decrypted), so > set force_dma_unencrypted() to always return true. > > Although, typically pKVM guests rely on restricted-dma-pools to > bounce traffic, with this change, it is possible to solely rely on > the default SWIOTLB for that (assuming the appropriate size is set > from the command line) > > Signed-off-by: Mostafa Saleh > --- > This change is critical for the ongoing refactoring of the DMA-API[1] > that will break protected guests under pKVM with this patch. That is > due to this rework will make the state of the SWIOTLB and restricted > dma pools depends on the value returned by cc_platform_has() > > [1] https://lore.kernel.org/all/20260522042815.370873-1-aneesh.kumar@kernel.org/ > --- > arch/arm64/include/asm/hypervisor.h | 13 +++++++++++++ > arch/arm64/include/asm/mem_encrypt.h | 3 ++- > arch/arm64/kernel/rsi.c | 12 ------------ > arch/arm64/mm/init.c | 15 ++++++++++++++- > drivers/virt/coco/pkvm-guest/arm-pkvm-guest.c | 3 +++ > 5 files changed, 32 insertions(+), 14 deletions(-) > > diff --git a/arch/arm64/include/asm/hypervisor.h b/arch/arm64/include/asm/hypervisor.h > index a12fd897c877..8889a0ba1ec5 100644 > --- a/arch/arm64/include/asm/hypervisor.h > +++ b/arch/arm64/include/asm/hypervisor.h > @@ -3,6 +3,9 @@ > #define _ASM_ARM64_HYPERVISOR_H > > #include > +#include > + > +DECLARE_STATIC_KEY_FALSE(pkvm_guest); > > void kvm_init_hyp_services(void); > bool kvm_arm_hyp_service_available(u32 func_id); > @@ -10,8 +13,18 @@ void kvm_arm_target_impl_cpu_init(void); > > #ifdef CONFIG_ARM_PKVM_GUEST > void pkvm_init_hyp_services(void); > + > +static inline bool is_protected_kvm_guest(void) > +{ > + return static_branch_unlikely(&pkvm_guest); > +} > #else > static inline void pkvm_init_hyp_services(void) { }; > + > +static inline bool is_protected_kvm_guest(void) > +{ > + return false; > +} > #endif > > static inline void kvm_arch_init_hyp_services(void) > ... > diff --git a/drivers/virt/coco/pkvm-guest/arm-pkvm-guest.c b/drivers/virt/coco/pkvm-guest/arm-pkvm-guest.c > index d66291def0f4..26fe9c3f22e3 100644 > --- a/drivers/virt/coco/pkvm-guest/arm-pkvm-guest.c > +++ b/drivers/virt/coco/pkvm-guest/arm-pkvm-guest.c > @@ -17,6 +17,7 @@ > #include > > static size_t pkvm_granule; > +DEFINE_STATIC_KEY_FALSE_RO(pkvm_guest); > Do we need EXPORT_SYMBOL on this? > > static int arm_smccc_do_one_page(u32 func_id, phys_addr_t phys) > { > @@ -120,4 +121,6 @@ void pkvm_init_hyp_services(void) > > if (kvm_arm_hyp_service_available(ARM_SMCCC_KVM_FUNC_MMIO_GUARD)) > arm64_ioremap_prot_hook_register(&mmio_guard_ioremap_hook); > + > + static_branch_enable(&pkvm_guest); > } > -- > 2.54.0.1032.g2f8565e1d1-goog -aneesh