[Intel-gfx] [PATCH] drm/i915: De-inline intel_context_init()
Chris Wilson
chris at chris-wilson.co.uk
Thu Jan 24 10:24:51 UTC 2019
Quoting Tvrtko Ursulin (2019-01-24 09:12:34)
>
> On 24/01/2019 08:37, Chris Wilson wrote:
> > Nip some inline spaghetti in the bud before the problem gets too bad.
> >
> > Signed-off-by: Chris Wilson <chris at chris-wilson.co.uk>
> > Cc: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
> > ---
> > drivers/gpu/drm/i915/i915_gem_context.c | 8 ++++++++
> > drivers/gpu/drm/i915/i915_gem_context.h | 7 ++-----
> > 2 files changed, 10 insertions(+), 5 deletions(-)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_gem_context.c b/drivers/gpu/drm/i915/i915_gem_context.c
> > index fae68c4c4683..93e84751370f 100644
> > --- a/drivers/gpu/drm/i915/i915_gem_context.c
> > +++ b/drivers/gpu/drm/i915/i915_gem_context.c
> > @@ -321,6 +321,14 @@ static u32 default_desc_template(const struct drm_i915_private *i915,
> > return desc;
> > }
> >
> > +void
> > +intel_context_init(struct intel_context *ce,
> > + struct i915_gem_context *ctx,
> > + struct intel_engine_cs *engine)
> > +{
> > + ce->gem_context = ctx;
> > +}
> > +
> > static struct i915_gem_context *
> > __create_hw_context(struct drm_i915_private *dev_priv,
> > struct drm_i915_file_private *file_priv)
> > diff --git a/drivers/gpu/drm/i915/i915_gem_context.h b/drivers/gpu/drm/i915/i915_gem_context.h
> > index 47d82ce7ba6a..096d57958247 100644
> > --- a/drivers/gpu/drm/i915/i915_gem_context.h
> > +++ b/drivers/gpu/drm/i915/i915_gem_context.h
> > @@ -364,12 +364,9 @@ static inline void i915_gem_context_put(struct i915_gem_context *ctx)
> > kref_put(&ctx->ref, i915_gem_context_release);
> > }
> >
> > -static inline void
> > +void
> > intel_context_init(struct intel_context *ce,
> > struct i915_gem_context *ctx,
> > - struct intel_engine_cs *engine)
> > -{
> > - ce->gem_context = ctx;
> > -}
> > + struct intel_engine_cs *engine);
> >
> > #endif /* !__I915_GEM_CONTEXT_H__ */
> >
>
> I guess that will be more robust.
>
> Reviewed-by: Tvrtko Ursulin <tvrtko.ursulin at intel.com>
>
> Finger crossed for a quick CI pass since I'll delay the SSEU rebase
> until this gets in.
Pushed with just the compile/BAT checking...
-Chris
More information about the Intel-gfx
mailing list