The problem that malc saw with sb16 was a major think-o on my part
with the whole interface. We can't re-use the const sub-arrays of
the original MemoryRegionPortio array because they have the wrong
offset for the MemoryRegion to which it is attached -- the lookup
in find_portio fails. We must adjust the offsets of the old_portio
array to be based against the MemoryRegion.
Which means we can easily eliminate the major complaint that came
with the previous round of comments -- the double PORTIO_END_OF_LIST
and the explicit marking of the ranges. All we require of users
is that the array be sorted by offset.
The entire patch set is at
git://repo.or.cz/qemu/rth.git mem-api-isa
and is of course based on
git://git.kernel.org/pub/scm/virt/kvm/qemu-kvm.git memory/master
Avi, the first two patches are fixes for compile errors in your
tree. They probably ought to be squashed when next you rebase.