Quoting Tvrtko Ursulin (2018-02-20 10:47:42) > From: Tvrtko Ursulin <[email protected]> > > Commit fe49789fab97 ("drm/i915: Deconstruct execute fence") re-arranged > the code and moved the i915_gem_request_execute tracepoint to before the > global seqno is assigned to the request. > > We need to move the tracepoint a bit later so this information is once > again available.
Ok, iirc my thinking was just to have the tracing early to show the entry to i915_request_submit. If you are happy with having it at the end with the details filled in, so am I :) > Signed-off-by: Tvrtko Ursulin <[email protected]> > Fixes: fe49789fab97 ("drm/i915: Deconstruct execute fence") > Cc: Chris Wilson <[email protected]> > Cc: Tvrtko Ursulin <[email protected]> > Cc: Jani Nikula <[email protected]> > Cc: Joonas Lahtinen <[email protected]> > Cc: Rodrigo Vivi <[email protected]> > Cc: [email protected] Reviewed-by: Chris Wilson <[email protected]> -Chris _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
