Quoting Jani Nikula (2019-11-25 10:51:53) > On Thu, 21 Nov 2019, Michal Wajdeczko <michal.wajdec...@intel.com> wrote: > > Today VGT implements GGTT ballooning on its own, using some private > > static structure to hold info about reserved GGTT nodes and then > > manually update vm.reserved size owned by i915_ggtt. > > > > As i915_ggtt already manages some custom reserved nodes (like uc_fw) > > move VGT ballooning support to i915_ggtt and make it more generic > > for possible future reuse in other scenarios. > > > > As a bonus we can drop another place in driver that uses static data. > > > > Signed-off-by: Michal Wajdeczko <michal.wajdec...@intel.com> > > Cc: Xiong Zhang <xiong.y.zh...@intel.com> > > Cc: Chris Wilson <ch...@chris-wilson.co.uk> > > Cc: Jani Nikula <jani.nik...@intel.com> > > --- > > drivers/gpu/drm/i915/i915_gem_gtt.c | 47 +++++++++++++++++++ > > drivers/gpu/drm/i915/i915_gem_gtt.h | 18 ++++++++ > > Maybe not part of this patch, but should we start moving this stuff out > of the top level instead of addming more?
It's on my todo list to split this between the HW pagetable management and GEM and other interfaces on top. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
