[PATCH v10 0/3] RISC-V: KVM: Validate SBI STA shmem alignment

Jiakai Xu posted 3 patches 4 weeks, 1 day ago
arch/riscv/kvm/vcpu_sbi_sta.c                 | 16 ++-
.../selftests/kvm/include/kvm_util_types.h    |  2 +
tools/testing/selftests/kvm/steal_time.c      | 98 ++++++++++++++++---
3 files changed, 95 insertions(+), 21 deletions(-)
[PATCH v10 0/3] RISC-V: KVM: Validate SBI STA shmem alignment
Posted by Jiakai Xu 4 weeks, 1 day ago
This series fixes a missing validation in the RISC-V KVM SBI
steal-time accounting (STA) register handling.

Patch 1 validates the configured SBI STA shared memory GPA at
KVM_SET_ONE_REG, enforcing the 64-byte alignment requirement
defined by the SBI specification or allowing INVALID_GPA to explicitly
disable steal-time accounting. This prevents invalid userspace state
from reaching runtime code paths and avoids WARN_ON() triggers during
KVM_RUN.

Patch 2 refactors existing UAPI tests from steal_time_init() into
a dedicated check_steal_time_uapi() function for better code organization,
and adds an empty stub for RISC-V.

Patch 3 fills in the RISC-V stub with tests that verify KVM correctly
enforces the STA alignment requirements.

Jiakai Xu (3):
  RISC-V: KVM: Validate SBI STA shmem alignment in 
    kvm_sbi_ext_sta_set_reg()
  KVM: selftests: Refactor UAPI tests into dedicated function
  RISC-V: KVM: selftests: Add RISC-V SBI STA shmem alignment tests

 arch/riscv/kvm/vcpu_sbi_sta.c                 | 16 ++-
 .../selftests/kvm/include/kvm_util_types.h    |  2 +
 tools/testing/selftests/kvm/steal_time.c      | 98 ++++++++++++++++---
 3 files changed, 95 insertions(+), 21 deletions(-)

-- 
2.34.1
Re: [PATCH v10 0/3] RISC-V: KVM: Validate SBI STA shmem alignment
Posted by Anup Patel 5 days, 9 hours ago
On Tue, Mar 3, 2026 at 6:39 AM Jiakai Xu <xujiakai2025@iscas.ac.cn> wrote:
>
> This series fixes a missing validation in the RISC-V KVM SBI
> steal-time accounting (STA) register handling.
>
> Patch 1 validates the configured SBI STA shared memory GPA at
> KVM_SET_ONE_REG, enforcing the 64-byte alignment requirement
> defined by the SBI specification or allowing INVALID_GPA to explicitly
> disable steal-time accounting. This prevents invalid userspace state
> from reaching runtime code paths and avoids WARN_ON() triggers during
> KVM_RUN.
>
> Patch 2 refactors existing UAPI tests from steal_time_init() into
> a dedicated check_steal_time_uapi() function for better code organization,
> and adds an empty stub for RISC-V.
>
> Patch 3 fills in the RISC-V stub with tests that verify KVM correctly
> enforces the STA alignment requirements.
>
> Jiakai Xu (3):
>   RISC-V: KVM: Validate SBI STA shmem alignment in
>     kvm_sbi_ext_sta_set_reg()
>   KVM: selftests: Refactor UAPI tests into dedicated function
>   RISC-V: KVM: selftests: Add RISC-V SBI STA shmem alignment tests
>
>  arch/riscv/kvm/vcpu_sbi_sta.c                 | 16 ++-
>  .../selftests/kvm/include/kvm_util_types.h    |  2 +
>  tools/testing/selftests/kvm/steal_time.c      | 98 ++++++++++++++++---
>  3 files changed, 95 insertions(+), 21 deletions(-)
>
> --
> 2.34.1
>

Queued this series for Linux-7.1

Thanks,
Anup
Re: [PATCH v10 0/3] RISC-V: KVM: Validate SBI STA shmem alignment
Posted by Jiakai Xu 2 weeks, 2 days ago
Hi,

Just a gentle ping on this series.
Is there anything else needed before it can be applied?

Thanks,
Jiakai