On 16.07.25 14:54, Fuad Tabba wrote:
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
Yeah, KVM_SW_PROTECTED_VM doesn't need the populate function.
--
Cheers,
David / dhildenb
