Re: [kvm-unit-tests PATCH v8 11/35] powerpc/sprs: Specify SPRs with data rather than code

Thomas Huth
2024-04-16T09:48:51Z

On 05/04/2024 10.35, Nicholas Piggin wrote:
> A significant rework that builds an array of 'struct spr', where each
> element describes an SPR. This makes various metadata about the SPR
> like name and access type easier to carry and use.
>
> Hypervisor privileged registers are described despite not being used
> at the moment for completeness, but also the code might one day be
> reused for a hypervisor-privileged test.
>
> Acked-by: Thomas Huth <thuth@redhat.com>
> Signed-off-by: Nicholas Piggin <npiggin@gmail.com>
> ---
> lib/powerpc/asm/reg.h | 2 +
> powerpc/sprs.c | 647 +++++++++++++++++++++++++++++-------------
> 2 files changed, 457 insertions(+), 192 deletions(-)

I tried to merge this patch, but it seems to break running the tests on
Ubuntu Focal with Clang 11 in the Travis-CI:

https://app.travis-ci.com/github/huth/kvm-unit-tests/jobs/620577246

Could you please have a look?

Thanks,
Thomas