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

Reply via email to