On Tue, 12 Jun 2012 12:07:09 -0700 Kenneth Graunke <kenneth at whitecape.org> wrote: > With fixes and updates from Ben Widawsky and comments from Paul Berry. > > This should not be pushed until libdrm 2.4.36 is released with Ben's > hardware context support. 2.4.36 has been released. I screwed up the announce though. > > I haven't hooked up the context destroy function yet as I'm not entirely > sure about tear-down, but they will be freed on fd close, so it should > only be a problem for long running processes that open and destroy many > GL contexts. > > Cc: intel-gfx at lists.freedesktop.org > Signed-off-by: Kenneth Graunke <kenneth at whitecape.org> > Signed-off-by: Ben Widawsky <ben at bwidawsk.net> > --- > configure.ac | 2 +- > src/mesa/drivers/dri/i965/brw_context.c | 1 + > src/mesa/drivers/dri/i965/brw_vtbl.c | 13 +++++++++---- > src/mesa/drivers/dri/intel/intel_batchbuffer.c | 9 +++++++-- > src/mesa/drivers/dri/intel/intel_context.c | 2 ++ > src/mesa/drivers/dri/intel/intel_context.h | 2 ++ > 6 files changed, 22 insertions(+), 7 deletions(-) > > diff --git a/configure.ac b/configure.ac > index 29ee87b..e668c69 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -33,7 +33,7 @@ USER_CXXFLAGS="$CXXFLAGS" > dnl Versions for external dependencies > LIBDRM_REQUIRED=2.4.24 > LIBDRM_RADEON_REQUIRED=2.4.31 > -LIBDRM_INTEL_REQUIRED=2.4.34 > +LIBDRM_INTEL_REQUIRED=2.4.36 > LIBDRM_NVVIEUX_REQUIRED=2.4.33 > LIBDRM_NOUVEAU_REQUIRED=2.4.33 > DRI2PROTO_REQUIRED=2.6 > diff --git a/src/mesa/drivers/dri/i965/brw_context.c b/src/mesa/drivers/dri/i965/brw_context.c > index f7073cd..d4159c7 100644 > --- a/src/mesa/drivers/dri/i965/brw_context.c > +++ b/src/mesa/drivers/dri/i965/brw_context.c > @@ -298,6 +298,7 @@ brwCreateContext(int api, > > brw->prim_restart.in_progress = false; > brw->prim_restart.enable_cut_index = false; > + intel->hw_ctx = drm_intel_gem_context_create(intel->bufmgr); > > brw_init_state( brw ); > > diff --git a/src/mesa/drivers/dri/i965/brw_vtbl.c b/src/mesa/drivers/dri/i965/brw_vtbl.c > index 5699749..88e6dd1 100644 > --- a/src/mesa/drivers/dri/i965/brw_vtbl.c > +++ b/src/mesa/drivers/dri/i965/brw_vtbl.c > @@ -166,11 +166,16 @@ static void brw_new_batch( struct intel_context *intel ) > { > struct brw_context *brw = brw_context(&intel->ctx); > > - /* Mark all context state as needing to be re-emitted. > - * This is probably not as severe as on 915, since almost all of our state > - * is just in referenced buffers. > + /* If the kernel supports hardware contexts, then most hardware state is > + * preserved between batches; we only need to re-emit state that is required > + * to be in every batch. Otherwise we need to re-emit all the state that > + * would otherwise be stored in the context (which for all intents and > + * purposes means everything). > */ > - brw->state.dirty.brw |= BRW_NEW_CONTEXT | BRW_NEW_BATCH; > + if (intel->hw_ctx == NULL) > + brw->state.dirty.brw |= BRW_NEW_CONTEXT; > + > + brw->state.dirty.brw |= BRW_NEW_BATCH; > > /* Assume that the last command before the start of our batch was a > * primitive, for safety. > diff --git a/src/mesa/drivers/dri/intel/intel_batchbuffer.c b/src/mesa/drivers/dri/intel/intel_batchbuffer.c > index 76a69f7..7ba141d 100644 > --- a/src/mesa/drivers/dri/intel/intel_batchbuffer.c > +++ b/src/mesa/drivers/dri/intel/intel_batchbuffer.c > @@ -188,8 +188,13 @@ do_flush_locked(struct intel_context *intel) > if (ret == 0) { > if (unlikely(INTEL_DEBUG & DEBUG_AUB) && intel->vtbl.annotate_aub) > intel->vtbl.annotate_aub(intel); > - ret = drm_intel_bo_mrb_exec(batch->bo, 4*batch->used, NULL, 0, 0, > - flags); > + if (intel->hw_ctx == NULL || batch->is_blit) { > + ret = drm_intel_bo_mrb_exec(batch->bo, 4*batch->used, NULL, 0, 0, > + flags); > + } else { > + ret = drm_intel_gem_bo_context_exec(batch->bo, intel->hw_ctx, > + 4 * batch->used, flags); > + } > } > } > > diff --git a/src/mesa/drivers/dri/intel/intel_context.c b/src/mesa/drivers/dri/intel/intel_context.c > index c2f7d29..fc244c1 100644 > --- a/src/mesa/drivers/dri/intel/intel_context.c > +++ b/src/mesa/drivers/dri/intel/intel_context.c > @@ -593,6 +593,8 @@ intelInitContext(struct intel_context *intel, > if (intelScreen->bufmgr == NULL) > return false; > > + intel->hw_ctx = NULL; > + > /* Can't rely on invalidate events, fall back to glViewport hack */ > if (!driContextPriv->driScreenPriv->dri2.useInvalidate) { > intel->saved_viewport = functions->Viewport; > diff --git a/src/mesa/drivers/dri/intel/intel_context.h b/src/mesa/drivers/dri/intel/intel_context.h > index cc3ee0d..29ab187 100644 > --- a/src/mesa/drivers/dri/intel/intel_context.h > +++ b/src/mesa/drivers/dri/intel/intel_context.h > @@ -225,6 +225,8 @@ struct intel_context > > int urb_size; > > + drm_intel_context *hw_ctx; > + > struct intel_batchbuffer batch; > > drm_intel_bo *first_post_swapbuffers_batch; -- Ben Widawsky, Intel Open Source Technology Center