On 11/26/18 11:07 AM, Emilio G. Cota wrote: > The main reason why I added the qemu_plugin_insn_append calls > was to avoid reading the instructions twice from guest memory, > because I was worried that doing so might somehow alter the > guest's execution, e.g. what if we read a cross-page instruction, > and both pages mapped to the same TLB entry? We'd end up having > more TLB misses because instrumentation was enabled.
A better solution for this, I think is to change direct calls from cpu_ldl_code(env, pc); to translator_ldl_code(dc_base, env, pc); instead of passing around a new argument separate from DisasContextBase? r~