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