[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_off
From: |
Avi Kivity |
Subject: |
Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio |
Date: |
Sun, 18 Sep 2011 19:46:28 +0300 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:6.0.2) Gecko/20110906 Thunderbird/6.0.2 |
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<address@hidden>
>>
>> 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
- [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Jan Kiszka, 2011/09/18
- [Qemu-devel] [PATCH] memory: Eliminate region offset, Jan Kiszka, 2011/09/18
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Avi Kivity, 2011/09/18
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Jan Kiszka, 2011/09/18
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio,
Avi Kivity <=
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Jan Kiszka, 2011/09/18
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Avi Kivity, 2011/09/19
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Jan Kiszka, 2011/09/19
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Avi Kivity, 2011/09/19
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Jan Kiszka, 2011/09/19
- Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Avi Kivity, 2011/09/19
Re: [Qemu-devel] [PATCH] isa: Avoid using obsolete memory_region_set_offset for old portio, Richard Henderson, 2011/09/18