ptp: kvm: Use decrypted memory in confidential guest on x86
Subsystems (3)
PATCH v2 1/1 ptp: kvm: Use decrypted memory in confidential guest on x86
KVM_HC_CLOCK_PAIRING currently fails inside SEV-SNP guests because the guest passes an address to static data to the host. In confidential computing the host can't access arbitrary guest memory so handling the hypercall runs into an "rmpfault". To make the hypercall work, the guest needs to explicitly mark the memory as decrypted. Do that in kvm_arch_ptp_init(), but retain the previous behavior for non-confidential guests to save us from having to allocate memory.
Add a new arch-specific function (kvm_arch_ptp_exit()) to free the allocation and mark the memory as encrypted again.
---
Hi,
I would love to not allocate a whole page just for this driver, swiotlb is decrypted but I don't have access to a 'struct device' here. Does anyone have any suggestion?
Jeremi
Changes since v1:
- forgot to commit include/linux/ptp_kvm.h
drivers/ptp/ptp_kvm_arm.c | 4 +++ drivers/ptp/ptp_kvm_common.c | 1 + drivers/ptp/ptp_kvm_x86.c | 59 +++++++++++++++++++++++++++++------- include/linux/ptp_kvm.h | 1 + 4 files changed, 54 insertions(+), 11 deletions(-)
drivers/ptp/ptp_kvm_arm.c +4 -0
@@ -22,6 +22,10 @@ int kvm_arch_ptp_init(void)return 0 ; } +void kvm_arch_ptp_exit (void ) +{ +} +int kvm_arch_ptp_get_clock (struct timespec64 *ts) {return kvm_arch_ptp_get_crosststamp (NULL , ts,NULL );
drivers/ptp/ptp_kvm_common.c +1 -0
@@ -130,6 +130,7 @@ static struct kvm_ptp_clock kvm_ptp_clock; static void __exit ptp_kvm_exit(void) {ptp_clock_unregister (kvm_ptp_clock.ptp_clock); +kvm_arch_ptp_exit (); }static int __init ptp_kvm_init (void )
drivers/ptp/ptp_kvm_x86.c +48 -11
92 lines not on this page · raw message
include/linux/ptp_kvm.h +1 -0
@@ -14,6 +14,7 @@ struct timespec64;struct clocksource ;int kvm_arch_ptp_init (void ); +void kvm_arch_ptp_exit (void );int kvm_arch_ptp_get_clock (struct timespec64 *ts);int kvm_arch_ptp_get_crosststamp (u64 *cycle,struct timespec64 *tspec,struct clocksource **cs);
Signature · 1 line
2.25.1
Jakub Kicinski
maintainerQuoted 10 lines · On Mon, 27 Feb 2023 15:58:19 +0000 Jeremi Piotrowski wrote:
KVM_HC_CLOCK_PAIRINGcurrently fails inside SEV-SNP guests because the guest passes an address to static data to the host. In confidential computing the host can't access arbitrary guest memory so handling the hypercall runs into an "rmpfault". To make the hypercall work, the guest needs to explicitly mark the memory as decrypted. Do that inkvm_arch_ptp_init(), but retain the previous behavior for non-confidential guests to save us from having to allocate memory.Add a new arch-specific function (
kvm_arch_ptp_exit()) to free the allocation and mark the memory as encrypted again.
# Form letter - net-next is closed
The merge window for v6.3 has begun and therefore net-next is closed for new drivers, features, code refactoring and optimizations. We are currently accepting bug fixes only.
Please repost when net-next reopens after Mar 6th.
RFC patches sent for review only are obviously welcome at any time.