On 10.08.2017 20:30, Konrad Rzeszutek Wilk wrote: > On Thu, Aug 10, 2017 at 04:42:02PM +0200, Paolo Bonzini wrote: >> On 10/08/2017 15:35, David Hildenbrand wrote: >>> We already always set that type but don't check if it is supported. Also >>> for nVMX, we only support WB for now. Let's just require it. >>> >>> Signed-off-by: David Hildenbrand <david@xxxxxxxxxx> >> >> It may only happen on a broken nested hypervisor, so the patch is okay. > > Is there one in the wild? I think only the following scenario would see a change: Hypervisor supports but doesn't announce the WB memory type for EPT. KVM runs as nested hypervisor. It will now refuse to use EPT but should continue to work without EPT. In this case, the hypervisor would have a BUG and we as a nested hypervisor would do the right thing. >> >> Paolo >> -- Thanks, David
