diff mbox series

[v6,25/50] tcg: Include CF_COUNT_MASK in CF_HASH_MASK

Message ID 20171016172609.23422-26-richard.henderson@linaro.org
State Superseded
Headers show
Series tcg tb_lock removal | expand

Commit Message

Richard Henderson Oct. 16, 2017, 5:25 p.m. UTC 
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>

---
 include/exec/exec-all.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.13.6

Comments

Emilio Cota Oct. 18, 2017, 4:31 a.m. UTC | #1 
On Mon, Oct 16, 2017 at 10:25:44 -0700, Richard Henderson wrote:
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>


Does this patch work with icount for you? On my end, aarch64
stops booting with -icount 1.

		E.
Richard Henderson Oct. 20, 2017, 2:27 a.m. UTC | #2 
On 10/17/2017 09:31 PM, Emilio G. Cota wrote:
> On Mon, Oct 16, 2017 at 10:25:44 -0700, Richard Henderson wrote:

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

> 

> Does this patch work with icount for you? On my end, aarch64

> stops booting with -icount 1.


Found it -- cpu_io_recompile needs the same sort of fix as in patch 24.  I'll
post a v7 shortly.


r~
diff mbox series

Patch

diff --git a/include/exec/exec-all.h b/include/exec/exec-all.h
index 352abc7450..0fdb72bb22 100644
--- a/include/exec/exec-all.h
+++ b/include/exec/exec-all.h
@@ -327,7 +327,7 @@  struct TranslationBlock {
 #define CF_INVALID     0x80000 /* TB is stale. Setters must acquire tb_lock */
 #define CF_PARALLEL    0x100000 /* Generate code for a parallel context */
 /* cflags' mask for hashing/comparison */
-#define CF_HASH_MASK (CF_PARALLEL)
+#define CF_HASH_MASK   (CF_COUNT_MASK | CF_PARALLEL)
 
     /* Per-vCPU dynamic tracing state used to generate this TB */
     uint32_t trace_vcpu_dstate;