[Intel-gfx] [PATCH 32/43] drm/i915: Move shmem object setup to its own file

Chris Wilson chris at chris-wilson.co.uk
Wed Mar 6 17:24:10 UTC 2019


Quoting Matthew Auld (2019-03-06 17:05:47)
> On 06/03/2019 14:25, Chris Wilson wrote:
> > Split the plain old shmem object into its own file to start decluttering
> > i915_gem.c
> > 
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> 
> [snip]
> 
> > +
> > +const struct drm_i915_gem_object_ops i915_gem_shmem_ops = {
> > +     .flags = I915_GEM_OBJECT_HAS_STRUCT_PAGE |
> > +              I915_GEM_OBJECT_IS_SHRINKABLE,
> > +
> > +     .get_pages = shmem_get_pages_gtt,
> > +     .put_pages = shmem_put_pages_gtt,
> > +
> > +     .pwrite = shmem_pwrite_gtt,
> 
> I never really understood the _gtt thing here...maybe because of the 
> prepare_gtt?

Historical accident, and prepare_gtt is pretty much the reason why, but
mostly I recall it being !phys. I guess now would be a perfect time to
add a drop of consistency and drop the _gtt suffix.
-Chris


More information about the Intel-gfx mailing list