Bug 109405 - [BISECTED] [REGRESSION] KHR-GL45.enhanced_layouts.glsl_contant_immutablity aborts in second execution
Summary: [BISECTED] [REGRESSION] KHR-GL45.enhanced_layouts.glsl_contant_immutablity ab...
Status: RESOLVED FIXED
Alias: None
Product: Mesa
Classification: Unclassified
Component: glsl-compiler (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: mesa-dev
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-01-21 15:23 UTC by Andrés Gómez García
Modified: 2019-01-21 20:58 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Andrés Gómez García 2019-01-21 15:23:28 UTC
After:

--

commit 64b8c86d37ebb1e1d286c69d642d52b7bcf051d3 (danger/master)
Author: Timothy Arceri <tarceri@itsqueeze.com>
Date:   Thu Jan 17 17:16:29 2019 +1100

    glsl: be much more aggressive when skipping shader compilation
    
    Currently we only add a cache key for a shader once it is linked.
    However games like Team Fortress 2 compile a whole bunch of shaders
    which are never actually linked. These compiled shaders can take
    up a bunch of memory.
    
    This patch changes things so that we add the key for the shader to
    the cache as soon as it is compiled. This means on a warm cache we
    can avoid the wasted memory from these shaders. Worst case scenario
    is we need to compile the shaders at link time but this can happen
    anyway if the shader has been evicted from the cache.
    
    Reduces memory use in Team Fortress 2 from 1.3GB -> 770MB on a
    warm cache from start up to the game menu.
    
    Acked-by: Marek Olšák <marek.olsak@amd.com>

--

The following VK-GL-CTS test is failing in a second execution with the i965 driver, shader cache activated, and the x11_egl target:

--

local@5207cd755898:~/vk-gl-cts/build/external/openglcts/modules$ ./glcts --deqp-case="KHR-GL45.enhanced_layouts.glsl_contant_immutablity"                                                                         
Writing test log into TestResults.qpa
dEQP Core git-117ce3699c7e2d3d04f1760b25399aedd5dc90fa (0x117ce369) starting..
  target implementation = 'X11 EGL'
ATTENTION: default value of option vblank_mode overridden by environment.

Test case 'KHR-GL45.enhanced_layouts.glsl_contant_immutablity'..
  Pass (Pass)

DONE!

Test run totals:
  Passed:        1/1 (100.0%)
  Failed:        0/1 (0.0%)
  Not supported: 0/1 (0.0%)
  Warnings:      0/1 (0.0%)
local@5207cd755898:~/vk-gl-cts/build/external/openglcts/modules$ ./glcts --deqp-case="KHR-GL45.enhanced_layouts.glsl_contant_immutablity"                                                                         
Writing test log into TestResults.qpa
dEQP Core git-117ce3699c7e2d3d04f1760b25399aedd5dc90fa (0x117ce369) starting..
  target implementation = 'X11 EGL'
ATTENTION: default value of option vblank_mode overridden by environment.

Test case 'KHR-GL45.enhanced_layouts.glsl_contant_immutablity'..
Segmentation fault (core dumped)
Comment 1 tele42k3 2019-01-21 15:43:35 UTC
Hello, this commit was backed out and replaced by https://gitlab.freedesktop.org/mesa/mesa/commit/6ca652faf368427e3e6d57ef5456f78203b8207e . Did your bisect account for this?
Comment 2 Timothy Arceri 2019-01-21 20:58:45 UTC
Yeah I accidentally tested the initial patch with a 32bit build of mesa and 64bit build of piglit. This patch was reverted and the later replaced by the commit in comment 2.

I'm assuming this is no longer an issue and closing. Please reopen if that is not the case.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.