On to, 2016-07-28 at 12:36 +0100, Chris Wilson wrote: > On Thu, Jul 28, 2016 at 01:38:14PM +0300, Joonas Lahtinen wrote: > > > > On ke, 2016-07-27 at 12:14 +0100, Chris Wilson wrote: > > > > > > Since i915_gem_obj_ggtt_pin() is an idiom breaking curry function for > > > i915_gem_object_ggtt_pin(), spare us the confustion and remove it. > > confustion should be combustion or confusion > > > > > > > > @@ -3741,7 +3742,8 @@ i915_gem_object_ggtt_pin(struct > > > drm_i915_gem_object *obj, > > > struct i915_vma *vma; > > > int ret; > > > > > > - BUG_ON(!view); > > > + if (!view) > > > + view = &i915_ggtt_view_normal; > > > > > Convert other calling sites to take advantage of this if, > Patches have already been sent to do so, and will be along later as part > of the VMA handling outside of execbuf. > > > > > and also fix the error handling; > Hmm, what have I missed? > > > > > "i915_is_ggtt(vm) ? &i915_ggtt_view_normal : NULL" > > is used in i915_gem_object_pin and i915_gem_obj_lookup_or_create_vma > They are queued for removal. > Those two calling sites will rely on the BUG_ON(!view) and pass NULL to trigger it. So the BUG on should be upper level temporarily to help bisecting in future. 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
