Re: [Qemu-devel] [PATCH 17/21] introduce memory_region_get_address() and use it in kvm/ioapic

[Peter Maydell]

On 23 April 2013 22:46, Paolo Bonzini <address@hidden> wrote:
> Il 23/04/2013 23:39, Peter Maydell ha scritto:

>> and our new function is
>>  MemoryRegionSection address_space_find_region(AddressSpace *as,
>>                                                MemoryRegion *mr);
>>
>> ?
>>
>> (we don't have to do the tidyup first, but the new function
>> signature makes more sense viewed in the light of the others)
>
> I can look at it for 1.6, since I've already forward ported Avi's IOMMU
> patches and added s/DMAContext/AddressSpace/ on top.
>
> What about 1.5 though?

Ah, I was going to ask if this was a 1.5 series. If so, maybe
just add and use that new function? Anyway, I don't want to
bikeshed the function too much, as long as it takes an
AddressSpace* to search and lets the caller distinguish
"couldn't find the region" from valid actual addresses.

thanks
-- PMM