Re: [PATCH v3 0/4] KVM: arm64: selftests: arch_timer_edge_cases fixes

[Miguel Luis <miguel.luis@oracle.com>]

Hi Sebastian,

Thanks in advance.

> On 5 Jun 2025, at 10:36, Sebastian Ott <sebott@redhat.com> wrote:
> 
> Some small fixes for arch_timer_edge_cases that I stumbled upon
> while debugging failures for this selftest on ampere-one.
> 

I’ve tested this test under three circumstances:

1) Original (!HAS_EL2 && !HAS_EL2_E2H0)

2) HAS_EL2 && HAS_EL2_E2H0

3) HAS_EL2 && !HAS_EL2_E2H0


Tests 1) and 2) returned in approx. the same amount of real time (about 12s) although 3) doesn’t
seem to return at all.

I’ve tested it on ampere-one.

Miguel

> Changes since v1:
> * determine effective counter width based on suggestions from Marc
> Changes since v2:
> * new patch to fix xval initialization
> 
> I've done tests with this on various machines - no issues during
> several hundreds of test runs.
> 
> v1: https://lore.kernel.org/kvmarm/20250509143312.34224-1-sebott@redhat.com/
> v2: https://lore.kernel.org/kvmarm/20250527142434.25209-1-sebott@redhat.com/
> 
> Sebastian Ott (4):
>  KVM: arm64: selftests: fix help text for arch_timer_edge_cases
>  KVM: arm64: selftests: fix thread migration in arch_timer_edge_cases
>  KVM: arm64: selftests: arch_timer_edge_cases - fix xval init
>  KVM: arm64: selftests: arch_timer_edge_cases - determine effective counter width
> 
> .../kvm/arm64/arch_timer_edge_cases.c         | 39 ++++++++++++-------
> 1 file changed, 25 insertions(+), 14 deletions(-)
> 
> 
> base-commit: 0ff41df1cb268fc69e703a08a57ee14ae967d0ca
> -- 
> 2.49.0
> 
>