Thanks Chris! Have a nice weekend. :) > -----Original Message----- > From: Chris Wilson [mailto:[email protected]] > Sent: Friday, May 12, 2017 6:12 PM > To: [email protected] > Cc: Dong, Chuanxiao <[email protected]>; Chris Wilson > <[email protected]> > Subject: [PATCH v2] drm/i915: set initialised only when init_context callback > is NULL > > From: Chuanxiao Dong <[email protected]> > > During execlist_context_deferred_alloc() we presumed that the context is > uninitialised (we only just allocated the state object for it!) and chose to > optimise away the later call to engine->init_context() if > engine->init_context were NULL. This breaks with GVT's contexts that are > marked as pre-initialised to avoid us annoyingly calling > engine->init_context(). The fix is to not override ce->initialised if it > is already true. > > Cc: Chris Wilson <[email protected]> > Signed-off-by: Chuanxiao Dong <[email protected]> > Link: http://patchwork.freedesktop.org/patch/msgid/1494497262-24855-1- > [email protected] > Reviewed-by: Chris Wilson <[email protected]> > Signed-off-by: Chris Wilson <[email protected]> > --- > drivers/gpu/drm/i915/intel_lrc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_lrc.c > b/drivers/gpu/drm/i915/intel_lrc.c > index 319d9a8f37ca..9a1192d61538 100644 > --- a/drivers/gpu/drm/i915/intel_lrc.c > +++ b/drivers/gpu/drm/i915/intel_lrc.c > @@ -1956,7 +1956,7 @@ static int execlists_context_deferred_alloc(struct > i915_gem_context *ctx, > > ce->ring = ring; > ce->state = vma; > - ce->initialised = engine->init_context == NULL; > + ce->initialised |= engine->init_context == NULL; > > return 0; > > -- > 2.11.0
_______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
