exec: Introduce helper to set dirty flags.
This new helper/hook is used in the next patch to add an extra call in a single place. Signed-off-by: Anthony PERARD <anthony.perard@citrix.com> Reviewed-by: Avi Kivity <avi@redhat.com>
This commit is contained in:
		
							parent
							
								
									910b38e4dc
								
							
						
					
					
						commit
						51d7a9eb2b
					
				
							
								
								
									
										52
									
								
								exec.c
								
								
								
								
							
							
						
						
									
										52
									
								
								exec.c
								
								
								
								
							| 
						 | 
					@ -3417,6 +3417,18 @@ int cpu_memory_rw_debug(CPUArchState *env, target_ulong addr,
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
#else
 | 
					#else
 | 
				
			||||||
 | 
					
 | 
				
			||||||
 | 
					static void invalidate_and_set_dirty(target_phys_addr_t addr,
 | 
				
			||||||
 | 
					                                     target_phys_addr_t length)
 | 
				
			||||||
 | 
					{
 | 
				
			||||||
 | 
					    if (!cpu_physical_memory_is_dirty(addr)) {
 | 
				
			||||||
 | 
					        /* invalidate code */
 | 
				
			||||||
 | 
					        tb_invalidate_phys_page_range(addr, addr + length, 0);
 | 
				
			||||||
 | 
					        /* set dirty bit */
 | 
				
			||||||
 | 
					        cpu_physical_memory_set_dirty_flags(addr, (0xff & ~CODE_DIRTY_FLAG));
 | 
				
			||||||
 | 
					    }
 | 
				
			||||||
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
 | 
					void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
 | 
				
			||||||
                            int len, int is_write)
 | 
					                            int len, int is_write)
 | 
				
			||||||
{
 | 
					{
 | 
				
			||||||
| 
						 | 
					@ -3462,13 +3474,7 @@ void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
 | 
				
			||||||
                /* RAM case */
 | 
					                /* RAM case */
 | 
				
			||||||
                ptr = qemu_get_ram_ptr(addr1);
 | 
					                ptr = qemu_get_ram_ptr(addr1);
 | 
				
			||||||
                memcpy(ptr, buf, l);
 | 
					                memcpy(ptr, buf, l);
 | 
				
			||||||
                if (!cpu_physical_memory_is_dirty(addr1)) {
 | 
					                invalidate_and_set_dirty(addr1, l);
 | 
				
			||||||
                    /* invalidate code */
 | 
					 | 
				
			||||||
                    tb_invalidate_phys_page_range(addr1, addr1 + l, 0);
 | 
					 | 
				
			||||||
                    /* set dirty bit */
 | 
					 | 
				
			||||||
                    cpu_physical_memory_set_dirty_flags(
 | 
					 | 
				
			||||||
                        addr1, (0xff & ~CODE_DIRTY_FLAG));
 | 
					 | 
				
			||||||
                }
 | 
					 | 
				
			||||||
                qemu_put_ram_ptr(ptr);
 | 
					                qemu_put_ram_ptr(ptr);
 | 
				
			||||||
            }
 | 
					            }
 | 
				
			||||||
        } else {
 | 
					        } else {
 | 
				
			||||||
| 
						 | 
					@ -3534,13 +3540,7 @@ void cpu_physical_memory_write_rom(target_phys_addr_t addr,
 | 
				
			||||||
            /* ROM/RAM case */
 | 
					            /* ROM/RAM case */
 | 
				
			||||||
            ptr = qemu_get_ram_ptr(addr1);
 | 
					            ptr = qemu_get_ram_ptr(addr1);
 | 
				
			||||||
            memcpy(ptr, buf, l);
 | 
					            memcpy(ptr, buf, l);
 | 
				
			||||||
            if (!cpu_physical_memory_is_dirty(addr1)) {
 | 
					            invalidate_and_set_dirty(addr1, l);
 | 
				
			||||||
                /* invalidate code */
 | 
					 | 
				
			||||||
                tb_invalidate_phys_page_range(addr1, addr1 + l, 0);
 | 
					 | 
				
			||||||
                /* set dirty bit */
 | 
					 | 
				
			||||||
                cpu_physical_memory_set_dirty_flags(
 | 
					 | 
				
			||||||
                    addr1, (0xff & ~CODE_DIRTY_FLAG));
 | 
					 | 
				
			||||||
            }
 | 
					 | 
				
			||||||
            qemu_put_ram_ptr(ptr);
 | 
					            qemu_put_ram_ptr(ptr);
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
        len -= l;
 | 
					        len -= l;
 | 
				
			||||||
| 
						 | 
					@ -3666,13 +3666,7 @@ void cpu_physical_memory_unmap(void *buffer, target_phys_addr_t len,
 | 
				
			||||||
                l = TARGET_PAGE_SIZE;
 | 
					                l = TARGET_PAGE_SIZE;
 | 
				
			||||||
                if (l > access_len)
 | 
					                if (l > access_len)
 | 
				
			||||||
                    l = access_len;
 | 
					                    l = access_len;
 | 
				
			||||||
                if (!cpu_physical_memory_is_dirty(addr1)) {
 | 
					                invalidate_and_set_dirty(addr1, l);
 | 
				
			||||||
                    /* invalidate code */
 | 
					 | 
				
			||||||
                    tb_invalidate_phys_page_range(addr1, addr1 + l, 0);
 | 
					 | 
				
			||||||
                    /* set dirty bit */
 | 
					 | 
				
			||||||
                    cpu_physical_memory_set_dirty_flags(
 | 
					 | 
				
			||||||
                        addr1, (0xff & ~CODE_DIRTY_FLAG));
 | 
					 | 
				
			||||||
                }
 | 
					 | 
				
			||||||
                addr1 += l;
 | 
					                addr1 += l;
 | 
				
			||||||
                access_len -= l;
 | 
					                access_len -= l;
 | 
				
			||||||
            }
 | 
					            }
 | 
				
			||||||
| 
						 | 
					@ -3978,13 +3972,7 @@ static inline void stl_phys_internal(target_phys_addr_t addr, uint32_t val,
 | 
				
			||||||
            stl_p(ptr, val);
 | 
					            stl_p(ptr, val);
 | 
				
			||||||
            break;
 | 
					            break;
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
        if (!cpu_physical_memory_is_dirty(addr1)) {
 | 
					        invalidate_and_set_dirty(addr1, 4);
 | 
				
			||||||
            /* invalidate code */
 | 
					 | 
				
			||||||
            tb_invalidate_phys_page_range(addr1, addr1 + 4, 0);
 | 
					 | 
				
			||||||
            /* set dirty bit */
 | 
					 | 
				
			||||||
            cpu_physical_memory_set_dirty_flags(addr1,
 | 
					 | 
				
			||||||
                (0xff & ~CODE_DIRTY_FLAG));
 | 
					 | 
				
			||||||
        }
 | 
					 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -4051,13 +4039,7 @@ static inline void stw_phys_internal(target_phys_addr_t addr, uint32_t val,
 | 
				
			||||||
            stw_p(ptr, val);
 | 
					            stw_p(ptr, val);
 | 
				
			||||||
            break;
 | 
					            break;
 | 
				
			||||||
        }
 | 
					        }
 | 
				
			||||||
        if (!cpu_physical_memory_is_dirty(addr1)) {
 | 
					        invalidate_and_set_dirty(addr1, 2);
 | 
				
			||||||
            /* invalidate code */
 | 
					 | 
				
			||||||
            tb_invalidate_phys_page_range(addr1, addr1 + 2, 0);
 | 
					 | 
				
			||||||
            /* set dirty bit */
 | 
					 | 
				
			||||||
            cpu_physical_memory_set_dirty_flags(addr1,
 | 
					 | 
				
			||||||
                (0xff & ~CODE_DIRTY_FLAG));
 | 
					 | 
				
			||||||
        }
 | 
					 | 
				
			||||||
    }
 | 
					    }
 | 
				
			||||||
}
 | 
					}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in New Issue