Quoting Tvrtko Ursulin (2021-01-26 17:13:24) > > > On 06/01/2020 23:47, Matthew Auld wrote: > > Attempt to split i915_gem_gtt.[ch] into more manageable chunks. > > > +void clear_pages(struct i915_vma *vma) > > +{ > > + GEM_BUG_ON(!vma->pages); > > + > > + if (vma->pages != vma->obj->mm.pages) { > > + sg_free_table(vma->pages); > > + kfree(vma->pages); > > + } > > + vma->pages = NULL; > > + > > + memset(&vma->page_sizes, 0, sizeof(vma->page_sizes)); > > +} > > We probably want to add a prefix now that the function is exported. Or follow through on eliminating the callback. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx