[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 06/11] tcg: Introduce tb_mark_invalid() and t
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH v3 06/11] tcg: Introduce tb_mark_invalid() and tb_is_invalid() |
Date: |
Thu, 14 Jul 2016 15:12:33 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.1.1 |
On 14/07/2016 15:00, Sergey Fedorov wrote:
> > > +}
> > > +
> > > +static inline bool tb_is_invalid(TranslationBlock *tb)
> > > +{
> > > + return cpu_tb_cpu_state_is_invalidated(tb->pc, tb->cs_base,
> > > tb->flags);
> > > +}
> > Also why are we passing three pointers to parts of TranslationBlock? Why
> > not just pass tb directly and be done with it?
>
> I'm not sure we want to include exec/exec-all.h in target-*/cpu.h
We don't, exec/exec-all.h is TCG-specific while cpu.h isn't.
Implementing tb_mark_invalid/tb_is_invalid in target-* without the
indirection would be possible, but it would require splitting that out
into a new header such as target-*/exec.h.
Paolo
[Qemu-devel] [PATCH v3 11/11] tcg: Merge tb_find_slow() and tb_find_fast(), Sergey Fedorov, 2016/07/12
[Qemu-devel] [PATCH v3 10/11] tcg: Avoid bouncing tb_lock between tb_gen_code() and tb_add_jump(), Sergey Fedorov, 2016/07/12
Re: [Qemu-devel] [PATCH v3 00/11] Reduce lock contention on TCG hot-path, Paolo Bonzini, 2016/07/13