exec: make io_mem_unassigned private
There is no reason to avoid a recompile before accessing unassigned memory. In the end it will be treated as MMIO anyway. Reviewed-by: Richard Henderson <rth@twiddle.net> Signed-off-by: Paolo Bonzini <pbonzini@redhat.com>
This commit is contained in:
		
							parent
							
								
									8f3e03cb73
								
							
						
					
					
						commit
						0844e00762
					
				
							
								
								
									
										4
									
								
								exec.c
									
									
									
									
									
								
							
							
						
						
									
										4
									
								
								exec.c
									
									
									
									
									
								
							@ -66,8 +66,8 @@ AddressSpace address_space_io;
 | 
				
			|||||||
AddressSpace address_space_memory;
 | 
					AddressSpace address_space_memory;
 | 
				
			||||||
DMAContext dma_context_memory;
 | 
					DMAContext dma_context_memory;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
MemoryRegion io_mem_rom, io_mem_unassigned, io_mem_notdirty;
 | 
					MemoryRegion io_mem_rom, io_mem_notdirty;
 | 
				
			||||||
static MemoryRegion io_mem_subpage_ram;
 | 
					static MemoryRegion io_mem_unassigned, io_mem_subpage_ram;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
@ -111,7 +111,6 @@ void cpu_physical_memory_write_rom(hwaddr addr,
 | 
				
			|||||||
                                   const uint8_t *buf, int len);
 | 
					                                   const uint8_t *buf, int len);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
extern struct MemoryRegion io_mem_rom;
 | 
					extern struct MemoryRegion io_mem_rom;
 | 
				
			||||||
extern struct MemoryRegion io_mem_unassigned;
 | 
					 | 
				
			||||||
extern struct MemoryRegion io_mem_notdirty;
 | 
					extern struct MemoryRegion io_mem_notdirty;
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#endif
 | 
					#endif
 | 
				
			||||||
 | 
				
			|||||||
@ -68,10 +68,7 @@ static inline DATA_TYPE glue(io_read, SUFFIX)(CPUArchState *env,
 | 
				
			|||||||
 | 
					
 | 
				
			||||||
    physaddr = (physaddr & TARGET_PAGE_MASK) + addr;
 | 
					    physaddr = (physaddr & TARGET_PAGE_MASK) + addr;
 | 
				
			||||||
    env->mem_io_pc = retaddr;
 | 
					    env->mem_io_pc = retaddr;
 | 
				
			||||||
    if (mr != &io_mem_rom
 | 
					    if (mr != &io_mem_rom && mr != &io_mem_notdirty && !can_do_io(env)) {
 | 
				
			||||||
        && mr != &io_mem_unassigned
 | 
					 | 
				
			||||||
        && mr != &io_mem_notdirty
 | 
					 | 
				
			||||||
            && !can_do_io(env)) {
 | 
					 | 
				
			||||||
        cpu_io_recompile(env, retaddr);
 | 
					        cpu_io_recompile(env, retaddr);
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
@ -218,10 +215,7 @@ static inline void glue(io_write, SUFFIX)(CPUArchState *env,
 | 
				
			|||||||
    MemoryRegion *mr = iotlb_to_region(physaddr);
 | 
					    MemoryRegion *mr = iotlb_to_region(physaddr);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
    physaddr = (physaddr & TARGET_PAGE_MASK) + addr;
 | 
					    physaddr = (physaddr & TARGET_PAGE_MASK) + addr;
 | 
				
			||||||
    if (mr != &io_mem_rom
 | 
					    if (mr != &io_mem_rom && mr != &io_mem_notdirty && !can_do_io(env)) {
 | 
				
			||||||
        && mr != &io_mem_unassigned
 | 
					 | 
				
			||||||
        && mr != &io_mem_notdirty
 | 
					 | 
				
			||||||
            && !can_do_io(env)) {
 | 
					 | 
				
			||||||
        cpu_io_recompile(env, retaddr);
 | 
					        cpu_io_recompile(env, retaddr);
 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
				
			|||||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user