[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Qemu-stable][PATCH v2] tcg: Fix occasional TCG broken
From: |
Andreas Färber |
Subject: |
Re: [Qemu-devel] [Qemu-stable][PATCH v2] tcg: Fix occasional TCG broken problem when ldst optimization enabled |
Date: |
Fri, 22 Mar 2013 13:35:51 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130307 Thunderbird/17.0.4 |
Am 22.03.2013 12:56, schrieb Yeongkyoon Lee:
> is_tcg_gen_code() checks the upper limit of TCG generated code range wrong, so
> that TCG could get broken occasionally only when CONFIG_QEMU_LDST_OPTIMIZATION
> enabled. The reason is code_gen_buffer_max_size does not cover the upper range
> up to (TCG_MAX_OP_SIZE * OPC_BUF_SIZE), thus code_gen_buffer_max_size should
> be
> modified to code_gen_buffer_size.
>
> Signed-off-by: Yeongkyoon Lee <address@hidden>
> ---
>
> Here's is the promised patch with Aurelien Jarno for TCG broken problem, which
> is supposed to be applied to 1.3.x and 1.4.x releases as well as master.
In that case either the author or the committer should add
Cc: address@hidden
to the commit message please.
Andreas
> Thanks to Aurelien Jarno and Stefan Weil.
>
> translate-all.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/translate-all.c b/translate-all.c
> index 1f3237e..72bea9b 100644
> --- a/translate-all.c
> +++ b/translate-all.c
> @@ -1308,11 +1308,11 @@ static void tb_link_page(TranslationBlock *tb,
> tb_page_addr_t phys_pc,
> /* check whether the given addr is in TCG generated code buffer or not */
> bool is_tcg_gen_code(uintptr_t tc_ptr)
> {
> - /* This can be called during code generation, code_gen_buffer_max_size
> + /* This can be called during code generation, code_gen_buffer_size
> is used instead of code_gen_ptr for upper boundary checking */
> return (tc_ptr >= (uintptr_t)tcg_ctx.code_gen_buffer &&
> tc_ptr < (uintptr_t)(tcg_ctx.code_gen_buffer +
> - tcg_ctx.code_gen_buffer_max_size));
> + tcg_ctx.code_gen_buffer_size));
> }
> #endif
>
>
--
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg