Joerg Roedel wrote: >>> >>> >> This still has the same issue as the previous patchset: if the guest >> enables some other bit >> in MSR_IA32_DEBUCTLMSR, we silently ignore it. We should either pr_unimpl() >> on such bits or >> not handle them (ultimately injecting a #GP). >> > > Thats not true. The patch saves the MSR value in vmcb->save.dbgctl. This > value is returned on reads of that MSR. So no bit is ignored. This value > in the VMCB is also used as the guests copy of that MSR if LBR > virtualization is enabled.
Right, my mistake. -- Any sufficiently difficult bug is indistinguishable from a feature. ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ kvm-devel mailing list kvm-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/kvm-devel