All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Hildenbrand <david@redhat.com>
To: kvm@vger.kernel.org
Cc: Paolo Bonzini <pbonzini@redhat.com>,
	Thomas Huth <thuth@redhat.com>,
	Janosch Frank <frankja@linux.ibm.com>,
	linux-s390@vger.kernel.org, Cornelia Huck <cohuck@redhat.com>,
	Christian Borntraeger <borntraeger@de.ibm.com>,
	David Hildenbrand <david@redhat.com>
Subject: [kvm-unit-tests PULL 09/17] s390x: smp: Test all CRs on initial reset
Date: Thu, 30 Apr 2020 17:24:22 +0200	[thread overview]
Message-ID: <20200430152430.40349-10-david@redhat.com> (raw)
In-Reply-To: <20200430152430.40349-1-david@redhat.com>

From: Janosch Frank <frankja@linux.ibm.com>

All CRs are set to 0 and CRs 0 and 14 are set to pre-defined values,
so we also need to test 1-13 and 15 for 0.

And while we're at it, let's also set some values to cr 1, 7 and 13, so
we can actually be sure that they will be zeroed.

Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
Reviewed-by: Cornelia Huck <cohuck@redhat.com>
Reviewed-by: Christian Borntraeger <borntraeger@de.ibm.com>
Reviewed-by: David Hildenbrand <david@redhat.com>
Message-Id: <20200424093356.11931-1-frankja@linux.ibm.com>
Signed-off-by: David Hildenbrand <david@redhat.com>
---
 s390x/smp.c | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/s390x/smp.c b/s390x/smp.c
index 1641979..08fa770 100644
--- a/s390x/smp.c
+++ b/s390x/smp.c
@@ -195,16 +195,28 @@ static void test_sense_running(void)
 	report_prefix_pop();
 }
 
+/* Used to dirty registers of cpu #1 before it is reset */
+static void test_func_initial(void)
+{
+	lctlg(1, 0x42000UL);
+	lctlg(7, 0x43000UL);
+	lctlg(13, 0x44000UL);
+	set_flag(1);
+}
+
 static void test_reset_initial(void)
 {
 	struct cpu_status *status = alloc_pages(0);
 	struct psw psw;
+	int i;
 
 	psw.mask = extract_psw_mask();
-	psw.addr = (unsigned long)test_func;
+	psw.addr = (unsigned long)test_func_initial;
 
 	report_prefix_push("reset initial");
+	set_flag(0);
 	smp_cpu_start(1, psw);
+	wait_for_flag();
 
 	sigp_retry(1, SIGP_INITIAL_CPU_RESET, 0, NULL);
 	sigp(1, SIGP_STORE_STATUS_AT_ADDRESS, (uintptr_t)status, NULL);
@@ -215,6 +227,10 @@ static void test_reset_initial(void)
 	report(!status->fpc, "fpc");
 	report(!status->cputm, "cpu timer");
 	report(!status->todpr, "todpr");
+	for (i = 1; i <= 13; i++) {
+		report(status->crs[i] == 0, "cr%d == 0", i);
+	}
+	report(status->crs[15] == 0, "cr15 == 0");
 	report_prefix_pop();
 
 	report_prefix_push("initialized");
-- 
2.25.3

  parent reply	other threads:[~2020-04-30 15:25 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-30 15:24 [kvm-unit-tests PULL 00/17] s390x updates David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 01/17] MAINTAINERS: Add Janosch as a s390x maintainer David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 02/17] MAINTAINERS: s390x: add myself as reviewer David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 03/17] MAINTAINERS: s390x: add linux-s390 list David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 04/17] s390x: Add stsi 3.2.2 tests David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 05/17] s390x/smp: fix detection of "running" David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 06/17] s390x: STFLE operates on doublewords David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 07/17] s390x/smp: add minimal test for sigp sense running status David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 08/17] s390x: unittests: Use smp parameter David Hildenbrand
2020-04-30 15:24 ` David Hildenbrand [this message]
2020-04-30 15:24 ` [kvm-unit-tests PULL 10/17] s390x: smp: Dirty fpc before initial reset test David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 11/17] s390x: smp: Test stop and store status on a running and stopped cpu David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 12/17] s390x: smp: Test local interrupts after cpu reset David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 13/17] s390x: smp: Loop if secondary cpu returns into cpu setup again David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 14/17] s390x: smp: Remove unneeded cpu loops David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 15/17] s390x: smp: Use full PSW to bringup new cpu David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 16/17] s390x: smp: Add restart when running test David Hildenbrand
2020-04-30 15:24 ` [kvm-unit-tests PULL 17/17] s390x: Fix library constant definitions David Hildenbrand
2020-05-04 16:44 ` [kvm-unit-tests PULL 00/17] s390x updates Paolo Bonzini

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=20200430152430.40349-10-david@redhat.com \
    --to=david@redhat.com \
    --cc=borntraeger@de.ibm.com \
    --cc=cohuck@redhat.com \
    --cc=frankja@linux.ibm.com \
    --cc=kvm@vger.kernel.org \
    --cc=linux-s390@vger.kernel.org \
    --cc=pbonzini@redhat.com \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.