On 18/07/2016 19:25, Sergey Fedorov wrote: >> > @@ -753,14 +753,14 @@ static inline void >> > cpu_get_invalid_tb_cpu_state(target_ulong *pc, >> > target_ulong *cs_base, >> > uint32_t *flags) >> > { >> > - *cs_base = -1; /* npc must be a multible of 4 */ >> > + *flags = TB_FLAG_MMU_MASK; >> > } > Hmm, not sure if it is really simpler to follow. Maybe " |= 1;" anyway?
|= 1 has the problem that tb_mark_invalid doesn't pass TB's tuple into cpu_get_invalid_tb_cpu_state, and I didn't want to change that. I'll add a comment, /* TB_FLAG_MMU_MASK is not a valid MMU index, which makes it is an * impossible flag combination for valid TBs. */ Paolo