Quoting Andi Shyti (2020-07-05 23:01:29) > Hi Chris, > > > +static int gen6_fixup_ggtt(struct i915_vma *vma) > > you create this function here and remove it in patch 21. This > series is a bit confusing, can we have a final version of it? It get's removed because the next patches reorder all the pinning around this central function. Until that occurs, the fixup occurs after we do the pinning. And those patches depend on this to provide the central pinning. So it's a circular dependency and this patch needs to provide the fixup so that it works by itself. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
