Re: [kvm-unit-tests PATCH v2 5/7] s390x: sie: Add first SIE test

Janosch Frank
2020-11-30T12:43:33Z

On 11/30/20 11:45 AM, Cornelia Huck wrote:
> On Fri, 27 Nov 2020 08:06:27 -0500
> Janosch Frank <frankja@linux.ibm.com> wrote:
>
>> Let's check if we get the correct interception data on a few
>> diags. This commit is more of an addition of boilerplate code than a
>> real test.
>>
>> Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
>> ---
>> s390x/Makefile | 1 +
>> s390x/sie.c | 125 ++++++++++++++++++++++++++++++++++++++++++++
>> s390x/unittests.cfg | 3 ++
>> 3 files changed, 129 insertions(+)
>> create mode 100644 s390x/sie.c
>>
>
> (...)
>
>> +static void sie(struct vm *vm)
>> +{
>> + while (vm->sblk->icptcode == 0) {
>> + sie64a(vm->sblk, &vm->save_area);
>> + if (vm->sblk->icptcode == 32)
>
> Can you maybe add #defines for the intercept codes you're checking for?

Sure ICTP_VALIDITY and ICPT_INSTRUCTION would make sense.

>
>> + handle_validity(vm);
>> + }
>> + vm->save_area.guest.grs[14] = vm->sblk->gg14;
>> + vm->save_area.guest.grs[15] = vm->sblk->gg15;
>> +}
>> +
>> +static void sblk_cleanup(struct vm *vm)
>> +{
>> + vm->sblk->icptcode = 0;
>> +}
>> +
>> +static void intercept_diag_10(void)
>> +{
>> + u32 instr = 0x83020010;
>> +
>> + vm.sblk->gpsw.addr = PAGE_SIZE * 2;
>> + vm.sblk->gpsw.mask = 0x0000000180000000ULL;
>> +
>> + memset(guest_instr, 0, PAGE_SIZE);
>> + memcpy(guest_instr, &instr, 4);
>> + sie(&vm);
>> + report(vm.sblk->icptcode == 4 && vm.sblk->ipa == 0x8302 && vm.sblk->ipb == 0x100000,
>
> Again, some #defines might help here, making clear that 0x8302 means
> diag. (The ipb value is clear enough :) Maybe you can also assemble
> instr out of pre-made pieces? Or factor out some code to a common
> function?

Yes, a diag test function that has the code as a parameter would make
this look nicer and I could also test ipa against the first 16bits of instr.

>
>> + "Diag 10 intercept");
>> + sblk_cleanup(&vm);
>> +}
>> +
>> +static void intercept_diag_44(void)
>> +{
>> + u32 instr = 0x83020044;
>> +
>> + vm.sblk->gpsw.addr = PAGE_SIZE * 2;
>> + vm.sblk->gpsw.mask = 0x0000000180000000ULL;
>> +
>> + memset(guest_instr, 0, PAGE_SIZE);
>> + memcpy(guest_instr, &instr, 4);
>> + sie(&vm);
>> + report(vm.sblk->icptcode == 4 && vm.sblk->ipa == 0x8302 && vm.sblk->ipb == 0x440000,
>> + "Diag 44 intercept");
>> + sblk_cleanup(&vm);
>> +}
>> +
>> +static void intercept_diag_9c(void)
>> +{
>> + u32 instr = 0x8302009c;
>> +
>> + vm.sblk->gpsw.addr = PAGE_SIZE * 2;
>> + vm.sblk->gpsw.mask = 0x0000000180000000ULL;
>> +
>> + memset(guest_instr, 0, PAGE_SIZE);
>> + memcpy(guest_instr, &instr, 4);
>> + sie(&vm);
>> + report(vm.sblk->icptcode == 4 && vm.sblk->ipa == 0x8302 && vm.sblk->ipb == 0x9c0000,
>> + "Diag 9c intercept");
>> + sblk_cleanup(&vm);
>> +}
>
> (...)
>