Proxmox-Port/qemu
1
0
Fork 0
mirror of https://github.com/qemu/qemu.git synced 2025-08-15 22:31:15 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Remove bogus cpu_physical_memory_rw

Browse Source 
Userspace doesn't have physical memory, so cpu_physical_memory_rw
makes no sense.  This is only used to implement cpu_memory_rw_debug, so
just implement that directly instead.

Signed-off-by: Paul Brook <paul@codesourcery.com>
This commit is contained in:
Paul Brook 2010-03-01 00:08:59 +00:00
parent 6d9a13042d
commit a68fe89caf
1 changed files with 9 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
exec.c
View File

@ -3106,8 +3106,8 @@ static void io_mem_init(void)
/* physical memory access (slow version, mainly for debug) */ /* physical memory access (slow version, mainly for debug) */
#if defined(CONFIG_USER_ONLY) #if defined(CONFIG_USER_ONLY)
void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf, int cpu_memory_rw_debug(CPUState *env, target_ulong addr,
int len, int is_write) uint8_t *buf, int len, int is_write)
{ {
int l, flags; int l, flags;
target_ulong page; target_ulong page;
@ -3120,23 +3120,21 @@ void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
l = len; l = len;
flags = page_get_flags(page); flags = page_get_flags(page);
if (!(flags & PAGE_VALID)) if (!(flags & PAGE_VALID))
return; return -1;
if (is_write) { if (is_write) {
if (!(flags & PAGE_WRITE)) if (!(flags & PAGE_WRITE))
return; return -1;
/* XXX: this code should not depend on lock_user */ /* XXX: this code should not depend on lock_user */
if (!(p = lock_user(VERIFY_WRITE, addr, l, 0))) if (!(p = lock_user(VERIFY_WRITE, addr, l, 0)))
/* FIXME - should this return an error rather than just fail? */ return -1;
return;
memcpy(p, buf, l); memcpy(p, buf, l);
unlock_user(p, addr, l); unlock_user(p, addr, l);
} else { } else {
if (!(flags & PAGE_READ)) if (!(flags & PAGE_READ))
return; return -1;
/* XXX: this code should not depend on lock_user */ /* XXX: this code should not depend on lock_user */
if (!(p = lock_user(VERIFY_READ, addr, l, 1))) if (!(p = lock_user(VERIFY_READ, addr, l, 1)))
/* FIXME - should this return an error rather than just fail? */ return -1;
return;
memcpy(buf, p, l); memcpy(buf, p, l);
unlock_user(p, addr, 0); unlock_user(p, addr, 0);
} }
@ -3144,6 +3142,7 @@ void cpu_physical_memory_rw(target_phys_addr_t addr, uint8_t *buf,
buf += l; buf += l;
addr += l; addr += l;
} }
return 0;
} }
#else #else
@ -3641,8 +3640,6 @@ void stq_phys(target_phys_addr_t addr, uint64_t val)
cpu_physical_memory_write(addr, (const uint8_t *)&val, 8); cpu_physical_memory_write(addr, (const uint8_t *)&val, 8);
} }
#endif
/* virtual memory access for debug (includes writing to ROM) */ /* virtual memory access for debug (includes writing to ROM) */
int cpu_memory_rw_debug(CPUState *env, target_ulong addr, int cpu_memory_rw_debug(CPUState *env, target_ulong addr,
uint8_t *buf, int len, int is_write) uint8_t *buf, int len, int is_write)
@ -3661,11 +3658,9 @@ int cpu_memory_rw_debug(CPUState *env, target_ulong addr,
if (l > len) if (l > len)
l = len; l = len;
phys_addr += (addr & ~TARGET_PAGE_MASK); phys_addr += (addr & ~TARGET_PAGE_MASK);
#if !defined(CONFIG_USER_ONLY)
if (is_write) if (is_write)
cpu_physical_memory_write_rom(phys_addr, buf, l); cpu_physical_memory_write_rom(phys_addr, buf, l);
else else
#endif
cpu_physical_memory_rw(phys_addr, buf, l, is_write); cpu_physical_memory_rw(phys_addr, buf, l, is_write);
len -= l; len -= l;
buf += l; buf += l;
@ -3673,6 +3668,7 @@ int cpu_memory_rw_debug(CPUState *env, target_ulong addr,
} }
return 0; return 0;
} }
#endif
/* in deterministic execution mode, instructions doing device I/Os /* in deterministic execution mode, instructions doing device I/Os
must be at the end of the TB */ must be at the end of the TB */