mirror of
https://github.com/Motorhead1991/qemu.git
synced 2025-08-08 02:03:56 -06:00
s390x/ioinst: Rework memory access in STCRW instruction
Change the handler for STCRW to use the new logical memory access functions. Since STCRW is suppressed on protection/access exceptions, we also have to make sure to re-queue the CRW in case it could not be written to the memory. Signed-off-by: Thomas Huth <thuth@linux.vnet.ibm.com> Signed-off-by: Jens Freimann <jfrei@linux.vnet.ibm.com> Reviewed-by: David Hildenbrand <dahi@linux.vnet.ibm.com> Reviewed-by: Cornelia Huck <cornelia.huck@de.ibm.com> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
This commit is contained in:
parent
b7b6348ab4
commit
7f74f0aa74
3 changed files with 31 additions and 12 deletions
|
@ -254,10 +254,9 @@ void ioinst_handle_ssch(S390CPU *cpu, uint64_t reg1, uint32_t ipb)
|
|||
|
||||
void ioinst_handle_stcrw(S390CPU *cpu, uint32_t ipb)
|
||||
{
|
||||
CRW *crw;
|
||||
CRW crw;
|
||||
uint64_t addr;
|
||||
int cc;
|
||||
hwaddr len = sizeof(*crw);
|
||||
CPUS390XState *env = &cpu->env;
|
||||
|
||||
addr = decode_basedisp_s(env, ipb);
|
||||
|
@ -265,17 +264,16 @@ void ioinst_handle_stcrw(S390CPU *cpu, uint32_t ipb)
|
|||
program_interrupt(env, PGM_SPECIFICATION, 2);
|
||||
return;
|
||||
}
|
||||
crw = s390_cpu_physical_memory_map(env, addr, &len, 1);
|
||||
if (!crw || len != sizeof(*crw)) {
|
||||
program_interrupt(env, PGM_ADDRESSING, 2);
|
||||
goto out;
|
||||
}
|
||||
cc = css_do_stcrw(crw);
|
||||
/* 0 - crw stored, 1 - zeroes stored */
|
||||
setcc(cpu, cc);
|
||||
|
||||
out:
|
||||
s390_cpu_physical_memory_unmap(env, crw, len, 1);
|
||||
cc = css_do_stcrw(&crw);
|
||||
/* 0 - crw stored, 1 - zeroes stored */
|
||||
|
||||
if (s390_cpu_virt_mem_write(cpu, addr, &crw, sizeof(crw)) == 0) {
|
||||
setcc(cpu, cc);
|
||||
} else if (cc == 0) {
|
||||
/* Write failed: requeue CRW since STCRW is a suppressing instruction */
|
||||
css_undo_stcrw(&crw);
|
||||
}
|
||||
}
|
||||
|
||||
void ioinst_handle_stsch(S390CPU *cpu, uint64_t reg1, uint32_t ipb)
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue