[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 24/31] target/s390x: Simplify helper_lra
From: |
David Hildenbrand |
Subject: |
[PULL 24/31] target/s390x: Simplify helper_lra |
Date: |
Thu, 10 Oct 2019 13:33:49 +0200 |
From: Richard Henderson <address@hidden>
We currently call trigger_pgm_exception to set cs->exception_index
and env->int_pgm_code and then read the values back and then
reset cs->exception_index so that the exception is not delivered.
Instead, use the exception type that we already have directly
without ever triggering an exception that must be suppressed.
Reviewed-by: David Hildenbrand <address@hidden>
Signed-off-by: Richard Henderson <address@hidden>
Message-Id: <address@hidden>
Signed-off-by: David Hildenbrand <address@hidden>
---
target/s390x/mem_helper.c | 17 +++--------------
1 file changed, 3 insertions(+), 14 deletions(-)
diff --git a/target/s390x/mem_helper.c b/target/s390x/mem_helper.c
index e15aa296dd..4254548935 100644
--- a/target/s390x/mem_helper.c
+++ b/target/s390x/mem_helper.c
@@ -2361,34 +2361,23 @@ void HELPER(sturg)(CPUS390XState *env, uint64_t addr,
uint64_t v1)
/* load real address */
uint64_t HELPER(lra)(CPUS390XState *env, uint64_t addr)
{
- CPUState *cs = env_cpu(env);
- uint32_t cc = 0;
uint64_t asc = env->psw.mask & PSW_MASK_ASC;
uint64_t ret, tec;
- int old_exc, flags, exc;
+ int flags, exc, cc;
/* XXX incomplete - has more corner cases */
if (!(env->psw.mask & PSW_MASK_64) && (addr >> 32)) {
tcg_s390_program_interrupt(env, PGM_SPECIAL_OP, GETPC());
}
- old_exc = cs->exception_index;
exc = mmu_translate(env, addr, 0, asc, &ret, &flags, &tec);
if (exc) {
- /*
- * We don't care about ILEN or TEC, as we're not going to
- * deliver the exception -- thus resetting exception_index below.
- * TODO: clean this up.
- */
- trigger_pgm_exception(env, exc, ILEN_UNWIND);
cc = 3;
- }
- if (cs->exception_index == EXCP_PGM) {
- ret = env->int_pgm_code | 0x80000000;
+ ret = exc | 0x80000000;
} else {
+ cc = 0;
ret |= addr & ~TARGET_PAGE_MASK;
}
- cs->exception_index = old_exc;
env->cc_op = cc;
return ret;
--
2.21.0
- [PULL 17/31] target/s390x: Push trigger_pgm_exception lower in s390_cpu_tlb_fill, (continued)
- [PULL 20/31] target/s390x: Remove exc argument to mmu_translate_asce, David Hildenbrand, 2019/10/10
- [PULL 21/31] target/s390x: Return exception from mmu_translate, David Hildenbrand, 2019/10/10
- [PULL 22/31] target/s390x: Return exception from translate_pages, David Hildenbrand, 2019/10/10
- [PULL 23/31] target/s390x: Remove fail variable from s390_cpu_tlb_fill, David Hildenbrand, 2019/10/10
- [PULL 24/31] target/s390x: Simplify helper_lra,
David Hildenbrand <=
- [PULL 25/31] target/s390x: Rely on unwinding in s390_cpu_tlb_fill, David Hildenbrand, 2019/10/10
- [PULL 26/31] target/s390x: Rely on unwinding in s390_cpu_virt_mem_rw, David Hildenbrand, 2019/10/10
- [PULL 27/31] target/s390x: Remove ILEN_AUTO, David Hildenbrand, 2019/10/10
- [PULL 28/31] target/s390x: Remove ilen argument from trigger_access_exception, David Hildenbrand, 2019/10/10
- [PULL 29/31] target/s390x: Remove ilen argument from trigger_pgm_exception, David Hildenbrand, 2019/10/10
- [PULL 30/31] target/s390x: Remove ILEN_UNWIND, David Hildenbrand, 2019/10/10
- [PULL 31/31] s390x/tcg: MVCL: Exit to main loop if requested, David Hildenbrand, 2019/10/10
- Re: [PULL 00/31] s390x/tcg update, no-reply, 2019/10/10
- Re: [PULL 00/31] s390x/tcg update, Peter Maydell, 2019/10/14