From: Nico Boehr <nrb@linux.ibm.com>
To: kvm@vger.kernel.org, linux-s390@vger.kernel.org
Cc: frankja@linux.ibm.com, imbrenda@linux.ibm.com, thuth@redhat.com,
david@redhat.com
Subject: [PATCH kvm-unit-tests v1 6/8] s390x: Add more tests for STSCH
Date: Fri, 21 Jan 2022 16:09:29 +0100 [thread overview]
Message-ID: <20220121150931.371720-7-nrb@linux.ibm.com> (raw)
In-Reply-To: <20220121150931.371720-1-nrb@linux.ibm.com>
css_lib extensively uses STSCH, but two more cases deserve their own
tests:
- unaligned address for SCHIB. We check for misalignment by 1 and 2
bytes.
- channel not operational
- bit 47 in SID not set
- bit 5 of PMCW flags.
As per the principles of operation, bit 5 of the PMCW flags shall be
ignored by msch and always stored as zero by stsch.
Currently, QEMU requires this bit to always be zero on msch, which is
why this test currently fails. A fix was posted upstream
("[PATCH qemu] s390x/css: fix PMCW invalid mask").
Here's the QEMU PMCW invalid mask fix: https://lists.nongnu.org/archive/html/qemu-s390x/2021-12/msg00100.html
Signed-off-by: Nico Boehr <nrb@linux.ibm.com>
---
s390x/css.c | 74 +++++++++++++++++++++++++++++++++++++++++++++++++++++
1 file changed, 74 insertions(+)
diff --git a/s390x/css.c b/s390x/css.c
index aa3c22eb91bd..7a0a9ddc28ce 100644
--- a/s390x/css.c
+++ b/s390x/css.c
@@ -494,6 +494,78 @@ static void test_ssch(void)
report_prefix_pop();
}
+static void test_stsch(void)
+{
+ const int align_to = 4;
+ struct schib schib;
+ int cc;
+
+ if (!test_device_sid) {
+ report_skip("No device");
+ return;
+ }
+
+ report_prefix_push("Unaligned");
+ for (int i = 1; i < align_to; i *= 2) {
+ report_prefix_pushf("%d", i);
+
+ expect_pgm_int();
+ stsch(test_device_sid, (struct schib *)(alignment_test_page + i));
+ check_pgm_int_code(PGM_INT_CODE_SPECIFICATION);
+
+ report_prefix_pop();
+ }
+ report_prefix_pop();
+
+ report_prefix_push("Invalid subchannel number");
+ cc = stsch(0x0001ffff, &schib);
+ report(cc == 3, "Channel not operational");
+ report_prefix_pop();
+
+ report_prefix_push("Bit 47 in SID is zero");
+ expect_pgm_int();
+ stsch(0x0000ffff, &schib);
+ check_pgm_int_code(PGM_INT_CODE_OPERAND);
+ report_prefix_pop();
+}
+
+static void test_pmcw_bit5(void)
+{
+ int cc;
+ uint16_t old_pmcw_flags;
+
+ cc = stsch(test_device_sid, &schib);
+ if (cc) {
+ report_fail("stsch: sch %08x failed with cc=%d", test_device_sid, cc);
+ return;
+ }
+ old_pmcw_flags = schib.pmcw.flags;
+
+ report_prefix_push("Bit 5 set");
+
+ schib.pmcw.flags = old_pmcw_flags | BIT(15 - 5);
+ cc = msch(test_device_sid, &schib);
+ report(!cc, "MSCH cc == 0");
+
+ cc = stsch(test_device_sid, &schib);
+ report(!cc, "STSCH cc == 0");
+ report(!(schib.pmcw.flags & BIT(15 - 5)), "STSCH PMCW Bit 5 is clear");
+
+ report_prefix_pop();
+
+ report_prefix_push("Bit 5 clear");
+
+ schib.pmcw.flags = old_pmcw_flags & ~BIT(15 - 5);
+ cc = msch(test_device_sid, &schib);
+ report(!cc, "MSCH cc == 0");
+
+ cc = stsch(test_device_sid, &schib);
+ report(!cc, "STSCH cc == 0");
+ report(!(schib.pmcw.flags & BIT(15 - 5)), "STSCH PMCW Bit 5 is clear");
+
+ report_prefix_pop();
+}
+
static struct {
const char *name;
void (*func)(void);
@@ -509,6 +581,8 @@ static struct {
{ "msch", test_msch },
{ "stcrw", test_stcrw },
{ "ssch", test_ssch },
+ { "stsch", test_stsch },
+ { "pmcw bit 5 ignored", test_pmcw_bit5 },
{ NULL, NULL }
};
--
2.31.1
next prev parent reply other threads:[~2022-01-21 15:10 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-01-21 15:09 [PATCH kvm-unit-tests v1 0/8] s390x: Extend instruction interception tests Nico Boehr
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 1/8] s390x: Add more tests for MSCH Nico Boehr
2022-01-25 11:54 ` Thomas Huth
2022-01-25 12:57 ` Nico Boehr
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 2/8] s390x: Add test for PFMF low-address protection Nico Boehr
2022-01-25 13:00 ` Thomas Huth
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 3/8] s390x: Add sck tests Nico Boehr
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 4/8] s390x: Add tests for STCRW Nico Boehr
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 5/8] s390x: Add more tests for SSCH Nico Boehr
2022-01-21 15:09 ` Nico Boehr [this message]
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 7/8] s390x: Add tests for TSCH Nico Boehr
2022-01-21 15:09 ` [PATCH kvm-unit-tests v1 8/8] s390x: Add EPSW test Nico Boehr
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20220121150931.371720-7-nrb@linux.ibm.com \
--to=nrb@linux.ibm.com \
--cc=david@redhat.com \
--cc=frankja@linux.ibm.com \
--cc=imbrenda@linux.ibm.com \
--cc=kvm@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
--cc=thuth@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).