On 13/11/2015 11:23, Stefan Hajnoczi wrote: >> > tmp = mr->ops->old_mmio.read[ctz32(size)](mr->opaque, addr); >> > - trace_memory_region_ops_read(mr, addr, tmp, size); >> > + trace_memory_region_ops_read(mr->name, addr, tmp, size); > mr->name may be NULL. There is a memory_region_name() function that > always produces a real string. Perhaps it's best to use it.
It's also useful to have mr in the trace, for use within a debugger. Paolo
