Re: [PATCH 1/4] kvm: nVMX: Add support for "VMWRITE to any supported field"

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 02/05/2018 17:52, Jim Mattson wrote:
>>> vmcs_field_readonly is now unused.  With that removed,
>>>
>>> Reviewed-by: Paolo Bonzini <pbonzini@xxxxxxxxxx>
>> Actually, no.  The error must be kept if the host has disabled the
>> feature with a KVM_SET_MSR ioctl for MSR_IA32_VMX_MISC.
>>
> Coming back to this patch set...
> 
> Is there a good reason to allow userspace to clear
> MSR_IA32_VMX_MISC_VMWRITE_SHADOW_RO_FIELDS? It's not yet possible to
> migrate a VMX-capable VM (though I think that ability is imminent), so
> there are no compatibility issues. Why not just force this on?

I suppose that can be done, yes.  The only reason is that silicon ties
that bit to the availability of shadow VMCS, but we can certainly do
things differently than actual processors.

Technically it would be a small userspace API break, but we can live
with it.

Paolo



[Index of Archives]     [KVM ARM]     [KVM ia64]     [KVM ppc]     [Virtualization Tools]     [Spice Development]     [Libvirt]     [Libvirt Users]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite Questions]     [Linux Kernel]     [Linux SCSI]     [XFree86]

  Powered by Linux