On Tue, Jun 30, 2015 at 01:53:39PM +0100, Tvrtko Ursulin wrote: > From: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > > Currently only normal views were accounted which under-accounts > the usage as reported in debugfs and by the GET_APERTURE ioctl. > > Introduce new helper, i915_gem_obj_total_ggtt_size, and use it > from call sites which want to know how much GGTT space are > objects using. > > v2: Single loop in i915_gem_get_aperture_ioctl. (Chris Wilson) > > v3: Walk GGTT active/inactive lists in i915_gem_get_aperture_ioctl > for better efficiency. (Chris Wilson, Daniel Vetter) > > Signed-off-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> > Cc: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx> > Cc: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > Cc: Daniel Vetter <daniel.vetter@xxxxxxxx> One final request, and then Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> Please make the total_gtt_size() static to i915_debugfs.c, I don't want any new users :) > --- > Sorry guys for a period I was convinced "mm.bound_list" is actually > "vm.bound_list". :) Sometimes I wish it were. But atm, they are used for two distinct purposes. The global list is for the shrinker, and the active/inactive list is for eviction. All other use is coincidental. -Chris -- Chris Wilson, Intel Open Source Technology Centre _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/intel-gfx
