Skip to contentSkip to conversation

exec: Handle registrations of the entire address space

PATCH New Edgar E. Iglesias 1 participant 1 message Apr 5, 2011

lore
Share & tools
lore

Edgar E. Iglesias

author
#0 lore raw

PATCH 1/1 exec: Handle registrations of the entire address space

Hi,

While experimenting with some odd stuff I ran into trouble when registering the entire address space. The following patch fixes the problem for me.

Cheers

commit f9a9a0d305ff774efd289ab88be427d1da05edfb

Author: Edgar E. Iglesias <[email protected]>
Date:   Tue Apr 5 13:00:36 2011 +0200

exec: Handle registrations of the entire address space

Signed-off-by Edgar E. Iglesias <[email protected]>
exec.c +6 -2
index 964ce31..983c0db 100644
@@ -2611,6 +2611,7 @@ void cpu_register_physical_memory_offset(target_phys_addr_t start_addr,
     ram_addr_t orig_size = size;
     subpage_t *subpage;
 
+    assert(size);
     cpu_notify_set_memory(start_addr, size, phys_offset);
 
     if (phys_offset == IO_MEM_UNASSIGNED) {

@@ -2619,7 +2620,9 @@ void cpu_register_physical_memory_offset(target_phys_addr_t start_addr,
     region_offset &= TARGET_PAGE_MASK;
     size = (size + TARGET_PAGE_SIZE - 1) & TARGET_PAGE_MASK;
     end_addr = start_addr + (target_phys_addr_t)size;
-    for(addr = start_addr; addr != end_addr; addr += TARGET_PAGE_SIZE) {
+
+    addr = start_addr;
+    do {
         p = phys_page_find(addr >> TARGET_PAGE_BITS);
         if (p && p->phys_offset != IO_MEM_UNASSIGNED) {
             ram_addr_t orig_memory = p->phys_offset;

@@ -2671,7 +2674,8 @@ void cpu_register_physical_memory_offset(target_phys_addr_t start_addr,
             }
         }
         region_offset += TARGET_PAGE_SIZE;
-    }
+        addr += TARGET_PAGE_SIZE;
+    } while (addr != end_addr);
 
     /* since each CPU stores ram addresses in its TLB cache, we must
        reset the modified entries */