Mesa (master): mesa: Set gl_constants::MinMapBufferAlignment

Ian Romanick idr at kemper.freedesktop.org
Thu Jan 23 17:00:39 UTC 2014


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

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Wed Jan 22 08:28:49 2014 -0800

mesa: Set gl_constants::MinMapBufferAlignment

Leaving it set to zero isn't really correct since every allocation has
at least an alignment of 1 byte.  It also caused a problem in the i965
driver after I removed the MAX(64, ...) from the alignment calculation.
That's what I get for changing a patch without retesting it. :(

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=73907
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Cc: Lu Hua <huax.lu at intel.com>

---

 src/mesa/main/context.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mesa/main/context.c b/src/mesa/main/context.c
index 7c3b2d7..b7cd568 100644
--- a/src/mesa/main/context.c
+++ b/src/mesa/main/context.c
@@ -587,6 +587,7 @@ _mesa_init_constants(struct gl_context *ctx)
    ctx->Const.MaxSpotExponent = 128.0;
    ctx->Const.MaxViewportWidth = MAX_VIEWPORT_WIDTH;
    ctx->Const.MaxViewportHeight = MAX_VIEWPORT_HEIGHT;
+   ctx->Const.MinMapBufferAlignment = 1;
 
    /* Driver must override these values if ARB_viewport_array is supported. */
    ctx->Const.MaxViewports = 1;




More information about the mesa-commit mailing list