diff options
| author | Paolo Bonzini <pbonzini@redhat.com> | 2016-07-19 08:36:18 +0200 |
|---|---|---|
| committer | Paolo Bonzini <pbonzini@redhat.com> | 2016-09-13 19:08:43 +0200 |
| commit | 6d21e4208f382dd8ca1f7995a6dd9ea7ca281163 (patch) | |
| tree | 81569f437586f0525ac788a09826af4e0c8020bb /translate-all.c | |
| parent | 118b07308a8cedc16ef63d7ab243a95f1701db40 (diff) | |
| download | qemu-6d21e4208f382dd8ca1f7995a6dd9ea7ca281163.tar.gz qemu-6d21e4208f382dd8ca1f7995a6dd9ea7ca281163.tar.bz2 qemu-6d21e4208f382dd8ca1f7995a6dd9ea7ca281163.zip | |
tcg: Prepare TB invalidation for lockless TB lookup
When invalidating a translation block, set an invalid flag into the
TranslationBlock structure first. It is also necessary to check whether
the target TB is still valid after acquiring 'tb_lock' but before calling
tb_add_jump() since TB lookup is to be performed out of 'tb_lock' in
future. Note that we don't have to check 'last_tb'; an already invalidated
TB will not be executed anyway and it is thus safe to patch it.
Suggested-by: Sergey Fedorov <serge.fdrv@gmail.com>
Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
Diffstat (limited to 'translate-all.c')
| -rw-r--r-- | translate-all.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/translate-all.c b/translate-all.c index e753a50640..5a5499ffb6 100644 --- a/translate-all.c +++ b/translate-all.c @@ -773,6 +773,7 @@ static TranslationBlock *tb_alloc(target_ulong pc) tb = &tcg_ctx.tb_ctx.tbs[tcg_ctx.tb_ctx.nb_tbs++]; tb->pc = pc; tb->cflags = 0; + tb->invalid = false; return tb; } @@ -994,6 +995,8 @@ void tb_phys_invalidate(TranslationBlock *tb, tb_page_addr_t page_addr) uint32_t h; tb_page_addr_t phys_pc; + atomic_set(&tb->invalid, true); + /* remove the TB from the hash list */ phys_pc = tb->page_addr[0] + (tb->pc & ~TARGET_PAGE_MASK); h = tb_hash_func(phys_pc, tb->pc, tb->flags); |