sim-card
/
qemu
Archived
10
0
Fork 0
Code Releases Activity

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>
This commit is contained in:
Juan Quintela 2011-06-02 01:53:44 +00:00 committed by Stefan Hajnoczi
parent 6a33e7b609
commit 4429ab4419
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
exec.c
View File

@ -1208,12 +1208,16 @@ static inline void tb_alloc_page(TranslationBlock *tb,
unsigned int n, tb_page_addr_t page_addr) unsigned int n, tb_page_addr_t page_addr)
{ {
PageDesc *p; PageDesc *p;
TranslationBlock *last_first_tb; #ifndef CONFIG_USER_ONLY
bool page_already_protected;
#endif
tb->page_addr[n] = page_addr; tb->page_addr[n] = page_addr;
p = page_find_alloc(page_addr >> TARGET_PAGE_BITS, 1); p = page_find_alloc(page_addr >> TARGET_PAGE_BITS, 1);
tb->page_next[n] = p->first_tb; 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); p->first_tb = (TranslationBlock *)((long)tb | n);
invalidate_page_bitmap(p); 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 /* if some code is already present, then the pages are already
protected. So we handle the case where only the first TB is protected. So we handle the case where only the first TB is
allocated in a physical page */ allocated in a physical page */
if (!last_first_tb) { if (!page_already_protected) {
tlb_protect_code(page_addr); tlb_protect_code(page_addr);
} }
#endif #endif