Skip to content

KVM: arm64: pvtime: Fixes and a new cap

Andrew Jones kvm kvmarm 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest6 patches 3 repliesIn review
  2. v15 patches 14 repliesSuperseded

Changes in v2 from the cover letter

- ARM_SMCCC_HV_PV_TIME_FEATURES now also returns SMCCC_RET_NOT_SUPPORTED when steal time is not supported
- Added READ_ONCE() for the run_delay read
- Reworked kvm_put/get_guest to not require type as a parameter
- Added some more text to the documentation for KVM_CAP_STEAL_TIME
- Enough changed that I didn't pick up Steven's r-b's

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/6KVM: arm64: pvtime: steal-time is only supported when configuredarch/arm64/kvm/pvtime.c+2 -10 replies
2/6KVM: arm64: pvtime: Fix potential loss of stolen timearch/arm64/kvm/pvtime.c+3 -20 replies
3/6KVM: arm64: Drop type input from kvm_put_guestarch/arm64/kvm/pvtime.c include/linux/kvm_host.h+7 -60 replies
4/6KVM: arm64: pvtime: Fix stolen time accounting across migrationarch/arm64/include/asm/kvm_host.h arch/arm64/kvm/pvtime.c include/linux/kvm_host.h+29 -150 replies
5/6KVM: Documentation: Minor fixupsDocumentation/virt/kvm/api.rst+5 -40 replies
6/6arm64/x86: KVM: Introduce steal-time capDocumentation/virt/kvm/api.rst arch/arm64/include/asm/kvm_host.h arch/arm64/kvm/arm.c +3 more+22 -10 replies

Reviews trailers per patch

2 of 6 patches have a Reviewed-by or Acked-by.

PatchSteven Price
1/6
2/6
3/6
4/6
5/6R-b
6/6R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [PATCH v2 0/6] KVM: arm64: pvtime: Fixes and a new cap

v2:
- ARM_SMCCC_HV_PV_TIME_FEATURES now also returns SMCCC_RET_NOT_SUPPORTED when steal time is not supported
- Added READ_ONCE() for the run_delay read
- Reworked kvm_put/get_guest to not require type as a parameter
- Added some more text to the documentation for KVM_CAP_STEAL_TIME
- Enough changed that I didn't pick up Steven's r-b's

The first four patches in the series are fixes that come from testing and reviewing pvtime code while writing the QEMU support[*]. The last patch is only a convenience for userspace, and I wouldn't be heartbroken if it wasn't deemed worth it. The QEMU patches are currently written without the cap. However, if the cap is accepted, then I'll change the QEMU code to use it.

Thanks,
drew

[*] https://lists.gnu.org/archive/html/qemu-devel/2020-07/msg03856.html
(a v2 of this series will also be posted shortly)

Andrew Jones (6):
KVM: arm64: pvtime: steal-time is only supported when configured
KVM: arm64: pvtime: Fix potential loss of stolen time
KVM: arm64: Drop type input from kvm_put_guest
KVM: arm64: pvtime: Fix stolen time accounting across migration
KVM: Documentation: Minor fixups
arm64/x86: KVM: Introduce steal-time cap

 Documentation/virt/kvm/api.rst    | 22 ++++++++++++++++++----
 arch/arm64/include/asm/kvm_host.h |  2 +-
 arch/arm64/kvm/arm.c              |  3 +++
 arch/arm64/kvm/pvtime.c           | 29 +++++++++++++----------------
 arch/x86/kvm/x86.c                |  3 +++
 include/linux/kvm_host.h          | 31 ++++++++++++++++++++++++++-----
 include/uapi/linux/kvm.h          |  1 +
 7 files changed, 65 insertions(+), 26 deletions(-)
Signature · 6 lines
2.25.4

_______________________________________________
kvmarm mailing list
[email protected]
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm