On 1 September 2016 at 15:28, Marc Zyngier <marc.zyngier@xxxxxxx> wrote: > On 01/09/16 13:46, Christoffer Dall wrote: >> On Fri, Aug 19, 2016 at 01:38:13PM +0100, Marc Zyngier wrote: >>> + __skip_instr(vcpu); >> >> does this interact in any amusing way with single-step guest debugging? > > Ouch. Good point. Actually, nothing that uses kvm_skip_instr() works for > singlestep/watchpoint either (trapped sysreg, WFx, spurious traps, > MMIO). I guess that's something we need to fix overall. I vaguely remember having conversations about this way way back. All the "advance one insn" stuff (including how the singlestep state machine gets advanced) is probably on the wrong side of "return to userspace". thanks -- PMM -- To unsubscribe from this list: send the line "unsubscribe kvm" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
