On Wed, Aug 10, 2016 at 10:29:59AM +0300, Joonas Lahtinen wrote:
> On su, 2016-08-07 at 15:45 +0100, Chris Wilson wrote:
> > if (obj->pin_display) {
> > - mappable_size += i915_gem_obj_ggtt_size(obj);
> > - ++mappable_count;
> > + pin_size += obj->base.size;
> > + ++pin_count;
>
> variables names to form pin_display_*
No.
> > + seq_printf(m, "%u mapped objects, %llu bytes\n",
> > + mapped_count, mapped_size);
> > + seq_printf(m, "%u pinned objects, %llu bytes\n",
>
> "display pinned objects"
Disagree as well.
-Chris
--
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
