qemu-commits
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Qemu-commits] [qemu/qemu] 8973fe: tcg: Add separator in INDEX_op_call d


From: Peter Maydell
Subject: [Qemu-commits] [qemu/qemu] 8973fe: tcg: Add separator in INDEX_op_call dump
Date: Mon, 12 Jul 2021 03:08:21 -0700

  Branch: refs/heads/staging
  Home:   https://github.com/qemu/qemu
  Commit: 8973fe43bb6d80f01ea11686c29f98fc4dcae3a6
      
https://github.com/qemu/qemu/commit/8973fe43bb6d80f01ea11686c29f98fc4dcae3a6
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M tcg/tcg.c

  Log Message:
  -----------
  tcg: Add separator in INDEX_op_call dump

We lost the ',' following the called function name.

Fixes: 3e92aa34434
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 1797b08d244ce496d0b0f5027a75542a82c29038
      
https://github.com/qemu/qemu/commit/1797b08d244ce496d0b0f5027a75542a82c29038
  Author: Philippe Mathieu-Daudé <f4bug@amsat.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/alpha/translate.c
    M target/arm/translate-a64.c
    M target/arm/translate-sve.c
    M target/arm/translate.c
    M target/cris/translate.c
    M target/hppa/translate.c
    M target/i386/tcg/translate.c
    M target/m68k/translate.c
    M target/microblaze/translate.c
    M target/mips/tcg/translate.c
    M target/openrisc/translate.c
    M target/ppc/translate.c
    M target/rx/translate.c
    M target/s390x/translate.c
    M target/sh4/translate.c
    M target/sparc/translate.c
    M target/xtensa/translate.c

  Log Message:
  -----------
  tcg: Avoid including 'trace-tcg.h' in target translate.c

The root trace-events only declares a single TCG event:

  $ git grep -w tcg trace-events
  trace-events:115:# tcg/tcg-op.c
  trace-events:137:vcpu tcg guest_mem_before(TCGv vaddr, uint16_t info) 
"info=%d", "vaddr=0x%016"PRIx64" info=%d"

and only a tcg/tcg-op.c uses it:

  $ git grep -l trace_guest_mem_before_tcg
  tcg/tcg-op.c

therefore it is pointless to include "trace-tcg.h" in each target
(because it is not used). Remove it.

Signed-off-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Message-Id: <20210629050935.2570721-1-f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: f4cba756cb6c6173321c4aad2035d5a86100eef7
      
https://github.com/qemu/qemu/commit/f4cba756cb6c6173321c4aad2035d5a86100eef7
  Author: Liren Wei <lrwei@bupt.edu.cn>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/translate-all.c

  Log Message:
  -----------
  accel/tcg: Hoist tcg_tb_insert() up above tb_link_page()

TranslationBlocks not inserted into the corresponding region
tree shall be regarded as partially initialized objects, and
needs to be finalized first before inserting into QHT.

Signed-off-by: Liren Wei <lrwei@bupt.edu.cn>
Message-Id: 
<f9fc263f71e11b6308d8c1fbc0dd366bf4aeb532.1625404483.git.lrwei@bupt.edu.cn>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 834361efd9d52947663aa5b297693f8e352bef2a
      
https://github.com/qemu/qemu/commit/834361efd9d52947663aa5b297693f8e352bef2a
  Author: Liren Wei <lrwei@bupt.edu.cn>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/translate-all.c
    M include/tcg/tcg.h
    M tcg/region.c

  Log Message:
  -----------
  tcg: Bake tb_destroy() into tcg_region_tree

The function is called only at tcg_gen_code() when duplicated TBs
are translated by different threads, and when the tcg_region_tree
is reset. Bake it into the underlying GTree as its value destroy
function to unite these situations.
Also remove tcg_region_tree_traverse() which now becomes useless.

Signed-off-by: Liren Wei <lrwei@bupt.edu.cn>
Message-Id: 
<8dc352f08d038c4e7a1f5f56962398cdc700c3aa.1625404483.git.lrwei@bupt.edu.cn>
[rth: Name the new tb_tc_cmp parameter correctly.]
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: a4390647f7af6b8a539571b44ab537478f0ae548
      
https://github.com/qemu/qemu/commit/a4390647f7af6b8a539571b44ab537478f0ae548
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/tb-context.h
    M accel/tcg/translate-all.c
    M include/tcg/tcg.h
    M tcg/region.c

  Log Message:
  -----------
  tcg: Move tb_phys_invalidate_count to tb_ctx

We can call do_tb_phys_invalidate from an iocontext, which has
no per-thread tcg_ctx.  Move this to tb_ctx, which is global.
The actual update still takes place with a lock held, so only
an atomic set is required, not an atomic increment.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/457
Tested-by: Viktor Ashirov <vashirov@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: d3a2a1d80331b437bcfa0dc43f2c447d3104898e
      
https://github.com/qemu/qemu/commit/d3a2a1d80331b437bcfa0dc43f2c447d3104898e
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/translator.c
    M include/exec/translator.h

  Log Message:
  -----------
  accel/tcg: Introduce translator_use_goto_tb

Add a generic version of the common use_goto_tb test.

Various targets avoid the page crossing test for CONFIG_USER_ONLY,
but that is wrong: mmap and mprotect can change page permissions.

Reviewed-by: Max Filippov <jcmvbkbc@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: cba201f4c74a99b6ee0a43d66f5dae8f901b4006
      
https://github.com/qemu/qemu/commit/cba201f4c74a99b6ee0a43d66f5dae8f901b4006
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/alpha/translate.c

  Log Message:
  -----------
  target/alpha: Remove use_exit_tb

We have not needed to end a TB for I/O since ba3e7926691
("icount: clean up cpu_can_io at the entry to the block").
We do not need to use exit_tb for singlestep, which only
means generate one insn per TB.

Which leaves only singlestep_enabled, which means raise a
debug trap after every TB, which does not use exit_tb,
which would leave the function mis-named.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 3fd3442abe24f8cabcbb40b73ffe7de81e5db446
      
https://github.com/qemu/qemu/commit/3fd3442abe24f8cabcbb40b73ffe7de81e5db446
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/alpha/translate.c

  Log Message:
  -----------
  target/alpha: Remove in_superpage

The number of links across (normal) pages using this is low,
and it will shortly violate the contract for breakpoints.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 21a7e89eced0372cb465e0c9b0a19ffc4e7b75b1
      
https://github.com/qemu/qemu/commit/21a7e89eced0372cb465e0c9b0a19ffc4e7b75b1
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/alpha/translate.c

  Log Message:
  -----------
  target/alpha: Use translator_use_goto_tb

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 73fce314dbbf2d1c3cd411a34535ebe5d20e1e45
      
https://github.com/qemu/qemu/commit/73fce314dbbf2d1c3cd411a34535ebe5d20e1e45
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/arm/translate.c

  Log Message:
  -----------
  target/arm: Use DISAS_TOO_MANY for ISB and SB

Using gen_goto_tb directly misses the single-step check.
Let the branch or debug exception be emitted by arm_tr_tb_stop.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 0285162bdf5f35c5c80df43cfb8941c9105ccfb1
      
https://github.com/qemu/qemu/commit/0285162bdf5f35c5c80df43cfb8941c9105ccfb1
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/arm/translate-a64.c

  Log Message:
  -----------
  target/arm: Use translator_use_goto_tb for aarch64

We have not needed to end a TB for I/O since ba3e7926691
("icount: clean up cpu_can_io at the entry to the block"),
and gdbstub singlestep is handled by the generic function.

Drop the unused 'n' argument to use_goto_tb.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 97f11c81699a2ca5b68cd33f53e4e4997910dd60
      
https://github.com/qemu/qemu/commit/97f11c81699a2ca5b68cd33f53e4e4997910dd60
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/arm/translate.c

  Log Message:
  -----------
  target/arm: Use translator_use_goto_tb for aarch32

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: a50d52bc3bea94f6c815692b27b6d3649a69c787
      
https://github.com/qemu/qemu/commit/a50d52bc3bea94f6c815692b27b6d3649a69c787
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/avr/translate.c

  Log Message:
  -----------
  target/avr: Use translator_use_goto_tb

Single stepping is not the only reason not to use goto_tb.
If goto_tb is disallowed, and single-stepping is not enabled,
then use tcg_gen_lookup_and_goto_tb to indirectly chain.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 1810afd56a947f4b201a6211da69100403402026
      
https://github.com/qemu/qemu/commit/1810afd56a947f4b201a6211da69100403402026
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/avr/helper.h

  Log Message:
  -----------
  target/avr: Mark some helpers noreturn

All of these helpers end with cpu_loop_exit.

Reviewed-by: Michael Rolnik <mrolnik@gmail.com>
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: ca92d7f89b1bdc0cbe6a644d39f96cab23a33bd0
      
https://github.com/qemu/qemu/commit/ca92d7f89b1bdc0cbe6a644d39f96cab23a33bd0
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/cris/translate.c

  Log Message:
  -----------
  target/cris: Use translator_use_goto_tb

The test for singlestepping is done in translator_use_goto_tb,
so we may elide it from cris_tr_tb_stop.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 57f914983c5a73dcff14403e7b2080bd7664cbf5
      
https://github.com/qemu/qemu/commit/57f914983c5a73dcff14403e7b2080bd7664cbf5
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/hppa/translate.c

  Log Message:
  -----------
  target/hppa: Use translator_use_goto_tb

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: b473534d5df82042d1b2c9c651d3e80772ce0f4b
      
https://github.com/qemu/qemu/commit/b473534d5df82042d1b2c9c651d3e80772ce0f4b
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/i386/tcg/translate.c

  Log Message:
  -----------
  target/i386: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: fbf565c4e010e749b7536ccec4eae38729791fac
      
https://github.com/qemu/qemu/commit/fbf565c4e010e749b7536ccec4eae38729791fac
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/m68k/translate.c

  Log Message:
  -----------
  target/m68k: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Acked-by: Laurent Vivier <laurent@vivier.eu>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 725930c2a57d37f925b9b28f1655961a231f7d20
      
https://github.com/qemu/qemu/commit/725930c2a57d37f925b9b28f1655961a231f7d20
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/microblaze/translate.c

  Log Message:
  -----------
  target/microblaze: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 97eea3c19bda9537df35c43382f954335dceafff
      
https://github.com/qemu/qemu/commit/97eea3c19bda9537df35c43382f954335dceafff
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/mips/tcg/translate.c

  Log Message:
  -----------
  target/mips: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 34f5e75a9427e00d0f9d745a2dc20a955ec935bf
      
https://github.com/qemu/qemu/commit/34f5e75a9427e00d0f9d745a2dc20a955ec935bf
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/mips/tcg/translate.c

  Log Message:
  -----------
  target/mips: Fix missing else in gen_goto_tb

Do not emit dead code for the singlestep_enabled case,
after having exited the TB with a debug exception.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 6082414e3f240de12987f5136b5fdd314379262c
      
https://github.com/qemu/qemu/commit/6082414e3f240de12987f5136b5fdd314379262c
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/nios2/translate.c

  Log Message:
  -----------
  target/nios2: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: adf1f3dee69f88a66fc36c962dbf74d28de84b65
      
https://github.com/qemu/qemu/commit/adf1f3dee69f88a66fc36c962dbf74d28de84b65
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/openrisc/translate.c

  Log Message:
  -----------
  target/openrisc: Use translator_use_goto_tb

Reorder the control statements to allow using the page boundary
check from translator_use_goto_tb().

Reviewed-by: Stafford Horne <shorne@gmail.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 6e9cc373ec5871bfd8aca36e319ded67e48ca58f
      
https://github.com/qemu/qemu/commit/6e9cc373ec5871bfd8aca36e319ded67e48ca58f
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/ppc/translate.c

  Log Message:
  -----------
  target/ppc: Use translator_use_goto_tb

Reviewed-by: Luis Pires <luis.pires@eldorado.org.br>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: c54d50c1d6d0607ba3b1f803a9284e4b55124522
      
https://github.com/qemu/qemu/commit/c54d50c1d6d0607ba3b1f803a9284e4b55124522
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/riscv/translate.c

  Log Message:
  -----------
  target/riscv: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Alistair Francis <alistair.francis@wdc.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: f3f713cc151086ca39d4f97270594fd8c43e17e5
      
https://github.com/qemu/qemu/commit/f3f713cc151086ca39d4f97270594fd8c43e17e5
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/rx/translate.c

  Log Message:
  -----------
  target/rx: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 7379c0c82e8164b55dbf605ba9b3988442a73025
      
https://github.com/qemu/qemu/commit/7379c0c82e8164b55dbf605ba9b3988442a73025
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/s390x/translate.c

  Log Message:
  -----------
  target/s390x: Use translator_use_goto_tb

Reviewed-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 95cc1cb39e12fbb5cca3688f8a7ce22116846520
      
https://github.com/qemu/qemu/commit/95cc1cb39e12fbb5cca3688f8a7ce22116846520
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/s390x/translate.c

  Log Message:
  -----------
  target/s390x: Remove use_exit_tb

We have not needed to end a TB for I/O since ba3e7926691
("icount: clean up cpu_can_io at the entry to the block").

In use_goto_tb, the check for singlestep_enabled is in the
generic translator_use_goto_tb.  In s390x_tr_tb_stop, the
check for singlestep_enabled is in the preceding do_debug test.

Which leaves only FLAG_MASK_PER: fold that test alone into
the two callers of use_exit tb.

Reviewed-by: David Hildenbrand <david@redhat.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 3f1e20984519ad3823438b73ea10035c0b2a9ffd
      
https://github.com/qemu/qemu/commit/3f1e20984519ad3823438b73ea10035c0b2a9ffd
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/sh4/translate.c

  Log Message:
  -----------
  target/sh4: Use translator_use_goto_tb

Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 5645aa2e76c4ab8ab1418d5d0fa732d06765d44f
      
https://github.com/qemu/qemu/commit/5645aa2e76c4ab8ab1418d5d0fa732d06765d44f
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/sparc/translate.c

  Log Message:
  -----------
  target/sparc: Use translator_use_goto_tb

Reviewed-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: d6b6f26170052452473fd8e674a29e337625dcdd
      
https://github.com/qemu/qemu/commit/d6b6f26170052452473fd8e674a29e337625dcdd
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/tricore/translate.c

  Log Message:
  -----------
  target/tricore: Use translator_use_goto_tb

Just use translator_use_goto_tb directly at the one call site,
rather than maintaining a local wrapper.

Reviewed-by: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 3806471563f9f1b568a32fdece189a1cecb5ca01
      
https://github.com/qemu/qemu/commit/3806471563f9f1b568a32fdece189a1cecb5ca01
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/tricore/translate.c

  Log Message:
  -----------
  target/tricore: Use tcg_gen_lookup_and_goto_ptr

The non-single-step case of gen_goto_tb may use
tcg_gen_lookup_and_goto_ptr to indirectly chain.

Reviewed-by: Bastian Koppelmann <kbastian@mail.uni-paderborn.de>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 70c6eb46d780a1ad1a53a4cfbb1052973271a62d
      
https://github.com/qemu/qemu/commit/70c6eb46d780a1ad1a53a4cfbb1052973271a62d
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/xtensa/translate.c

  Log Message:
  -----------
  target/xtensa: Use translator_use_goto_tb

Reviewed-by: Max Filippov <jcmvbkbc@gmail.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: d1c74ab3a1048a78b5fb8df06c1845e58111ee34
      
https://github.com/qemu/qemu/commit/d1c74ab3a1048a78b5fb8df06c1845e58111ee34
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M tcg/tcg.c

  Log Message:
  -----------
  tcg: Fix prologue disassembly

In tcg_region_prologue_set, we reset TCGContext.code_gen_ptr.
So do that after we've used it to dump the prologue contents.

Fixes: b0a0794a0f16
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 50b208b848d9497cf6d320b2d4a38a8f07354f5d
      
https://github.com/qemu/qemu/commit/50b208b848d9497cf6d320b2d4a38a8f07354f5d
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M target/i386/tcg/sysemu/bpt_helper.c

  Log Message:
  -----------
  target/i386: Use cpu_breakpoint_test in breakpoint_handler

The loop is performing a simple boolean test for the existence
of a BP_CPU breakpoint at EIP.  Plus it gets the iteration wrong,
if we happen to have a BP_GDB breakpoint at the same address.

We have a function for this: cpu_breakpoint_test.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Eduardo Habkost <ehabkost@redhat.com>
Message-Id: <20210620062317.1399034-1-richard.henderson@linaro.org>


  Commit: 4288eb26a08593d4ad53c07aca4f2193ab0d72af
      
https://github.com/qemu/qemu/commit/4288eb26a08593d4ad53c07aca4f2193ab0d72af
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/cpu-exec.c
    M accel/tcg/tcg-runtime.c

  Log Message:
  -----------
  accel/tcg: Move helper_lookup_tb_ptr to cpu-exec.c

This will allow additional code sharing.
No functional change.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 632cb63d9282ca58b016b6f95abcc5e42e5bda0e
      
https://github.com/qemu/qemu/commit/632cb63d9282ca58b016b6f95abcc5e42e5bda0e
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/cpu-exec.c
    R accel/tcg/tb-lookup.h

  Log Message:
  -----------
  accel/tcg: Move tb_lookup to cpu-exec.c

Now that we've moved helper_lookup_tb_ptr, the only user
of tb-lookup.h is cpu-exec.c; merge the contents in.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: abb0cd9349453d6a45af2ab9317e8f08408485f4
      
https://github.com/qemu/qemu/commit/abb0cd9349453d6a45af2ab9317e8f08408485f4
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/cpu-exec.c

  Log Message:
  -----------
  accel/tcg: Split out log_cpu_exec

Split out CPU_LOG_EXEC and CPU_LOG_TB_CPU logging from
cpu_tb_exec to a new function.  Perform only one pc
range check after a combined mask check.

Use the new function in lookup_tb_ptr.  This enables
CPU_LOG_TB_CPU between indirectly chained tbs.

Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: 7eabad361979bbf76dff4d91bc7af35e309c8c26
      
https://github.com/qemu/qemu/commit/7eabad361979bbf76dff4d91bc7af35e309c8c26
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M accel/tcg/cpu-exec.c

  Log Message:
  -----------
  accel/tcg: Log tb->cflags with -d exec

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: f4e01e30217b6778e478cf00975daed7a54bc051
      
https://github.com/qemu/qemu/commit/f4e01e30217b6778e478cf00975daed7a54bc051
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M include/tcg/tcg-opc.h
    M tcg/aarch64/tcg-target.h
    M tcg/arm/tcg-target.h
    M tcg/i386/tcg-target.h
    M tcg/mips/tcg-target.h
    M tcg/ppc/tcg-target.h
    M tcg/riscv/tcg-target.h
    M tcg/s390/tcg-target.h
    M tcg/sparc/tcg-target.h
    M tcg/tcg-op.c
    M tcg/tcg.c
    M tcg/tci/tcg-target.h

  Log Message:
  -----------
  tcg: Remove TCG_TARGET_HAS_goto_ptr

Since 6eea04347eb6, all tcg backends support goto_ptr.
Remove the conditional, making support mandatory.

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: ad1a706f386c2281adb0b09257d892735e405834
      
https://github.com/qemu/qemu/commit/ad1a706f386c2281adb0b09257d892735e405834
  Author: Richard Henderson <richard.henderson@linaro.org>
  Date:   2021-07-09 (Fri, 09 Jul 2021)

  Changed paths:
    M cpu.c
    M trace-events

  Log Message:
  -----------
  cpu: Add breakpoint tracepoints

Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


  Commit: bd38ae26cea0d1d6a97f930248df149204c210a2
      
https://github.com/qemu/qemu/commit/bd38ae26cea0d1d6a97f930248df149204c210a2
  Author: Peter Maydell <peter.maydell@linaro.org>
  Date:   2021-07-12 (Mon, 12 Jul 2021)

  Changed paths:
    M accel/tcg/cpu-exec.c
    M accel/tcg/tb-context.h
    R accel/tcg/tb-lookup.h
    M accel/tcg/tcg-runtime.c
    M accel/tcg/translate-all.c
    M accel/tcg/translator.c
    M cpu.c
    M include/exec/translator.h
    M include/tcg/tcg-opc.h
    M include/tcg/tcg.h
    M target/alpha/translate.c
    M target/arm/translate-a64.c
    M target/arm/translate-sve.c
    M target/arm/translate.c
    M target/avr/helper.h
    M target/avr/translate.c
    M target/cris/translate.c
    M target/hppa/translate.c
    M target/i386/tcg/sysemu/bpt_helper.c
    M target/i386/tcg/translate.c
    M target/m68k/translate.c
    M target/microblaze/translate.c
    M target/mips/tcg/translate.c
    M target/nios2/translate.c
    M target/openrisc/translate.c
    M target/ppc/translate.c
    M target/riscv/translate.c
    M target/rx/translate.c
    M target/s390x/translate.c
    M target/sh4/translate.c
    M target/sparc/translate.c
    M target/tricore/translate.c
    M target/xtensa/translate.c
    M tcg/aarch64/tcg-target.h
    M tcg/arm/tcg-target.h
    M tcg/i386/tcg-target.h
    M tcg/mips/tcg-target.h
    M tcg/ppc/tcg-target.h
    M tcg/region.c
    M tcg/riscv/tcg-target.h
    M tcg/s390/tcg-target.h
    M tcg/sparc/tcg-target.h
    M tcg/tcg-op.c
    M tcg/tcg.c
    M tcg/tci/tcg-target.h
    M trace-events

  Log Message:
  -----------
  Merge remote-tracking branch 'remotes/rth-gitlab/tags/pull-tcg-20210710' into 
staging

Add translator_use_goto_tb.
Cleanups in prep of breakpoint fixes.
Misc fixes.

# gpg: Signature made Sat 10 Jul 2021 16:29:14 BST
# gpg:                using RSA key 7A481E78868B4DB6A85A05C064DF38E8AF7E215F
# gpg:                issuer "richard.henderson@linaro.org"
# gpg: Good signature from "Richard Henderson <richard.henderson@linaro.org>" 
[full]
# Primary key fingerprint: 7A48 1E78 868B 4DB6 A85A  05C0 64DF 38E8 AF7E 215F

* remotes/rth-gitlab/tags/pull-tcg-20210710: (41 commits)
  cpu: Add breakpoint tracepoints
  tcg: Remove TCG_TARGET_HAS_goto_ptr
  accel/tcg: Log tb->cflags with -d exec
  accel/tcg: Split out log_cpu_exec
  accel/tcg: Move tb_lookup to cpu-exec.c
  accel/tcg: Move helper_lookup_tb_ptr to cpu-exec.c
  target/i386: Use cpu_breakpoint_test in breakpoint_handler
  tcg: Fix prologue disassembly
  target/xtensa: Use translator_use_goto_tb
  target/tricore: Use tcg_gen_lookup_and_goto_ptr
  target/tricore: Use translator_use_goto_tb
  target/sparc: Use translator_use_goto_tb
  target/sh4: Use translator_use_goto_tb
  target/s390x: Remove use_exit_tb
  target/s390x: Use translator_use_goto_tb
  target/rx: Use translator_use_goto_tb
  target/riscv: Use translator_use_goto_tb
  target/ppc: Use translator_use_goto_tb
  target/openrisc: Use translator_use_goto_tb
  target/nios2: Use translator_use_goto_tb
  ...

Signed-off-by: Peter Maydell <peter.maydell@linaro.org>


Compare: https://github.com/qemu/qemu/compare/d1987c811492...bd38ae26cea0



reply via email to

[Prev in Thread] Current Thread [Next in Thread]