From: Janosch Frank <frankja@linux.ibm.com>
To: qemu-devel@nongnu.org
Cc: borntraeger@de.ibm.com, thuth@redhat.com, cohuck@redhat.com,
david@redhat.com
Subject: [PATCH v3 4/5] pc-bios: s390x: Save io and external new PSWs before overwriting them
Date: Mon, 31 Aug 2020 11:09:09 -0400 [thread overview]
Message-ID: <20200831150910.317171-5-frankja@linux.ibm.com> (raw)
In-Reply-To: <20200831150910.317171-1-frankja@linux.ibm.com>
Currently we always overwrite the mentioned exception new PSWs before
loading the enabled wait PSW. Let's save the PSW before overwriting
and restore it right before starting the loaded kernel.
Signed-off-by: Janosch Frank <frankja@linux.ibm.com>
---
pc-bios/s390-ccw/jump2ipl.c | 4 +++
pc-bios/s390-ccw/netmain.c | 3 ++
pc-bios/s390-ccw/start.S | 62 +++++++++++++++++++++++++++----------
3 files changed, 52 insertions(+), 17 deletions(-)
diff --git a/pc-bios/s390-ccw/jump2ipl.c b/pc-bios/s390-ccw/jump2ipl.c
index 5b8352d257..bb94ba7550 100644
--- a/pc-bios/s390-ccw/jump2ipl.c
+++ b/pc-bios/s390-ccw/jump2ipl.c
@@ -14,6 +14,7 @@
#define RESET_PSW_MASK (PSW_MASK_SHORTPSW | PSW_MASK_64)
#define RESET_PSW ((uint64_t)&jump_to_IPL_addr | RESET_PSW_MASK)
+extern uint64_t psw_save_io[], psw_save_ext[];
static uint64_t *reset_psw = 0, save_psw, ipl_continue;
void write_reset_psw(uint64_t psw)
@@ -59,6 +60,9 @@ void jump_to_IPL_code(uint64_t address)
/* Ensure the guest output starts fresh */
sclp_print("\n");
+ memcpy(&lowcore->io_new_psw, psw_save_io, 16);
+ memcpy(&lowcore->external_new_psw, psw_save_ext, 16);
+
/*
* HACK ALERT.
* We use the load normal reset to keep r15 unchanged. jump_to_IPL_2
diff --git a/pc-bios/s390-ccw/netmain.c b/pc-bios/s390-ccw/netmain.c
index 056e93a818..74ef28fbc6 100644
--- a/pc-bios/s390-ccw/netmain.c
+++ b/pc-bios/s390-ccw/netmain.c
@@ -32,6 +32,7 @@
#include <time.h>
#include <pxelinux.h>
+#include "s390-arch.h"
#include "s390-ccw.h"
#include "cio.h"
#include "virtio.h"
@@ -43,6 +44,8 @@
extern char _start[];
void write_iplb_location(void) {}
+LowCore *lowcore; /* Yes, this *is* a pointer to address 0 */
+
#define KERNEL_ADDR ((void *)0L)
#define KERNEL_MAX_SIZE ((long)_start)
#define ARCH_COMMAND_LINE_SIZE 896 /* Taken from Linux kernel */
diff --git a/pc-bios/s390-ccw/start.S b/pc-bios/s390-ccw/start.S
index ce519300a1..939aac3a7c 100644
--- a/pc-bios/s390-ccw/start.S
+++ b/pc-bios/s390-ccw/start.S
@@ -34,7 +34,17 @@ remainder:
larl %r2,memsetxc
ex %r3,0(%r2)
done:
- j main /* And call C */
+ /* prepare i/o call handler */
+ larl %r1, io_new_code
+ larl %r2, io_new_psw
+ stg %r1, 8(%r2)
+ mvc 0x1f0(16),0(%r2)
+ /* prepare external call handler */
+ larl %r1, external_new_code
+ larl %r2, external_new_psw
+ stg %r1, 8(%r2)
+ mvc 0x1b0(16),0(%r2)
+ j main /* And call C */
memsetxc:
xc 0(1,%r1),0(%r1)
@@ -64,13 +74,16 @@ consume_sclp_int:
oi 6(%r15),0x2
lctlg %c0,%c0,0(%r15)
/* prepare external call handler */
- larl %r1, external_new_code
- stg %r1, 0x1b8
- larl %r1, external_new_mask
- mvc 0x1b0(8),0(%r1)
- /* load enabled wait PSW */
- larl %r1, enabled_wait_psw
- lpswe 0(%r1)
+ larl %r1, external_new_psw
+ lghi %r2, 0x1b0
+ /* Is the BIOS' external new PSW already set? */
+ clc 0(16, %r1), 0(%r2)
+ je load_ewait
+ /* No, save old PSW and write BIOS PSW */
+ larl %r3, psw_save_ext
+ mvc 0(16, %r3), 0x1b0
+ mvc 0x1b0(16),0(%r1)
+ j load_ewait
/*
* void consume_io_int(void)
@@ -84,11 +97,20 @@ consume_io_int:
oi 4(%r15), 0xff
lctlg %c6,%c6,0(%r15)
/* prepare i/o call handler */
- larl %r1, io_new_code
- stg %r1, 0x1f8
- larl %r1, io_new_mask
- mvc 0x1f0(8),0(%r1)
- /* load enabled wait PSW */
+ larl %r1, io_new_psw
+ lghi %r2, 0x1f0
+ /* Is the BIOS' PSW already set? */
+ larl %r3, load_ewait
+ clc 0(16, %r1), 0(%r2)
+ bcr 8, %r3
+ /* No, save old PSW and write BIOS PSW */
+ larl %r3, psw_save_io
+ mvc 0(16, %r3), 0x1f0
+ mvc 0x1f0(16),0(%r1)
+ j load_ewait
+
+load_ewait:
+ /* PSW is the correct one, time to load the enabled wait PSW */
larl %r1, enabled_wait_psw
lpswe 0(%r1)
@@ -107,11 +129,17 @@ io_new_code:
br %r14
.align 8
+ .globl psw_save_io
+ .globl psw_save_ext
disabled_wait_psw:
.quad 0x0002000180000000,0x0000000000000000
enabled_wait_psw:
.quad 0x0302000180000000,0x0000000000000000
-external_new_mask:
- .quad 0x0000000180000000
-io_new_mask:
- .quad 0x0000000180000000
+external_new_psw:
+ .quad 0x0000000180000000,0
+io_new_psw:
+ .quad 0x0000000180000000,0
+psw_save_io:
+ .quad 0,0
+psw_save_ext:
+ .quad 0,0
--
2.25.1
next prev parent reply other threads:[~2020-08-31 15:12 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-08-31 15:09 [PATCH v3 0/5] pc-bios: s390x: Cleanup part 2 Janosch Frank
2020-08-31 15:09 ` [PATCH v3 1/5] pc-bios: s390x: Fix bootmap.c zipl component entry data handling Janosch Frank
2020-08-31 15:09 ` [PATCH v3 2/5] pc-bios: s390x: Save PSW rework Janosch Frank
2020-08-31 15:27 ` Thomas Huth
2020-08-31 15:09 ` [PATCH v3 3/5] pc-bios: s390x: Use reset PSW if avaliable Janosch Frank
2020-09-01 16:59 ` Thomas Huth
2020-09-02 8:46 ` Janosch Frank
2020-09-02 9:50 ` Thomas Huth
2020-08-31 15:09 ` Janosch Frank [this message]
2020-09-01 17:22 ` [PATCH v3 4/5] pc-bios: s390x: Save io and external new PSWs before overwriting them Thomas Huth
2020-09-02 10:30 ` Janosch Frank
2020-09-02 16:55 ` Thomas Huth
2020-08-31 15:09 ` [PATCH v3 5/5] pc-bios: s390x: Go into disabled wait when encountering a PGM exception Janosch Frank
2020-09-01 17:24 ` Thomas Huth
2020-08-31 18:31 ` [PATCH v3 0/5] pc-bios: s390x: Cleanup part 2 no-reply
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=20200831150910.317171-5-frankja@linux.ibm.com \
--to=frankja@linux.ibm.com \
--cc=borntraeger@de.ibm.com \
--cc=cohuck@redhat.com \
--cc=david@redhat.com \
--cc=qemu-devel@nongnu.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).