Skip to content

s390x: Improve architectural compliance for diag308

Janosch Frank kvm linux-s390 2 versions – In review

Get series with b4
b4 am [email protected]

Versions newest first · linked by author and title

  1. v2 Latest3 patches 10 repliesIn review
  2. v12 patches 7 repliesSuperseded

Changes in v2 from the cover letter

* Fixed comment in extra patch
* Now using pre-defined reset psw
* Fixed some comments

Patches in v2 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3s390x: Fix initial cr0 load commentss390x/cstart64.S+2 -21 reply
2/3s390x: Add CR save arealib/s390x/asm-offsets.c lib/s390x/asm/arch_def.h lib/s390x/interrupt.c +2 more+11 -112 replies
3/3s390x: Load reset psw on diag308 resetlib/s390x/asm-offsets.c lib/s390x/asm/arch_def.h s390x/cstart64.S+20 -810 replies

Reviews trailers per patch

All 3 patches have a Reviewed-by or Acked-by.

PatchDavid HildenbrandThomas Huth
1/3R-b
2/3R-b R-b
3/3R-b R-b

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v2 thread

Cover letter [kvm-unit-tests PATCH v2 0/3] s390x: Improve architectural compliance for diag308

When testing diag308 subcodes 0/1 on lpar with virtual mem set up, I experienced spec PGMs and addressing PGMs due to the tests not setting short psw bit 12 and leaving the DAT bit on. The problem was not found under KVM/QEMU, because Qemu just ignores all cpu mask bits.

v1 -> v2:
* Fixed comment in extra patch
* Now using pre-defined reset psw
* Fixed some comments

Janosch Frank (3):
s390x: Fix initial cr0 load comments
s390x: Add CR save area
s390x: Load reset psw on diag308 reset

 lib/s390x/asm-offsets.c  |  3 ++-
 lib/s390x/asm/arch_def.h |  5 +++--
 lib/s390x/interrupt.c    |  4 ++--
 lib/s390x/smp.c          |  2 +-
 s390x/cstart64.S         | 38 ++++++++++++++++++++++++--------------
 5 files changed, 32 insertions(+), 20 deletions(-)
Signature · 1 line
2.20.1