On 08/27/2013 09:48 AM, Yongbok Kim wrote: > #if 0 > - qemu_log(TARGET_FMT_lx " %d %d => " TARGET_FMT_lx " %d (%d)\n", > + qemu_log(TARGET_FMT_lx " %d %d => %" HWADDR_PRIx " %d (%d)\n", > address, rw, access_type, *physical, *prot, ret); > #endif
While by itself correct, consider converting this to the trace infrastructure (with uint64_t instead of hwaddr). If it's not worth a trace, it's probably not worth keeping at all. r~
