[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line
From: |
Richard Henderson |
Subject: |
Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line |
Date: |
Thu, 15 Nov 2018 23:04:50 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.3.0 |
On 11/15/18 7:48 PM, Emilio G. Cota wrote:
> - Segfault in code_gen_buffer. This one I don't have a fix for,
> but it's *much* easier to reproduce when -tb-size is very small,
> e.g. "-tb-size 5 -smp 2" (BTW it crashes with x86_64 guests too.)
> So at first I thought the code cache flushing was the problem,
> but I don't see how that could be, at least from a TCGContext
> viewpoint -- I agree that clearing the hash table in
> tcg_region_assign is a good place to do so.
Ho hum.
diff --git a/accel/tcg/translate-all.c b/accel/tcg/translate-all.c
index 639f0b2728..115ea186e5 100644
--- a/accel/tcg/translate-all.c
+++ b/accel/tcg/translate-all.c
@@ -1831,10 +1831,6 @@ TranslationBlock *tb_gen_code(CPUState *cpu,
existing_tb = tb_link_page(tb, phys_pc, phys_page2);
/* if the TB already exists, discard what we just translated */
if (unlikely(existing_tb != tb)) {
- uintptr_t orig_aligned = (uintptr_t)gen_code_buf;
-
- orig_aligned -= ROUND_UP(sizeof(*tb), qemu_icache_linesize);
- atomic_set(&tcg_ctx->code_gen_ptr, (void *)orig_aligned);
return existing_tb;
}
tcg_tb_insert(tb);
We can't easily undo the hash table insert, and for a relatively rare
occurrence it's not worth the effort.
r~
- [Qemu-devel] [PATCH for-4.0 12/17] tcg/aarch64: Use TCG_TARGET_NEED_LDST_OOL_LABELS, (continued)
- [Qemu-devel] [PATCH for-4.0 12/17] tcg/aarch64: Use TCG_TARGET_NEED_LDST_OOL_LABELS, Richard Henderson, 2018/11/12
- [Qemu-devel] [PATCH for-4.0 14/17] tcg/arm: Add constraints for R0-R5, Richard Henderson, 2018/11/12
- [Qemu-devel] [PATCH for-4.0 16/17] tcg/arm: Force qemu_ld/st arguments into fixed registers, Richard Henderson, 2018/11/12
- [Qemu-devel] [PATCH for-4.0 15/17] tcg/arm: Reduce the number of temps for tcg_out_tlb_read, Richard Henderson, 2018/11/12
- [Qemu-devel] [PATCH for-4.0 17/17] tcg/arm: Use TCG_TARGET_NEED_LDST_OOL_LABELS, Richard Henderson, 2018/11/12
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, no-reply, 2018/11/13
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/13
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Richard Henderson, 2018/11/15
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/15
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Richard Henderson, 2018/11/15
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line,
Richard Henderson <=
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/15
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/16
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Richard Henderson, 2018/11/16
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/16
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Richard Henderson, 2018/11/16
- Re: [Qemu-devel] [PATCH for-4.0 00/17] tcg: Move softmmu out-of-line, Emilio G. Cota, 2018/11/16