On 2018.07.12 20:36:03 +0000, Bloomfield, Jon wrote: > > -----Original Message----- > > From: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Sent: Thursday, July 12, 2018 11:53 AM > > To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx > > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>; Zhenyu Wang > > <zhenyuw@xxxxxxxxxxxxxxx>; Bloomfield, Jon <jon.bloomfield@xxxxxxxxx>; > > Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>; Matthew Auld > > <matthew.william.auld@xxxxxxxxx> > > Subject: [PATCH 3/6] drm/i915/gtt: Disable read-only support under GVT > > > > GVT is not propagating the PTE bits, and is always setting the > > read-write bit, thus breaking read-only support. > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx> > > Cc: Jon Bloomfield <jon.bloomfield@xxxxxxxxx> > > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > > Cc: Matthew Auld <matthew.william.auld@xxxxxxxxx> > > --- > > drivers/gpu/drm/i915/i915_gem_gtt.c | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/gpu/drm/i915/i915_gem_gtt.c > > b/drivers/gpu/drm/i915/i915_gem_gtt.c > > index 6c0b438afe46..8e70a45b8a90 100644 > > --- a/drivers/gpu/drm/i915/i915_gem_gtt.c > > +++ b/drivers/gpu/drm/i915/i915_gem_gtt.c > > @@ -1662,8 +1662,12 @@ static struct i915_hw_ppgtt > > *gen8_ppgtt_create(struct drm_i915_private *i915) > > 1ULL << 48 : > > 1ULL << 32; > > > > - /* From bdw, there is support for read-only pages in the PPGTT */ > > - ppgtt->vm.has_read_only = true; > > + /* > > + * From bdw, there is support for read-only pages in the PPGTT. > > + * > > + * XXX GVT is not setting honouring the PTE bits. > > + */ > > + ppgtt->vm.has_read_only = !intel_vgpu_active(i915); > > > > i915_address_space_init(&ppgtt->vm, i915); > > > > -- > > 2.18.0 > > Is there a blocker that prevents gvt respecting this bit? I can't think of an obvious > reason why it would be a bad thing to support. I don't think any blocker for gvt support, we can respect that bit when shadowing. But we need capability check on host gvt when that support is ready. -- Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
