| Message ID | 20240619174036.483943-1-oliver.upton@linux.dev (mailing list archive) |
|---|---|
| Headers | show |
| Series | KVM: arm64: Allow userspace to modify CTR_EL0 | expand |
On Wed, 19 Jun 2024 17:40:26 +0000, Oliver Upton wrote: > As I'd mentioned on the list, here is my rework of Sebastian's CTR_EL0 > series. Changes this time around: > > - Drop the cross-validation of the guest's CTR_EL0 with CLIDR_EL1 and > the CCSIDR_EL1 hierarchy, instead independently checking these > registers against the system's CTR_EL0 value. > > [...] Applied to kvmarm/next, thanks! [01/10] KVM: arm64: Get sys_reg encoding from descriptor in idregs_debug_show() https://git.kernel.org/kvmarm/kvmarm/c/4e8ff73eb7ae [02/10] KVM: arm64: Make idregs debugfs iterator search sysreg table directly https://git.kernel.org/kvmarm/kvmarm/c/410db103f6eb [03/10] KVM: arm64: Use read-only helper for reading VM ID registers https://git.kernel.org/kvmarm/kvmarm/c/97ca3fcc15cc [04/10] KVM: arm64: Add helper for writing ID regs https://git.kernel.org/kvmarm/kvmarm/c/d7508d27dd88 [05/10] KVM: arm64: nv: Use accessors for modifying ID registers https://git.kernel.org/kvmarm/kvmarm/c/44241f34fac9 [06/10] KVM: arm64: unify code to prepare traps https://git.kernel.org/kvmarm/kvmarm/c/f1ff3fc5209a [07/10] KVM: arm64: Treat CTR_EL0 as a VM feature ID register https://git.kernel.org/kvmarm/kvmarm/c/2843cae26644 [08/10] KVM: arm64: show writable masks for feature registers https://git.kernel.org/kvmarm/kvmarm/c/bb4fa769dcdd [09/10] KVM: arm64: rename functions for invariant sys regs https://git.kernel.org/kvmarm/kvmarm/c/76d36012276a [10/10] KVM: selftests: arm64: Test writes to CTR_EL0 https://git.kernel.org/kvmarm/kvmarm/c/11a31be88fb6 -- Best, Oliver
On Thu, 20 Jun 2024, Oliver Upton wrote: > On Wed, 19 Jun 2024 17:40:26 +0000, Oliver Upton wrote: >> As I'd mentioned on the list, here is my rework of Sebastian's CTR_EL0 >> series. Changes this time around: >> >> - Drop the cross-validation of the guest's CTR_EL0 with CLIDR_EL1 and >> the CCSIDR_EL1 hierarchy, instead independently checking these >> registers against the system's CTR_EL0 value. >> >> [...] > > Applied to kvmarm/next, thanks! Great, thanks a lot! Sebastian
As I'd mentioned on the list, here is my rework of Sebastian's CTR_EL0 series. Changes this time around: - Drop the cross-validation of the guest's CTR_EL0 with CLIDR_EL1 and the CCSIDR_EL1 hierarchy, instead independently checking these registers against the system's CTR_EL0 value. - Rework the idregs debugfs interface to print all VM scoped feature ID registers, which now includes CTR_EL0. - Only reset the VM scoped value of CTR_EL0 once for a VM - Make feature ID register accesses go through read / write helpers, with the intention of abstracting the layout of the registers + adding sanity checks to writers. What I didn't do after all is come up with a better generic way to store ID registers at the VM scope, but the hope is that the accessors will make that trivial to change in the future. Oliver Upton (5): KVM: arm64: Get sys_reg encoding from descriptor in idregs_debug_show() KVM: arm64: Make idregs debugfs iterator search sysreg table directly KVM: arm64: Use read-only helper for reading VM ID registers KVM: arm64: Add helper for writing ID regs KVM: arm64: nv: Use accessors for modifying ID registers Sebastian Ott (5): KVM: arm64: unify code to prepare traps KVM: arm64: Treat CTR_EL0 as a VM feature ID register KVM: arm64: show writable masks for feature registers KVM: arm64: rename functions for invariant sys regs KVM: selftests: arm64: Test writes to CTR_EL0 arch/arm64/include/asm/kvm_emulate.h | 40 +-- arch/arm64/include/asm/kvm_host.h | 26 +- arch/arm64/kvm/arm.c | 2 +- arch/arm64/kvm/nested.c | 256 +++++++++--------- arch/arm64/kvm/pmu-emul.c | 2 +- arch/arm64/kvm/sys_regs.c | 140 ++++++---- .../selftests/kvm/aarch64/set_id_regs.c | 16 ++ 7 files changed, 262 insertions(+), 220 deletions(-) base-commit: 1613e604df0cd359cf2a7fbd9be7a0bcfacfabd0