Re: [kvm-unit-tests PATCH v2 0/6] powerpc improvements

On 7/20/26 15:00, Thomas Huth wrote:

On 19/07/2026 15.00, Chinmay Rath wrote:

quoted

This series aims to add a couple of new powerpc tests and improve the
powerpc build structure.

This is originally Nick's work. The last version (v10) of which can 
be seen here :
https://lore.kernel.org/linuxppc-dev/20240612052322.218726-1-npiggin@gmail.com/ (local) 

Couple of patches from the series are already merged.

Since it has been 2 years since this series was posted, I originally
posted this as a RFC. This v1 series has changes accomodating the 
review comments
received in the RFC :
https://lore.kernel.org/kvm/20260602064806.3101025-1-rathc@linux.ibm.com/ (local) 

v1:
https://lore.kernel.org/kvm/20260708105122.1177210-1-rathc@linux.ibm.com/ (local) 


I have rebased the patches to the upstream state.
For the patches which did not require any changes in the existing 
upstream code,
I have retained the "Reviewed-by"s and "Acked-by" (patch 3,5 & 6) from
Nick's series. I have also added Andrew's "Reviewed-by" received on
patch 2 in the RFC. Nick had originally introduced powernv tests as 
well.
Removed it for now since it was causing CI failure in the current 
upstream.

Link to Gitlab tree with patches :
https://gitlab.com/rathc/kvm-unit-tests/-/tree/ppc64
Link to Gitlab pipeline :
https://gitlab.com/rathc/kvm-unit-tests/-/pipelines/2688519679

Change log from RFC to v1:
1. Patch 2: Added Andrew's "Reviewed-by".
2. Patch 4: Removed usage of gitlab-ci group to run test - Thomas
    It would be nice to make use of the gitlab-ci group, but doing the
    change of all archs would make sense in a separate series. I look
    forward to doing that.
3. Patch 6: Removed addition of panic test in gitlab-ci group - Thomas

There was a discussion on the build failing in Travis CI with Clang, 
in the
RFC, which I have addressed there.

Change log from v1 to v2:
Patch 4: Addressed all comments by Thomas :
1. Removed extra blank line.
2. Removed accel=kvm test from gitlab-ci.
3. smp = 2 preserved in atmoics test.

Thanks, looks fine now, so I pushed it to the repository.

 Thomas

Thank you Thomas !