mbox series

[v4,00/18] target/s390: Use tcg unwinding for ilen

Message ID 20190927193925.23567-1-richard.henderson@linaro.org
Headers show
Series target/s390: Use tcg unwinding for ilen | expand

Message

Richard Henderson Sept. 27, 2019, 7:39 p.m. UTC
Based-on: <20190925125236.4043-1-david@redhat.com> \
  ("s390x/mmu: DAT translation rewrite")

Based-on: <20190926101627.23376-1-david@redhat.com> \
  ("s390x/mmu: Implement more facilities") \
  With the suggested follow-up for patch 2 re ilen.

Which should mean that this applies on top of David's
current s390x tree.  ;-)

Changes since v3:
  * Dropped the unrelated psw.addr wrapping patch.
  * Set ilen to 2 for breakpoints, which means that we can make
    the assignment within restore_state_to_opc unconditional.
  * Remove ILEN_UNWIND.  After the penultimate patch, we nowhere
    play odd games with ILEN.  Yay!

Patches without review or ack:

0001-target-s390x-Add-ilen-to-unwind-data
0009-target-s390x-Return-exception-from-mmu_translate
0018-target-s390x-Remove-ILEN_UNWIND (new)


r~


Richard Henderson (18):
  target/s390x: Add ilen to unwind data
  target/s390x: Remove ilen parameter from tcg_s390_program_interrupt
  target/s390x: Remove ilen parameter from s390_program_interrupt
  target/s390x: Use tcg_s390_program_interrupt in TCG helpers
  target/s390x: Push trigger_pgm_exception lower in s390_cpu_tlb_fill
  target/s390x: Handle tec in s390_cpu_tlb_fill
  target/s390x: Return exception from mmu_translate_real
  target/s390x: Remove exc argument to mmu_translate_asce
  target/s390x: Return exception from mmu_translate
  target/s390x: Return exception from translate_pages
  target/s390x: Remove fail variable from s390_cpu_tlb_fill
  target/s390x: Simplify helper_lra
  target/s390x: Rely on unwinding in s390_cpu_tlb_fill
  target/s390x: Rely on unwinding in s390_cpu_virt_mem_rw
  target/s390x: Remove ILEN_AUTO
  target/s390x: Remove ilen argument from trigger_access_exception
  target/s390x: Remove ilen argument from trigger_pgm_exception
  target/s390x: Remove ILEN_UNWIND

 target/s390x/cpu.h           |  7 +---
 target/s390x/internal.h      |  6 +--
 target/s390x/tcg_s390x.h     |  4 +-
 hw/s390x/s390-pci-inst.c     | 58 +++++++++++++-------------
 target/s390x/cc_helper.c     |  4 +-
 target/s390x/crypto_helper.c |  7 ++--
 target/s390x/diag.c          | 14 +++----
 target/s390x/excp_helper.c   | 60 ++++++++++++---------------
 target/s390x/fpu_helper.c    |  6 +--
 target/s390x/int_helper.c    | 15 +++----
 target/s390x/interrupt.c     |  9 ++--
 target/s390x/ioinst.c        | 40 +++++++++---------
 target/s390x/mem_helper.c    | 61 +++++++++++++--------------
 target/s390x/misc_helper.c   | 27 ++++--------
 target/s390x/mmu_helper.c    | 80 +++++++++++++++---------------------
 target/s390x/tcg-stub.c      |  4 +-
 target/s390x/translate.c     | 20 +++++++--
 17 files changed, 199 insertions(+), 223 deletions(-)

-- 
2.17.1

Comments

David Hildenbrand Oct. 1, 2019, 8:07 a.m. UTC | #1
On 27.09.19 21:39, Richard Henderson wrote:
> Based-on: <20190925125236.4043-1-david@redhat.com> \

>   ("s390x/mmu: DAT translation rewrite")

> 

> Based-on: <20190926101627.23376-1-david@redhat.com> \

>   ("s390x/mmu: Implement more facilities") \

>   With the suggested follow-up for patch 2 re ilen.

> 

> Which should mean that this applies on top of David's

> current s390x tree.  ;-)


Indeed, both are located at
	https://github.com/davidhildenbrand/qemu/tree/mmu

Once I have more feedback on "s390x/mmu: Implement more facilities", I
am going to pick up this series and give it a good test. Then send it
upstream (either via Conny, due to the CPU model change, or directly).
Sounds good?

-- 

Thanks,

David / dhildenb
Richard Henderson Oct. 1, 2019, 3:02 p.m. UTC | #2
On 10/1/19 1:07 AM, David Hildenbrand wrote:
> On 27.09.19 21:39, Richard Henderson wrote:

>> Based-on: <20190925125236.4043-1-david@redhat.com> \

>>   ("s390x/mmu: DAT translation rewrite")

>>

>> Based-on: <20190926101627.23376-1-david@redhat.com> \

>>   ("s390x/mmu: Implement more facilities") \

>>   With the suggested follow-up for patch 2 re ilen.

>>

>> Which should mean that this applies on top of David's

>> current s390x tree.  ;-)

> 

> Indeed, both are located at

> 	https://github.com/davidhildenbrand/qemu/tree/mmu

> 

> Once I have more feedback on "s390x/mmu: Implement more facilities", I

> am going to pick up this series and give it a good test. Then send it

> upstream (either via Conny, due to the CPU model change, or directly).

> Sounds good?

> 


Yep, thanks!


r~