This drops the internal benchmark startup time from 18 to ~1 seconds (for warm startup) on the gen9lp platform where I tested it.

Tested-by: Eero tammi...@intel.com

        - Eero

On 06.03.2018 00:07, Nanley Chery wrote:
Both the internal documentation and the results of testing this in the
CI suggest that this is unnecessary. Add the fixes tag because this
reduces an internal benchmark's startup time significantly. Eero
reported that a less optimal version of this patch reduced the startup
time of the benchmark by about 14 seconds.

Fixes: 710b1d2e665 "i965/tex_image: Flush certain subnormal ASTC channel values"
Cc: Eero Tamminen <eero.t.tammi...@intel.com>
Cc: Scott D Phillips <scott.d.phill...@intel.com>
  src/mesa/drivers/dri/i965/intel_tex_image.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/mesa/drivers/dri/i965/intel_tex_image.c 
index e25bc9a0c08..a0408b304d5 100644
--- a/src/mesa/drivers/dri/i965/intel_tex_image.c
+++ b/src/mesa/drivers/dri/i965/intel_tex_image.c
@@ -927,7 +927,7 @@ intelCompressedTexSubImage(struct gl_context *ctx, GLuint 
     struct brw_context *brw = (struct brw_context*) ctx;
     const struct gen_device_info *devinfo = &brw->screen->devinfo;
-   if (devinfo->gen == 9 && is_linear_astc)
+   if (devinfo->gen == 9 && !gen_device_info_is_9lp(devinfo) && is_linear_astc)
        flush_astc_denorms(ctx, dims, texImage,
                           xoffset, yoffset, zoffset,
                           width, height, depth);

mesa-dev mailing list

Reply via email to