Re: [PATCH v7 0/8] KVM: selftests: add powerpc support

[Madhavan Srinivasan <maddy@linux.ibm.com>]

On 10/6/26 3:58 PM, Ritesh Harjani (IBM) wrote:
> Patch v5 -> v7:
> ================
> (Suggestions from Sean):
> 1. Added kvm_is_ppc_pseries_lpar() helper in kvm_util.h to avoid sprinkling
>     multiple #ifdef __powerpc64__ checks.
> 2. Enable all tests via TEST_GEN_PROGS_COMMON for powerpc
> 3. Enable hardware_disable_test and skip when running on powerpc pseries LPAR
>     (nested).
> 4. Re-arranged the fixes in appropriate order.
>
> This is based off v2 of Sean PPC pre-enablement series [1].
> [1]: https://lore.kernel.org/kvm/20260902164123.2546762-1-seanjc@google.com/
>
> Patch v4 -> v5:
> ==============
> - Fixed the issues reported by Anushree while running on kvm on powervm
>    (nested).
> - Fixed memslot_perf_test on powerpc.
>
> PATCH v3 -> PATCH v4:
> ==================
> - Rebased on top of Sean's new page allocator patch series v2 [1]
>    This allowed to drop and patch-2 since it will now come from Sean's
>    v2.
> - Addressed review comments from Sean - squashed patches 6-10 in patch-1 of
>    this.
> - Dropped patch-5 to print vcpu_id when KVM_CREATE_VCPU ioctl fails.
> - Added additional patch-3 to this series to allow dirty_log_test to run on all
>    big-endian machines.
>
> [1]: https://lore.kernel.org/kvm/20260902164123.2546762-1-seanjc@google.com/
> [v3]: https://lore.kernel.org/linuxppc-dev/2d2c46be7f051c7f1c645afaec67f57067d37a81.1781093720.git.ritesh.list@gmail.com/
>
> Testing updates:
> ================
> I have done some fair amount of testing of the selftests both on real HW and
> using various combinations on qemu (powernv+LE (default), pseries+cap-nested-hv,
> pseries+cap-nested-papr, powernv+BE). There were few powerpc kvm kernel fixes
> which were identified as part of this work. I will send those patches separately
> as those anyway needs to go in separately.
>
> <pasted snippet of cover letter from previous versions>
> =======================================================
>
> Hi All,
>
> This series primarly adds KVM selftests support for powerpc (64-bit, BookS,
> radix MMU).
>
> This patch series is originally Nick's work. I have mainly only rebased it on
> the latest upstream tree. Since the rebase required few changes to all the four
> patches (Patch 1-4), I have dropped the earlier Acked-by from Michael Ellerman.
>
> Since the last series was posted three years ago [1], I am resetting the version
> to RFC. This rebase was done as part of a larger effort to improve the selftests
> infrastructure for Linux on PowerPC tree. Thanks to Harsh and Maddy for their
> help on this.
>
Sean/Paolo, hope you take this via your tree.
Acked-by: Madhavan Srinivasan <maddy@linux.ibm.com>
> Nicholas Piggin (1):
>    KVM: PPC: selftests: add support for powerpc
>
> Ritesh Harjani (IBM) (7):
>    KVM: selftests: Don't limit LE dirty-bitmap bitops to s390x
>    KVM: selftests: Move memslot_perf_test off the 256M ELF load address
>    KVM: selftests: Skip the max vCPU ID test when KVM_CAP_PPC_SMT is set
>    KVM: selftests: Add helper for checking if running on powerpc pseries LPAR
>    KVM: selftests: Skip kvm_create_max_vcpus on powerpc pseries LPAR
>    KVM: selftests: Only warn for idle-page checks on powerpc pseries LPAR
>    KVM: selftests: Skip hardware_disable_test on powerpc pseries LPAR
>
>   MAINTAINERS                                   |   2 +
>   tools/testing/selftests/kvm/Makefile          |   2 +-
>   tools/testing/selftests/kvm/Makefile.kvm      |   9 +
>   .../selftests/kvm/access_tracking_perf_test.c |  10 +
>   tools/testing/selftests/kvm/dirty_log_test.c  |  12 +-
>   .../selftests/kvm/hardware_disable_test.c     |   3 +
>   .../testing/selftests/kvm/include/kvm_util.h  |  26 +
>   .../selftests/kvm/include/powerpc/hcall.h     |  17 +
>   .../kvm/include/powerpc/kvm_util_arch.h       |  22 +
>   .../selftests/kvm/include/powerpc/ppc_asm.h   |  32 ++
>   .../selftests/kvm/include/powerpc/processor.h |  39 ++
>   .../selftests/kvm/include/powerpc/ucall.h     |  21 +
>   .../selftests/kvm/kvm_create_max_vcpus.c      |  14 +-
>   tools/testing/selftests/kvm/lib/guest_modes.c |  20 +-
>   tools/testing/selftests/kvm/lib/kvm_util.c    |   8 +
>   .../selftests/kvm/lib/powerpc/handlers.S      |  93 ++++
>   .../testing/selftests/kvm/lib/powerpc/hcall.c |  45 ++
>   .../selftests/kvm/lib/powerpc/processor.c     | 512 ++++++++++++++++++
>   .../testing/selftests/kvm/lib/powerpc/ucall.c |  22 +
>   .../testing/selftests/kvm/memslot_perf_test.c |   2 +-
>   20 files changed, 898 insertions(+), 13 deletions(-)
>   create mode 100644 tools/testing/selftests/kvm/include/powerpc/hcall.h
>   create mode 100644 tools/testing/selftests/kvm/include/powerpc/kvm_util_arch.h
>   create mode 100644 tools/testing/selftests/kvm/include/powerpc/ppc_asm.h
>   create mode 100644 tools/testing/selftests/kvm/include/powerpc/processor.h
>   create mode 100644 tools/testing/selftests/kvm/include/powerpc/ucall.h
>   create mode 100644 tools/testing/selftests/kvm/lib/powerpc/handlers.S
>   create mode 100644 tools/testing/selftests/kvm/lib/powerpc/hcall.c
>   create mode 100644 tools/testing/selftests/kvm/lib/powerpc/processor.c
>   create mode 100644 tools/testing/selftests/kvm/lib/powerpc/ucall.c
>
> --
> 2.39.5
>
>