Module: Mesa Branch: master Commit: b607d57630daa7d92a84c41abfd45cacbe63f3d2 URL: http://cgit.freedesktop.org/mesa/mesa/commit/?id=b607d57630daa7d92a84c41abfd45cacbe63f3d2
Author: Kenneth Graunke <[email protected]> Date: Sat Jun 8 10:01:34 2013 -0700 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 <[email protected]> Signed-off-by: Kenneth Graunke <[email protected]> --- src/mesa/drivers/dri/i965/gen6_blorp.cpp | 19 ------------------- 1 files changed, 0 insertions(+), 19 deletions(-) diff --git a/src/mesa/drivers/dri/i965/gen6_blorp.cpp b/src/mesa/drivers/dri/i965/gen6_blorp.cpp index c7bb815..bb87f6b 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(); - } } _______________________________________________ mesa-commit mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/mesa-commit
