[Intel-gfx] [PATCH v2 2/5] drm/i915: Add a new "remapped" gtt_view
Ville Syrjälä
ville.syrjala at linux.intel.com
Fri Sep 14 12:58:15 UTC 2018
On Thu, Sep 13, 2018 at 09:19:00PM +0100, Chris Wilson wrote:
> Quoting Ville Syrjala (2018-09-13 21:01:37)
> > diff --git a/drivers/gpu/drm/i915/selftests/i915_vma.c b/drivers/gpu/drm/i915/selftests/i915_vma.c
> > index ffa74290e054..4fc49c27f13c 100644
> > --- a/drivers/gpu/drm/i915/selftests/i915_vma.c
> > +++ b/drivers/gpu/drm/i915/selftests/i915_vma.c
> > @@ -395,8 +395,8 @@ assert_rotated(struct drm_i915_gem_object *obj,
> > return sg;
> > }
> >
> > -static unsigned int rotated_size(const struct intel_rotation_plane_info *a,
> > - const struct intel_rotation_plane_info *b)
> > +static unsigned int rotated_size(const struct intel_remapped_plane_info *a,
> > + const struct intel_remapped_plane_info *b)
> > {
> > return a->width * a->height + b->width * b->height;
> > }
> > @@ -406,7 +406,7 @@ static int igt_vma_rotate(void *arg)
> > struct drm_i915_private *i915 = arg;
> > struct i915_address_space *vm = &i915->ggtt.vm;
> > struct drm_i915_gem_object *obj;
> > - const struct intel_rotation_plane_info planes[] = {
> > + const struct intel_remapped_plane_info planes[] = {
> > { .width = 1, .height = 1, .stride = 1 },
> > { .width = 2, .height = 2, .stride = 2 },
> > { .width = 4, .height = 4, .stride = 4 },
>
> Could we prove our remapping vma works by doing an i915_vma_pin_iomap()
> and checking that a write into each page ends up in the correct address?
So write through a fence and check it lands in the right place?
Seems doable. Though we'd have to allow a fence for these vmas,
which I guess isn't really a problem.
--
Ville Syrjälä
Intel
More information about the Intel-gfx
mailing list