Re: [PATCH 18/33] drm/i915: Use VMA as the primary object for context state

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On ke, 2016-08-10 at 09:25 +0100, Chris Wilson wrote:
> On Wed, Aug 10, 2016 at 11:03:39AM +0300, Joonas Lahtinen wrote:
> > 
> > On su, 2016-08-07 at 15:45 +0100, Chris Wilson wrote:
> > > 
> > > -	if (!i915_gem_obj_ggtt_bound(ctx_obj))
> > > -		seq_puts(m, "\tNot bound in GGTT\n");
> > > -	else
> > > -		ggtt_offset = i915_gem_obj_ggtt_offset(ctx_obj);
> > > +	if (vma->flags & I915_VMA_GLOBAL_BIND)
> > > +		seq_printf(m, "\tBound in GGTT at 0x%x\n",
> > 0x%04x?
> You mean 0x08.
> 

Yep.

This was again rather complex to review, different things being mixed;

Reviewed-by: Joonas Lahtinen <joonas.lahtinen@xxxxxxxxxxxxxxx>

Regards, Joonas
-- 
Joonas Lahtinen
Open Source Technology Center
Intel Corporation
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux