The following changes since commit bad76ac319556dab2497429d473b49a237672e1c:

Merge remote-tracking branch 'remotes/vivier2/tags/trivial-branch-pull-request' into staging (2019-10-25 14:17:08 +0100)

https://github.com/rth7680/qemu.git tags/pull-tcg-20191025

for you to fetch changes up to 0ed1bfb046b740b70eed2cf3581e01768703b185:

translate-all: Remove tb_alloc (2019-10-25 10:15:25 -0400)

Improvements for TARGET_PAGE_BITS_VARY

Fix for TCI ld16u_i64.

Fix for segv on icount execute from i/o memory.

Two misc cleanups.

Alex Bennée (1):

cputlb: ensure _cmmu helper functions follow the naming standard

Clement Deschamps (1):

translate-all: fix uninitialized tb->orig_tb

Richard Henderson (8):

exec: Split out variable page size support to exec-vary.c

configure: Detect compiler support for __attribute__((alias))

exec: Use const alias for TARGET_PAGE_BITS_VARY

exec: Restrict TARGET_PAGE_BITS_VARY assert to CONFIG_DEBUG_TCG

exec: Promote TARGET_PAGE_MASK to target_long

exec: Cache TARGET_PAGE_MASK for TARGET_PAGE_BITS_VARY

cputlb: Fix tlb_vaddr_to_host

translate-all: Remove tb_alloc

Since 2ac01d6dafab, this function does only two things: assert a

lock is held, and call tcg_tb_alloc. It is used exactly once,

and its user has already done the assert.

Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>

Reviewed-by: Clement Deschamps <clement.deschamps@greensocs.com>

accel/tcg/translate-all.c | 20 ++------------------

1 file changed, 2 insertions(+), 18 deletions(-)

diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c

--- a/accel/tcg/translate-all.c

+++ b/accel/tcg/translate-all.c

@@ -XXX,XX +XXX,XX @@ void tcg_exec_init(unsigned long tb_size)

#endif

2.17.1

From: Stefan Weil <sw@weilnetz.de>

This fixes "make check-tcg" on a Debian x86_64 host.

Signed-off-by: Stefan Weil <sw@weilnetz.de>

Tested-by: Thomas Huth <thuth@redhat.com>

Message-Id: <20190410194838.10123-1-sw@weilnetz.de>

tcg/tci.c | 15 +++++++++++++++

1 file changed, 15 insertions(+)

diff --git a/tcg/tci.c b/tcg/tci.c

--- a/tcg/tci.c

+++ b/tcg/tci.c

@@ -XXX,XX +XXX,XX @@ static void tci_write_reg8(tcg_target_ulong *regs, TCGReg index, uint8_t value)

tci_write_reg(regs, index, value);

}

+static void

+tci_write_reg16(tcg_target_ulong *regs, TCGReg index, uint16_t value)

+{

+ tci_write_reg(regs, index, value);

+}

static void

tci_write_reg32(tcg_target_ulong *regs, TCGReg index, uint32_t value)

{

@@ -XXX,XX +XXX,XX @@ uintptr_t tcg_qemu_tb_exec(CPUArchState *env, uint8_t *tb_ptr)

tci_write_reg8(regs, t0, *(uint8_t *)(t1 + t2));

break;

case INDEX_op_ld8s_i32:

+ TODO();

+ break;

case INDEX_op_ld16u_i32:

TODO();

break;

@@ -XXX,XX +XXX,XX @@ uintptr_t tcg_qemu_tb_exec(CPUArchState *env, uint8_t *tb_ptr)

tci_write_reg8(regs, t0, *(uint8_t *)(t1 + t2));

break;

case INDEX_op_ld8s_i64:

+ TODO();

+ break;

case INDEX_op_ld16u_i64:

+ t0 = *tb_ptr++;

+ t1 = tci_read_r(regs, &tb_ptr);

+ t2 = tci_read_s32(&tb_ptr);

+ tci_write_reg16(regs, t0, *(uint16_t *)(t1 + t2));

+ break;

case INDEX_op_ld16s_i64:

TODO();

break;

2.17.1

From: Clement Deschamps <clement.deschamps@greensocs.com>

This fixes a segmentation fault in icount mode when executing

from an IO region.

TB is marked as CF_NOCACHE but tb->orig_tb is not initialized

(equals previous value in code_gen_buffer).

The issue happens in cpu_io_recompile() when it tries to invalidate orig_tb.

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

Signed-off-by: Clement Deschamps <clement.deschamps@greensocs.com>

Message-Id: <20191022140016.918371-1-clement.deschamps@greensocs.com>

accel/tcg/translate-all.c | 1 +

1 file changed, 1 insertion(+)

diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c

--- a/accel/tcg/translate-all.c

+++ b/accel/tcg/translate-all.c

@@ -XXX,XX +XXX,XX @@ TranslationBlock *tb_gen_code(CPUState *cpu,

tb->cs_base = cs_base;

tb->flags = flags;

tb->cflags = cflags;

+ tb->orig_tb = NULL;

tb->trace_vcpu_dstate = *cpu->trace_dstate;

tcg_ctx->tb_cflags = cflags;

tb_overflow:

2.17.1