Mesa (master): i965/blorp: Bump the batch space estimate

Jason Ekstrand jekstrand at kemper.freedesktop.org
Thu Apr 6 20:32:41 UTC 2017


Module: Mesa
Branch: master
Commit: c9c39812b91c8104bc0bea16053312547846249c
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=c9c39812b91c8104bc0bea16053312547846249c

Author: Jason Ekstrand <jason.ekstrand at intel.com>
Date:   Wed Apr  5 13:41:56 2017 -0700

i965/blorp: Bump the batch space estimate

Commit f938354362655a378d474c5f79c52cea9852ab91 recently increased the
alignment on vertex buffer data from 32 to 64.  This caused us to
consume a bit more batch than we were before and we now go over the
estimate by a small amount on certain blits on gen8+.  This commit bumps
then gen8 batch estimate by a bit to compensate.  Haswell and older
still seems to be well within the limit.

Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=100582
Reviewed-by: Iago Toral Quiroga <itoral at igalia.com>
Acked-by: Kenneth Graunke <kenneth at whitecape.org>
Cc: "13.0 17.0" <mesa-stable at lists.freedesktop.org>

---

 src/mesa/drivers/dri/i965/genX_blorp_exec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/genX_blorp_exec.c b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
index b7a23afab4..2066141128 100644
--- a/src/mesa/drivers/dri/i965/genX_blorp_exec.c
+++ b/src/mesa/drivers/dri/i965/genX_blorp_exec.c
@@ -181,7 +181,7 @@ genX(blorp_exec)(struct blorp_batch *batch,
    assert(batch->blorp->driver_ctx == batch->driver_batch);
    struct brw_context *brw = batch->driver_batch;
    struct gl_context *ctx = &brw->ctx;
-   const uint32_t estimated_max_batch_usage = GEN_GEN >= 8 ? 1800 : 1500;
+   const uint32_t estimated_max_batch_usage = GEN_GEN >= 8 ? 1920 : 1500;
    bool check_aperture_failed_once = false;
 
    /* Flush the sampler and render caches.  We definitely need to flush the




More information about the mesa-commit mailing list