On Thu, 06/18 18:47, Paolo Bonzini wrote: [snip]
> diff --git a/memory.c b/memory.c > index 6b77354..be385d4 100644 > --- a/memory.c > +++ b/memory.c > @@ -414,9 +414,6 @@ static MemTxResult > memory_region_read_with_attrs_accessor(MemoryRegion *mr, > uint64_t tmp = 0; > MemTxResult r; > > - if (mr->flush_coalesced_mmio) { > - qemu_flush_coalesced_mmio_buffer(); > - } > r = mr->ops->read_with_attrs(mr->opaque, addr, &tmp, size, attrs); > trace_memory_region_ops_read(mr, addr, tmp, size); > *value |= (tmp & mask) << shift; > @@ -449,9 +446,6 @@ static MemTxResult > memory_region_write_accessor(MemoryRegion *mr, > { > uint64_t tmp; > > - if (mr->flush_coalesced_mmio) { > - qemu_flush_coalesced_mmio_buffer(); > - } > tmp = (*value >> shift) & mask; > trace_memory_region_ops_write(mr, addr, tmp, size); > mr->ops->write(mr->opaque, addr, tmp, size); Why are memory_region_read_accessor and memory_region_write_with_attrs_accessor unchanged? Fam