On Thu, 8 Jun 2023 at 07:58, Nicholas Piggin <npiggin@xxxxxxxxx> wrote: > > Posting again, a couple of patches were merged and accounted for review > comments from last time. I saw some failures in the spr tests running on a power9 powernv system: $ TESTNAME=sprs TIMEOUT=90s ACCEL= ./powerpc/run powerpc/sprs.elf -smp 1 |grep FAIL FAIL: WORT ( 895): 0x00000000c0deba80 <==> 0x0000000000000000 $ MIGRATION=yes TESTNAME=sprs-migration TIMEOUT=90s ACCEL= ./powerpc/run powerpc/sprs.elf -smp 1 -append '-w' | grep FAIL FAIL: SRR0 ( 26): 0xcafefacec0debabc <==> 0x0000000000402244 FAIL: SRR1 ( 27): 0xc0000006409ebab6 <==> 0x8000000000001001 FAIL: CTRL ( 136): 0x00000000 <==> 0x00008001 FAIL: WORT ( 895): 0x00000000c0deba80 <==> 0x0000000000000000 FAIL: PIR (1023): 0x00000010 <==> 0x00000049 Linux 6.2.0-20-generic QEMU emulator version 7.2.0 (Debian 1:7.2+dfsg-5ubuntu2) On a power8 powernv: MIGRATION=yes TESTNAME=sprs-migration TIMEOUT=90s ACCEL= ./powerpc/run powerpc/sprs.elf -smp 1 -append '-w' |grep FAIL FAIL: SRR0 ( 26): 0xcafefacec0debabc <==> 0x0000000000402234 FAIL: SRR1 ( 27): 0xc0000006409ebab6 <==> 0x8000000000001000 FAIL: CTRL ( 136): 0x00000000 <==> 0x00008001 FAIL: PIR (1023): 0x00000060 <==> 0x00000030 Linux 5.4.0-146-generic QEMU emulator version 4.2.1 (Debian 1:4.2-3ubuntu6.26) Cheers, Joel > > Thanks, > Nick > > Nicholas Piggin (12): > powerpc: Report instruction address and MSR in unhandled exception > error > powerpc: Add some checking to exception handler install > powerpc: Abstract H_CEDE calls into a sleep functions > powerpc: Add ISA v3.1 (POWER10) support to SPR test > powerpc: Extract some common helpers and defines to headers > powerpc/sprs: Specify SPRs with data rather than code > powerpc/spapr_vpa: Add basic VPA tests > powerpc: Expand exception handler vector granularity > powerpc: Add support for more interrupts including HV interrupts > powerpc: Discover runtime load address dynamically > powerpc: Support powernv machine with QEMU TCG > powerpc/sprs: Test hypervisor registers on powernv machine > > lib/powerpc/asm/handlers.h | 2 +- > lib/powerpc/asm/hcall.h | 1 + > lib/powerpc/asm/ppc_asm.h | 9 + > lib/powerpc/asm/processor.h | 55 ++- > lib/powerpc/handlers.c | 10 +- > lib/powerpc/hcall.c | 4 +- > lib/powerpc/io.c | 27 +- > lib/powerpc/io.h | 6 + > lib/powerpc/processor.c | 79 ++++- > lib/powerpc/setup.c | 8 +- > lib/ppc64/asm/opal.h | 15 + > lib/ppc64/asm/vpa.h | 62 ++++ > lib/ppc64/opal-calls.S | 46 +++ > lib/ppc64/opal.c | 74 +++++ > powerpc/Makefile.ppc64 | 4 +- > powerpc/cstart64.S | 105 ++++-- > powerpc/run | 35 +- > powerpc/spapr_hcall.c | 9 +- > powerpc/spapr_vpa.c | 172 ++++++++++ > powerpc/sprs.c | 645 ++++++++++++++++++++++++++---------- > powerpc/tm.c | 20 +- > powerpc/unittests.cfg | 3 + > 22 files changed, 1133 insertions(+), 258 deletions(-) > create mode 100644 lib/ppc64/asm/opal.h > create mode 100644 lib/ppc64/asm/vpa.h > create mode 100644 lib/ppc64/opal-calls.S > create mode 100644 lib/ppc64/opal.c > create mode 100644 powerpc/spapr_vpa.c > > -- > 2.40.1 >