tcg: Fix occasional TCG broken problem when ldst optimization enabled
authorYeongkyoon Lee <yeongkyoon.lee@samsung.com>
Sat, 23 Mar 2013 09:52:18 +0000 (18:52 +0900)
committerYeongkyoon Lee <yeongkyoon.lee@samsung.com>
Mon, 25 Mar 2013 13:15:43 +0000 (22:15 +0900)
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.

This patch has been already applied to mainstream QEMU.
So it should be overwritten when applying QEMU version more than 1.3 in future.

Signed-off-by: Yeongkyoon Lee <yeongkyoon.lee@samsung.com>
exec.c

diff --git a/exec.c b/exec.c
index a64e70bda00a35aca1974443941687a02af8e7f6..8e6bf12671baebaf4ddb2f1cd44171d08f051990 100644 (file)
--- a/exec.c
+++ b/exec.c
@@ -1386,10 +1386,10 @@ void tb_link_page(TranslationBlock *tb,
 /* 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)code_gen_buffer &&
-            tc_ptr < (uintptr_t)(code_gen_buffer + code_gen_buffer_max_size));
+            tc_ptr < (uintptr_t)(code_gen_buffer + code_gen_buffer_size));
 }
 #endif