On 11/09/24 03:19, Edgecombe, Rick P wrote:
> On Tue, 2024-09-10 at 12:24 +0200, Paolo Bonzini wrote:
>> On 9/4/24 05:07, Rick Edgecombe wrote:
>>> +static int tdx_mem_page_record_premap_cnt(struct kvm *kvm, gfn_t gfn,
>>> + enum pg_level level, kvm_pfn_t
>>> pfn)
>>> +{
>>> + struct kvm_tdx *kvm_tdx = to_kvm_tdx(kvm);
>>> +
>>> + /* Returning error here to let TDP MMU bail out early. */
>>> + if (KVM_BUG_ON(level != PG_LEVEL_4K, kvm)) {
>>> + tdx_unpin(kvm, pfn);
>>> + return -EINVAL;
>>> + }
>>
>> Should this "if" already be part of patch 14, and in
>> tdx_sept_set_private_spte() rather than tdx_mem_page_record_premap_cnt()?
>
> Hmm, makes sense to me. Thanks.
It is already in patch 14, so just remove it from this patch
presumably.
