On 11/29/2011 06:19 PM, Andreas Färber wrote: > Am 29.11.2011 15:00, schrieb Avi Kivity: > > On 11/29/2011 02:47 PM, Andreas Färber wrote: > >>> > >>> addr += mmio->region_offset[idx]; > >>> idx = mmio->sub_io_index[idx]; > >>> + if (unlikely(idx == IO_MEM_RAM)) { > >>> + ram_addr_t raddr = /*mmio->base |*/ addr; > >>> + void *ptr = qemu_get_ram_ptr(raddr); > >> > >> ...and/or this seems to lead to "Bad RAM pointer" (or so) when there's > >> ELF code loaded into the subpage at that address despite being IO_MEM_RAM? > >> (Seen, e.g., if for RL78 I increase the page size from 12 to 16.) > >> > > > > addr is relative to the start of the page, instead of the start of the > > region. You can use mmio->region_offset[idx] to compensate. > > That's exactly what the first line of the snippet above does, no? > (sorry, there's still a confusing comment from an earlier attempt) >
Depends on whether ->region_offset has the right value or not, where's the code that adjusts it? For your case that does f00-fff it should be -0xf00 + ram_addr (of the start of the region). -- error compiling committee.c: too many arguments to function