On Tue, Nov 08, 2016 at 09:43:21AM +0200, Joonas Lahtinen wrote:
> On ma, 2016-11-07 at 13:59 +, Chris Wilson wrote:
> > @@ -56,6 +61,24 @@ int i915_gem_timeline_init(struct drm_i915_private *i915,
> > return 0;
> > }
> >
> > +int i915_gem_timeline_init(struct drm_i915_private *i915,
>
On ma, 2016-11-07 at 13:59 +, Chris Wilson wrote:
> @@ -56,6 +61,24 @@ int i915_gem_timeline_init(struct drm_i915_private *i915,
> return 0;
> }
>
> +int i915_gem_timeline_init(struct drm_i915_private *i915,
> + struct i915_gem_timeline *timeline,
> +
In order to simplify the lockdep annotation, as they become more complex
in the future with deferred execution and multiple paths through the
same functions, create a separate lockclass for the user timeline and
the hardware execution timeline.
We should only ever be locking the user timeline and