i965/tex_image: Avoid the ASTC LDR workaround on gen9lp
authorNanley Chery <nanley.g.chery@intel.com>
Sat, 24 Feb 2018 09:26:20 +0000 (01:26 -0800)
committerNanley Chery <nanley.g.chery@intel.com>
Tue, 1 May 2018 23:47:39 +0000 (16:47 -0700)
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 by about 17 seconds
(reported by Eero).

Fixes: 710b1d2e665 "i965/tex_image: Flush certain subnormal ASTC channel values"
Tested-by: Eero Tamminen <eero.t.tamminen@intel.com>
Acked-by: Kenneth Graunke <kenneth@whitecape.org>
src/mesa/drivers/dri/i965/intel_tex_image.c

index 3fd227ad05d09587cd5b9257fc095f48e27aeff0..856216ecf9e7e3f36e62fd4c46351cfe381a807d 100644 (file)
@@ -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);