Summary: | [SKL Bisected]many Ogles3conform cases core dumped | ||
---|---|---|---|
Product: | Mesa | Reporter: | lu hua <huax.lu> |
Component: | Drivers/DRI/i965 | Assignee: | Ben Widawsky <ben> |
Status: | VERIFIED FIXED | QA Contact: | Intel 3D Bugs Mailing List <intel-3d-bugs> |
Severity: | major | ||
Priority: | high | CC: | ben, idr, kaveh.nasri |
Version: | unspecified | ||
Hardware: | All | ||
OS: | Linux (All) | ||
Whiteboard: | |||
i915 platform: | i915 features: | ||
Attachments: |
Only reserve message header space once
Fix uniform pull constant payload size |
Description
lu hua
2015-02-13 02:50:21 UTC
Is this a regression? If so, is there a bisect? Bisect shows: 0ac4c272755c75108a10a84ce33bf6a6234985d3 is the first bad commit commit 0ac4c272755c75108a10a84ce33bf6a6234985d3 Author: Kristian Høgsberg <krh@bitplanet.net> AuthorDate: Wed Dec 10 14:59:26 2014 -0800 Commit: Kristian Høgsberg <krh@bitplanet.net> CommitDate: Thu Jan 8 10:13:32 2015 -0800 i965/skl: Always use a header for SIMD4x2 sampler messages SKL+ overloads the SIMD4x2 SIMD mode to mean either SIMD8D or SIMD4x2 depending on bit 22 in the message header. If the bit is 0 or there is no header we get SIMD8D. We always wand SIMD4x2 in vec4 and for fs pull constants, so use a message header in those cases and set bit 22 there. Based on an initial patch from Ken. Reviewed-by: Kenneth Graunke <kenneth@whitecape.org> Signed-off-by: Kristian Høgsberg <krh@bitplanet.net> I may have already fixed this in master. Please test master (e93566a15c61c33faa2e694aa18d18e544e857ff) Test on the latest mesa master branch(commit a2299bfbbd6ed96), It still exists. output: dEQP Core GL-CTS-2.0 (0x0052484b) starting.. target implementation = 'X11' Test case 'ES3-CTS.shaders.uniform_block.single_basic_type.shared.lowp_mat3'.. glcts: malloc.c:3266: __libc_calloc: Assertion `nclears >= 3' failed. Aborted (core dumped) Created attachment 113543 [details] [review] Only reserve message header space once Please test Created attachment 113682 [details] [review] Fix uniform pull constant payload size This is the patch for upstream. Ignore the previous. I've pushed the patch to master now, can you please test? run ./glcts --deqp-case=ES3-CTS.shaders.uniform_block.single_basic_type.shared.lowp_mat3 output: dEQP Core GL-CTS-2.0 (0x0052484b) starting.. target implementation = 'X11' Test case 'ES3-CTS.shaders.uniform_block.single_basic_type.shared.lowp_mat3'.. Vertex compile time = 11.887000 ms Fragment compile time = 5.257000 ms Link time = 14.381000 ms Pass (Pass) DONE! Test run totals: Passed: 1/1 (100.00%) Failed: 0/1 (0.00%) Not supported: 0/1 (0.00%) Warnings: 0/1 (0.00%) Verified.Fixed. |
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.