From b607d57630daa7d92a84c41abfd45cacbe63f3d2 Mon Sep 17 00:00:00 2001 From: Kenneth Graunke Date: Sat, 8 Jun 2013 10:01:34 -0700 Subject: [PATCH] i965: Don't emit PIPELINE_SELECT from BLORP. Now that we emit invariant state at startup (and never select the media pipeline), the 3D pipeline will always already be selected, even if BLORP is the first operation. So this is unnecessary. v2: Fix unused variable warning (intel_context is no longer used). Reviewed-by: Eric Anholt Signed-off-by: Kenneth Graunke --- src/mesa/drivers/dri/i965/gen6_blorp.cpp | 19 ------------------- 1 file changed, 19 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen6_blorp.cpp b/src/mesa/drivers/dri/i965/gen6_blorp.cpp index c7bb8151c0b..bb87f6bbe67 100644 --- a/src/mesa/drivers/dri/i965/gen6_blorp.cpp +++ b/src/mesa/drivers/dri/i965/gen6_blorp.cpp @@ -50,30 +50,11 @@ gen6_blorp_emit_batch_head(struct brw_context *brw, const brw_blorp_params *params) { struct gl_context *ctx = &brw->intel.ctx; - struct intel_context *intel = &brw->intel; /* To ensure that the batch contains only the resolve, flush the batch * before beginning and after finishing emitting the resolve packets. - * - * Ideally, we would not need to flush for the resolve op. But, I suspect - * that it's unsafe for CMD_PIPELINE_SELECT to occur multiple times in - * a single batch, and there is no safe way to ensure that other than by - * fencing the resolve with flushes. Ideally, we would just detect if - * a batch is in progress and do the right thing, but that would require - * the ability to *safely* access brw_context::state::dirty::brw - * outside of the brw_upload_state() codepath. */ intel_flush(ctx); - - /* CMD_PIPELINE_SELECT - * - * Select the 3D pipeline, as opposed to the media pipeline. - */ - { - BEGIN_BATCH(1); - OUT_BATCH(brw->CMD_PIPELINE_SELECT << 16); - ADVANCE_BATCH(); - } } -- 2.30.2