Quoting Tvrtko Ursulin (2017-09-12 16:21:06) > > On 12/09/2017 16:07, Chris Wilson wrote: > > Following the simplification to a single lookup loop in commit > > 170fa29b14fa ("drm/i915: Simplify eb_lookup_vmas()") and commit > > d1b48c1e7184 ("drm/i915: Replace execbuf vma ht with an idr"), we can go > > one setup further and reorder the error paths so that the state of the > > local variable obj is always known to the compiler and doesn't need the > > uninitialized_var markup to squelch a compiler warning. > > > > Signed-off-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx> > > Cc: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> [snip] > Looks good to me. > > Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin@xxxxxxxxx> Ta, and pushed. -Chris _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx