Add support for NoTagAccess memory attribute
Get series with b4
b4 am [email protected]Versions newest first · linked by author and title
- v2 Latest7 patches 11 repliesIn review
- v14 patches 12 repliesSuperseded
Changes in v2 from the cover letter
* Add KVM_CAP_ARM_MTE_PERM to handle migration.
* Add handling of NoTagAccess with Nested guest stage2.
* Add changes to split some of the kvm_pgtable_prot bits.
Patches in v2 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/7 | arm64: Update the values to binary from hexarch/ | +6 | 1 reply | |
| 2/7 | KVM: arm64: MTE: Update code commentsarch/ | +5 | 1 reply | |
| 3/7 | arm64: cpufeature: add Allocation Tag Access Permission (MTE_PERM) featurearch/ arch/ arch/ +1 more | +17 | 1 reply | |
| 4/7 | KVM: arm64: MTE: Add KVM_CAP_ARM_MTE_PERMDocumentation/ arch/ arch/ +1 more | +33 | 0 replies | |
| 5/7 | KVM: arm64: MTE: Use stage-2 NoTagAccess memory attribute if supportedDocumentation/ arch/ arch/ +4 more | +47 | 8 replies | |
| 6/7 | KVM: arm64: MTE: Nested guest supportarch/ arch/ arch/ +1 more | +59 | 0 replies | |
| 7/7 | KVM: arm64: Split some of the kvm_pgtable_prot bits into separate definesarch/ arch/ arch/ +1 more | +36 | 0 replies |
Reviews trailers per patch
2 of 7 patches have a Reviewed-by or Acked-by.
| Patch | Catalin Marinas |
|---|---|
| 1/7 | A-b |
| 2/7 | |
| 3/7 | R-b |
| 4/7 | |
| 5/7 | |
| 6/7 | |
| 7/7 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.
Cover letter [PATCH v2 0/7] Add support for NoTagAccess memory attribute
A VMM allows assigning different types of memory regions to the guest and not all memory regions support storing allocation tags. Currently, the kernel doesn't allow enabling the MTE feature in the guest if any of the assigned memory regions don't allow MTE. This prevents the usage of MTE in the guest even though the guest will never use these memory regions as allocation-tagged memory.
One example of such a configuration is a VFIO passthrough-enabled guest. Enabling MTE with such config results in failure as shown below(kvmtool VMM)
[ 617.921030] vfio-pci 0000:01:00.0: resetting [ 618.024719] vfio-pci 0000:01:00.0: reset done Error: 0000:01:00.0: failed to register region with KVM Warning: [0abc:aced] Error activating emulation for BAR 0 Error: 0000:01:00.0: failed to configure regions Warning: Failed init: vfio__init
Fatal: Initialisation failed
This patch series provides a way to enable MTE in such configs. Even though NoTagAccess can only be used with cacheable mapping, having the ability to map both MTE capable and non-capable VMAs allow us to support VFIO with the MTE capability enabled.
Also, there is a possibility of cachable device memory. That memory, if exposed to guest as WB and the guest enables MTE, may trigger SErrors (Arm ARM mention this as "[RBCFRK] The result of an access to an Allocation Tag where Allocation Tag storage is not provided is IMPLEMENTATION DEFINED"). With FEAT_MTE_PERM KVM could trap and do necessary corrective action.
Another use case is virtiofs dax support, which can use a page cache region as a virtio-shm region. We can use MTE_PERM to enable MTE in this config.
In summary, different types of memory, whether WB MMIO or RAM presented as virtio-(pmem, shm etc.) backed by files in the VMM, the VM should be aware it is not standard RAM and should not attempt to enable MTE on it. If it does (either by mistake or malice), FEAT_MTE_PERM will trap and cause a VM exit with KVM_VM_EXIT_MEMORY_FAULT as the reason.
A VM exit with exit reason KVM_VM_EXIT_MEMORY_FAULT, gives sufficient flexibility for any future fault-handling schemes we come up with (One possible use case is to allocate additional allocation tag space for schemes that want to use this for supporting smaller allocation tag pool and retry the guest instruction again). For the current implementation where we expect the guest to be terminated, this can also be achieved from within the hypervisor.
Implementation notes:
For non-MTE-allowed memory, the hypervisor will install stage2 translation with NoTagAccess memory attributes. Guest access of allocation tags with these memory regions will result in a VM Exit. One detail to note here is that NoTagAccess memory attribute can only be applied to Normal cacheable memory ie, using the attribute value of MemAttr[3:0] = 0b0100 implies Normal, NoTagAccess, write-back cacheable memory region. No other memory attribute value will trap the allocation tag access.
Migration notes:
The feature is only exposed to an EL2 guest only if it is capable of nested virtualization. Otherwise, a read of ID_AA64PFR2_EL1_FPMR will not expose MTE_PERM feature. We also want to make sure that an EL2 guest using this feature as part of its stage2 translation can only migrate to a target that supports the feature in the hardware. This is achieved by using KVM_CAP_ARM_MTE_PERM.
Nested virtualization notes:
This being a stage2 translation attribute, it is exposed to EL2 guest only if it is capable of a VirtualEL2 state. When an EL1 guest is started with MTE_PERM capability enabled, the EL2 hypervisor will look at the EL1 stage2 tables to determine whether a NoTagAccess attribute needs to be inserted in the shadow stage2 table at EL2. (Limitation, upstream nested virt support disables MTE in EL1 guest and in a similar fashion, we don't allow MTE_PERM with EL1 guest for now. This also mean we can drop the patch "KVM: arm64: MTE: Nested guest support" because the feature is only used by EL2 guest for now.)
Changes from v1:
* Add KVM_CAP_ARM_MTE_PERM to handle migration.
* Add handling of NoTagAccess with Nested guest stage2.
* Add changes to split some of the kvm_pgtable_prot bits.
Aneesh Kumar K.V (Arm) (7):
arm64: Update the values to binary from hex
KVM: arm64: MTE: Update code comments
arm64: cpufeature: add Allocation Tag Access Permission (MTE_PERM)
feature
KVM: arm64: MTE: Add KVM_CAP_ARM_MTE_PERM
KVM: arm64: MTE: Use stage-2 NoTagAccess memory attribute if supported
KVM: arm64: MTE: Nested guest support
KVM: arm64: Split some of the kvm_pgtable_prot bits into separate
defines
Documentation/virt/kvm/api.rst | 17 ++++++++++ arch/arm64/include/asm/cpufeature.h | 5 +++ arch/arm64/include/asm/kvm_emulate.h | 5 +++ arch/arm64/include/asm/kvm_host.h | 7 +++++ arch/arm64/include/asm/kvm_nested.h | 10 ++++++ arch/arm64/include/asm/kvm_pgtable.h | 9 ++++-- arch/arm64/include/asm/memory.h | 14 +++++---- arch/arm64/kernel/cpufeature.c | 9 ++++++ arch/arm64/kvm/arm.c | 11 +++++++ arch/arm64/kvm/hyp/nvhe/mem_protect.c | 2 +- arch/arm64/kvm/hyp/pgtable.c | 43 +++++++++++++++---------- arch/arm64/kvm/mmu.c | 45 ++++++++++++++++++++------- arch/arm64/kvm/nested.c | 28 +++++++++++++++++ arch/arm64/kvm/sys_regs.c | 15 ++++++--- arch/arm64/tools/cpucaps | 1 + include/linux/kvm_host.h | 10 ++++++ include/uapi/linux/kvm.h | 2 ++ 17 files changed, 191 insertions(+), 42 deletions(-)
base-commit: 56e6a3499e14716b9a28a307bb6d18c10e95301e
Signature · 1 line
2.43.0