Hi,

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 
b/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 
dims,
                          !_mesa_is_srgb_format(gl_format);
     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
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to