On 28/03/2015 19:58, Michael S. Tsirkin wrote: >> > +void memory_region_ram_resize(MemoryRegion *mr, ram_addr_t newsize, Error >> > **errp) >> > +{ >> > + assert(mr->terminates); > Why? Does "terminates" guarantee that ram_addr > is valid? In any case, I think this needs a comment.
More or less, but not really; I was just cutting/pasting it from other functions that use ram_addr. I will clean it up. Paolo