On 09/18/2011 07:29 PM, Jan Kiszka wrote:
On 2011-09-18 17:57, Avi Kivity wrote:
> On 09/18/2011 03:54 PM, Jan Kiszka wrote:
>> From: Jan Kiszka<jan.kis...@siemens.com>
>>
>> We can express the offset of old portio completely via
>> MemoryRegionPortio::offset by splitting up regions of different offsets
>> and adjusting those offsets appropriately.
>
> Please split into two patches - core and isa.
They depend on each other.
How can memory.c depend on isa.c?
If you make the core patch add both mr->offset and mrp->offset, then
change isa to drop memory_region_set_offset(), instead adding the delta
to mrp->offset, does that not work out?
> And I
> don't want to remove memory_region_set_offset() until everything (that
> can potentially use it, at least) has been converted.
IMO it's easier to fix those potential users before converting them. You
need to review them anyway to decide if an offset might be needed, and
which one precisely.
Are you aware of any candidates? For PIO, there should be none now.
For pio, none, but mmio has some:
hw/sh7750.c: cpu_register_physical_memory_offset(0x1f000000, 0x1000,
hw/sh7750.c: cpu_register_physical_memory_offset(0xff000000, 0x1000,
hw/sh7750.c: cpu_register_physical_memory_offset(0x1f800000, 0x1000,
hw/sh7750.c: cpu_register_physical_memory_offset(0xff800000, 0x1000,
hw/sh7750.c: cpu_register_physical_memory_offset(0x1fc00000, 0x1000,
hw/sh7750.c: cpu_register_physical_memory_offset(0xffc00000, 0x1000,
hw/sh_intc.c: cpu_register_physical_memory_offset(P4ADDR(address), 4,
hw/sh_intc.c: cpu_register_physical_memory_offset(A7ADDR(address), 4,
--
error compiling committee.c: too many arguments to function
