v2: - fix the issue that patchew reported - further refactor patch 2 to make it better
Mainly I am doing the translation and then we can get rid of that. Finally we won't need to recompile for its debugging any more. I did spend some time on considering which DPRINTF to remove, which one to convert to trace_*, also I am more careful this time with the naming, e.g., I named all the error cases with trace_vtd_err_*, so that we can easily get them when as long as we have a HMP interface, using: trace-event vtd_err* on The next two patches are simple cleanups. Please review. Thanks. (I really want this series to be in asap... I should just do this months ago) Peter Xu (3): intel_iommu: switching the rest DPRINTF to trace intel_iommu: cleanup vtd_{do_}iommu_translate() intel_iommu: cleanup vtd_interrupt_remap_msi() hw/i386/intel_iommu.c | 421 +++++++++++++++-------------------------- hw/i386/intel_iommu_internal.h | 1 + hw/i386/trace-events | 44 +++++ 3 files changed, 196 insertions(+), 270 deletions(-) -- 2.7.4