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.
