sie_is_diag_icpt() checks if the intercept is due to an expected
diagnose call and is valid.
It subsumes pv_icptdata_check_diag.
Signed-off-by: Nina Schoetterl-Glausch <nsg@xxxxxxxxxxxxx>
---
s390x/Makefile | 1 +
lib/s390x/pv_icptdata.h | 42 -----------------------------
lib/s390x/sie-icpt.h | 39 +++++++++++++++++++++++++++
lib/s390x/sie-icpt.c | 60 +++++++++++++++++++++++++++++++++++++++++
s390x/pv-diags.c | 9 +++----
s390x/pv-icptcode.c | 12 ++++-----
s390x/pv-ipl.c | 8 +++---
7 files changed, 114 insertions(+), 57 deletions(-)
delete mode 100644 lib/s390x/pv_icptdata.h
create mode 100644 lib/s390x/sie-icpt.h
create mode 100644 lib/s390x/sie-icpt.c
diff --git a/s390x/Makefile b/s390x/Makefile
index 23342bd6..0ad8d021 100644
--- a/s390x/Makefile
+++ b/s390x/Makefile
@@ -111,6 +111,7 @@ cflatobjs += lib/s390x/css_lib.o
cflatobjs += lib/s390x/malloc_io.o
cflatobjs += lib/s390x/uv.o
cflatobjs += lib/s390x/sie.o
+cflatobjs += lib/s390x/sie-icpt.o
cflatobjs += lib/s390x/fault.o
OBJDIRS += lib/s390x
diff --git a/lib/s390x/pv_icptdata.h b/lib/s390x/pv_icptdata.h
deleted file mode 100644
index 4746117e..00000000
--- a/lib/s390x/pv_icptdata.h
+++ /dev/null
@@ -1,42 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-only */
-/*
- * Commonly used checks for PV SIE intercept data
- *
- * Copyright IBM Corp. 2023
- * Author: Janosch Frank <frankja@xxxxxxxxxxxxx>
- */
-
-#ifndef _S390X_PV_ICPTDATA_H_
-#define _S390X_PV_ICPTDATA_H_
-
-#include <sie.h>
-
-/*
- * Checks the diagnose instruction intercept data for consistency with
- * the constants defined by the PV SIE architecture
- *
- * Supports: 0x44, 0x9c, 0x288, 0x308, 0x500
- */
-static bool pv_icptdata_check_diag(struct vm *vm, int diag)
-{
- int icptcode;
-
- switch (diag) {
- case 0x44:
- case 0x9c:
- case 0x288:
- case 0x308:
- icptcode = ICPT_PV_NOTIFY;
- break;
- case 0x500:
- icptcode = ICPT_PV_INSTR;
- break;
- default:
- /* If a new diag is introduced add it to the cases above! */
- assert(0);
- }
-
- return vm->sblk->icptcode == icptcode && vm->sblk->ipa == 0x8302 &&
- vm->sblk->ipb == 0x50000000 && vm->save_area.guest.grs[5] == diag;
-}
-#endif
diff --git a/lib/s390x/sie-icpt.h b/lib/s390x/sie-icpt.h
new file mode 100644
index 00000000..604a7221
--- /dev/null
+++ b/lib/s390x/sie-icpt.h
@@ -0,0 +1,39 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/*
+ * Functionality for SIE interception handling.
+ *
+ * Copyright IBM Corp. 2024
+ */
+
+#ifndef _S390X_SIE_ICPT_H_
+#define _S390X_SIE_ICPT_H_
+
+#include <libcflat.h>
+#include <sie.h>
+
+struct diag_itext {
+ uint64_t opcode : 8;
+ uint64_t r_1 : 4;
+ uint64_t r_2 : 4;
+ uint64_t r_base : 4;
+ uint64_t displace : 12;
+ uint64_t zero : 16;
+ uint64_t : 16;
+};
+
+struct diag_itext sblk_ip_as_diag(struct kvm_s390_sie_block *sblk);
+
+/**
+ * sie_is_diag_icpt() - Check if intercept is due to diagnose instruction
+ * @vm: the guest
+ * @diag: the expected diagnose code
+ *
+ * Check that the intercept is due to diagnose @diag and valid.
+ * For protected virtualization, check that the intercept data meets additional
+ * constraints.
+ *
+ * Returns: true if intercept is due to a valid and has matching diagnose code
+ */
+bool sie_is_diag_icpt(struct vm *vm, unsigned int diag);
+
+#endif /* _S390X_SIE_ICPT_H_ */
diff --git a/lib/s390x/sie-icpt.c b/lib/s390x/sie-icpt.c
new file mode 100644
index 00000000..17064424
--- /dev/null
+++ b/lib/s390x/sie-icpt.c
@@ -0,0 +1,60 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+/*
+ * Functionality for SIE interception handling.
+ *
+ * Copyright IBM Corp. 2024
+ */
+
+#include <sie-icpt.h>
+
+struct diag_itext sblk_ip_as_diag(struct kvm_s390_sie_block *sblk)
+{
+ union {
+ struct {
+ uint64_t ipa : 16;
+ uint64_t ipb : 32;
+ uint64_t : 16;
+ };
+ struct diag_itext diag;
+ } instr = { .ipa = sblk->ipa, .ipb = sblk->ipb };
+
+ return instr.diag;
+}
+
+bool sie_is_diag_icpt(struct vm *vm, unsigned int diag)
+{
+ struct diag_itext instr = sblk_ip_as_diag(vm->sblk);
+ uint8_t icptcode;
+ uint64_t code;
+
+ switch (diag) {
+ case 0x44:
+ case 0x9c:
+ case 0x288:
+ case 0x308:
+ icptcode = ICPT_PV_NOTIFY;
+ break;
+ case 0x500:
+ icptcode = ICPT_PV_INSTR;
+ break;
+ default:
+ /* If a new diag is introduced add it to the cases above! */
+ assert_msg(false, "unknown diag 0x%x", diag);
+ }
+
+ if (sie_is_pv(vm)) {
+ if (instr.r_1 != 0 || instr.r_2 != 2 || instr.r_base != 5)
+ return false;
+ if (instr.displace)
+ return false;
+ } else {
+ icptcode = ICPT_INST;
+ }
+ if (vm->sblk->icptcode != icptcode)
+ return false;
+ if (instr.opcode != 0x83 || instr.zero)
+ return false;
+ code = instr.r_base ? vm->save_area.guest.grs[instr.r_base] : 0;
+ code = (code + instr.displace) & 0xffff;
+ return code == diag;
+}
diff --git a/s390x/pv-diags.c b/s390x/pv-diags.c
index 3193ad99..09b83d59 100644
--- a/s390x/pv-diags.c
+++ b/s390x/pv-diags.c
@@ -9,7 +9,7 @@
*/
#include <libcflat.h>
#include <snippet.h>
-#include <pv_icptdata.h>
+#include <sie-icpt.h>
#include <sie.h>
#include <sclp.h>
#include <asm/facility.h>
@@ -32,8 +32,7 @@ static void test_diag_500(void)
size_gbin, size_hdr, SNIPPET_UNPACK_OFF);
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x500),
- "intercept values");
+ report(sie_is_diag_icpt(&vm, 0x500), "intercept values");
report(vm.save_area.guest.grs[1] == 1 &&
vm.save_area.guest.grs[2] == 2 &&
vm.save_area.guest.grs[3] == 3 &&
@@ -45,7 +44,7 @@ static void test_diag_500(void)
*/
vm.sblk->iictl = IICTL_CODE_OPERAND;
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x9c) &&
+ report(sie_is_diag_icpt(&vm, 0x9c) &&
vm.save_area.guest.grs[0] == PGM_INT_CODE_OPERAND,
"operand exception");
@@ -57,7 +56,7 @@ static void test_diag_500(void)
vm.sblk->iictl = IICTL_CODE_SPECIFICATION;
/* Inject PGM, next exit should be 9c */
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x9c) &&
+ report(sie_is_diag_icpt(&vm, 0x9c) &&
vm.save_area.guest.grs[0] == PGM_INT_CODE_SPECIFICATION,
"specification exception");
diff --git a/s390x/pv-icptcode.c b/s390x/pv-icptcode.c
index d7c47d6f..5293306b 100644
--- a/s390x/pv-icptcode.c
+++ b/s390x/pv-icptcode.c
@@ -13,7 +13,7 @@
#include <smp.h>
#include <sclp.h>
#include <snippet.h>
-#include <pv_icptdata.h>
+#include <sie-icpt.h>
#include <asm/facility.h>
#include <asm/barrier.h>
#include <asm/sigp.h>
@@ -47,7 +47,7 @@ static void test_validity_timing(void)
size_gbin, size_hdr, SNIPPET_UNPACK_OFF);
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x44), "spt done");
+ report(sie_is_diag_icpt(&vm, 0x44), "spt done");
stck(&time_exit);
tmp = vm.sblk->cputm;
mb();
@@ -258,7 +258,7 @@ static void test_validity_asce(void)
/* Try if we can still do an entry with the correct asce */
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x44), "re-entry with valid CR1");
+ report(sie_is_diag_icpt(&vm, 0x44), "re-entry with valid CR1");
uv_destroy_guest(&vm);
free_pages(pgd_new);
report_prefix_pop();
@@ -294,7 +294,7 @@ static void run_icpt_122_tests_prefix(unsigned long prefix)
sie(&vm);
/* Guest indicates that it has been setup via the diag 0x44 */
- assert(pv_icptdata_check_diag(&vm, 0x44));
+ assert(sie_is_diag_icpt(&vm, 0x44));
/* If the pages have not been shared these writes will cause exceptions */
ptr = (uint32_t *)prefix;
WRITE_ONCE(ptr, 0);
@@ -328,7 +328,7 @@ static void test_icpt_112(void)
/* Setup of the guest's state for 0x0 prefix */
sie(&vm);
- assert(pv_icptdata_check_diag(&vm, 0x44));
+ assert(sie_is_diag_icpt(&vm, 0x44));
/* Test on standard 0x0 prefix */
run_icpt_122_tests_prefix(0);
@@ -348,7 +348,7 @@ static void test_icpt_112(void)
/* Try a re-entry after everything has been imported again */
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x9c) &&
+ report(sie_is_diag_icpt(&vm, 0x9c) &&
vm.save_area.guest.grs[0] == 42,
"re-entry successful");
report_prefix_pop();
diff --git a/s390x/pv-ipl.c b/s390x/pv-ipl.c
index cc46e7f7..61a1e0c0 100644
--- a/s390x/pv-ipl.c
+++ b/s390x/pv-ipl.c
@@ -11,7 +11,7 @@
#include <sie.h>
#include <sclp.h>
#include <snippet.h>
-#include <pv_icptdata.h>
+#include <sie-icpt.h>
#include <asm/facility.h>
#include <asm/uv.h>
@@ -35,7 +35,7 @@ static void test_diag_308(int subcode)
/* First exit is a diag 0x500 */
sie(&vm);
- assert(pv_icptdata_check_diag(&vm, 0x500));
+ assert(sie_is_diag_icpt(&vm, 0x500));
/*
* The snippet asked us for the subcode and we answer by
@@ -46,7 +46,7 @@ static void test_diag_308(int subcode)
/* Continue after diag 0x500, next icpt should be the 0x308 */
sie(&vm);
- assert(pv_icptdata_check_diag(&vm, 0x308));
+ assert(sie_is_diag_icpt(&vm, 0x308));
assert(vm.save_area.guest.grs[2] == subcode);
/*
@@ -118,7 +118,7 @@ static void test_diag_308(int subcode)
* see a diagnose 0x9c PV instruction notification.
*/
sie(&vm);
- report(pv_icptdata_check_diag(&vm, 0x9c) &&
+ report(sie_is_diag_icpt(&vm, 0x9c) &&
vm.save_area.guest.grs[0] == 42,
"continue after load");
--
2.44.0
