Hi Xiaoyao, <snip> > Not really the same result. > > The two-step patches I proposed doesn't produce the below thing of this > original patch. It doesn't make sense to select > KVM_GENERIC_GMEM_POPULATE for KVM_SW_PROTECTED_VM from the name. I think I see where you're going. That said, other than in the configuration files, in all the actual code, the purpose of KVM_GENERIC_PRIVATE_MEMis to guard kvm_gmem_populate(). So I disagree with you, and I think that this should be one patch that fixes the name. That said, I agree with you regarding: - select KVM_GENERIC_PRIVATE_MEM if KVM_SW_PROTECTED_VM + select KVM_GENERIC_MEMORY_ATTRIBUTES if KVM_SW_PROTECTED_VM + select KVM_GMEM if KVM_SW_PROTECTED_VM Cheers, /fuad
