diff options
author | Juan Quintela | 2011-06-02 03:53:44 +0200 |
---|---|---|
committer | Stefan Hajnoczi | 2011-06-24 16:34:52 +0200 |
commit | 4429ab4419609f3a0b470e4b15c37c6cb5a6a400 (patch) | |
tree | f7143cc43e39295631ad69e8a41a0ab3a4b35b13 /exec.c | |
parent | w32: Remove redundant definitions of PRI*64 (diff) | |
download | qemu-4429ab4419609f3a0b470e4b15c37c6cb5a6a400.tar.gz qemu-4429ab4419609f3a0b470e4b15c37c6cb5a6a400.tar.xz qemu-4429ab4419609f3a0b470e4b15c37c6cb5a6a400.zip |
exec: last_first_tb was only used in !ONLY_USER case
Once there, use a better variable name.
Signed-off-by: Juan Quintela <quintela@redhat.com>
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
Signed-off-by: Stefan Hajnoczi <stefanha@linux.vnet.ibm.com>
Diffstat (limited to 'exec.c')
-rw-r--r-- | exec.c | 10 |
1 files changed, 7 insertions, 3 deletions
@@ -1208,12 +1208,16 @@ static inline void tb_alloc_page(TranslationBlock *tb, unsigned int n, tb_page_addr_t page_addr) { PageDesc *p; - TranslationBlock *last_first_tb; +#ifndef CONFIG_USER_ONLY + bool page_already_protected; +#endif tb->page_addr[n] = page_addr; p = page_find_alloc(page_addr >> TARGET_PAGE_BITS, 1); tb->page_next[n] = p->first_tb; - last_first_tb = p->first_tb; +#ifndef CONFIG_USER_ONLY + page_already_protected = p->first_tb != NULL; +#endif p->first_tb = (TranslationBlock *)((long)tb | n); invalidate_page_bitmap(p); @@ -1249,7 +1253,7 @@ static inline void tb_alloc_page(TranslationBlock *tb, /* if some code is already present, then the pages are already protected. So we handle the case where only the first TB is allocated in a physical page */ - if (!last_first_tb) { + if (!page_already_protected) { tlb_protect_code(page_addr); } #endif |