On Wed, Apr 19, 2023 at 3:38 AM Zhi Wang <zhi.wang.linux@xxxxxxxxx> wrote:
>
> On Tue, 18 Apr 2023 19:09:04 +0000
> Vishal Annapurve <vannapurve@xxxxxxxxxx> wrote:
>
> > > +static int tdx_map_gpa(struct kvm_vcpu *vcpu)
> > > +{
> > > + struct kvm *kvm = vcpu->kvm;
> > > + gpa_t gpa = tdvmcall_a0_read(vcpu);
> > > + gpa_t size = tdvmcall_a1_read(vcpu);
> > > + gpa_t end = gpa + size;
> > > +
> > > + if (!IS_ALIGNED(gpa, PAGE_SIZE) || !IS_ALIGNED(size, PAGE_SIZE) ||
> > > + end < gpa ||
> > > + end > kvm_gfn_shared_mask(kvm) << (PAGE_SHIFT + 1) ||
> > > + kvm_is_private_gpa(kvm, gpa) != kvm_is_private_gpa(kvm, end)) {
> > > + tdvmcall_set_return_code(vcpu, TDG_VP_VMCALL_INVALID_OPERAND);
> > > + return 1;
> > > + }
> > > +
> > > + return tdx_vp_vmcall_to_user(vcpu);
> >
> > This will result into exits to userspace for MMIO regions as well. Does it make
> > sense to only exit to userspace for guest physical memory regions backed by
> > memslots?
> >
> I think this is necessary as when passing a PCI device to a TD, the guest needs to convert a MMIO region from private to shared, which is not backed by memslots.
KVM could internally handle conversion of regions not backed by
private memslots instead of exiting to userspace. This could save time
during guest boot process.
What would be the expectations from userspace for handling mapgpa
operations on MMIO regions? Is it to just convert memory attributes?
> > > +}
> > > +
>
