Bug 100161 - [regression] SaschaWillems shadowmapping misrenders
Summary: [regression] SaschaWillems shadowmapping misrenders
Status: RESOLVED NOTOURBUG
Alias: None
Product: Mesa
Classification: Unclassified
Component: Drivers/Vulkan/intel (show other bugs)
Version: git
Hardware: Other All
: medium normal
Assignee: Intel 3D Bugs Mailing List
QA Contact: Intel 3D Bugs Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2017-03-11 17:42 UTC by Grazvydas Ignotas
Modified: 2017-03-11 19:31 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments

Description Grazvydas Ignotas 2017-03-11 17:42:20 UTC
on Skylake GT2 this commit broke shadowmapping demo:

commit ea744912b3fb2fe4a4d74dcb4f7bc8a07a8c6cf5
Author: Nanley Chery
Date:   Tue Jan 31 11:25:31 2017 -0800

    anv/cmd_buffer: Conditionally choose the sampled image surface state
    
    Signed-off-by: Nanley Chery
    Reviewed-by: Jason Ekstrand

Reverting it fixes the issue.
The demos are also continuously printing:
genX_cmd_buffer.c:507 ASSERT: iview->vk_format == att->format
Comment 1 Jason Ekstrand 2017-03-11 18:04:55 UTC
Given that you're seeing that assert, the Sascha demo is doing at least one thing wrong with the API.  If it's also doing layout transitions wrong, it would be very easy for that commit to cause problems.  Run with validation enabled and report any errors against the Sascha demos.  Once it's passing validation again, feel free to re-open and we can talk about driver bugs.

Nanley, did you see this change and are you aware of exactly what's going wrong?
Comment 2 Grazvydas Ignotas 2017-03-11 19:31:48 UTC
I've noticed the precompiled binaries are quite old, and indeed building them from source gives a correctly working shadowmapping demo without asserts. I've asked for update of the binaries:
https://github.com/SaschaWillems/Vulkan/issues/301


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.