Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expect shader_primitive_index tests to fail on AMD RADV POLARIS12. #2754

Merged

Conversation

jimblandy
Copy link
Member

On Fedora, mesa-vulkan-drivers 22.0.3 AMD RADV POLARIS12 just colors both the NE and SW corners red. There's no blue. (NW and SE are white, as expected.) I suspect this means that the primitive ID is always being passed as zero.

This is probably not just due to a difference in rasterization: I tried enlarging the texture to 4x4 and shifting the triangles to ensure that there was a pixel in each quadrant that was fully covered.

With Vulkan "llvmpipe (LLVM 14.0.0, 256 bits)", the unmodified test passes.

This is probably limited to Linux, but wgpu_tests::common::TestParameters doesn't have a slot for that.

Fixes #2751.

@jimblandy jimblandy added external: driver-bug A driver is causing the bug, though we may still want to work around it api: vulkan Issues with Vulkan labels Jun 9, 2022
@jimblandy jimblandy self-assigned this Jun 9, 2022
@jimblandy jimblandy requested a review from cwfitzgerald June 9, 2022 06:00
@jimblandy jimblandy force-pushed the expect-fail-primitive-index-amd branch from 32885bd to eb05749 Compare June 9, 2022 06:01
On Fedora, mesa-vulkan-drivers 22.0.3 AMD RADV POLARIS12 just colors
both the NE and SW corners red. There's no blue. (NW and SE are white,
as expected.) I suspect
this means that the primitive ID is always being passed as zero.

This is probably not just due to a difference in rasterization: I
tried enlarging the texture to 4x4 and shifting the triangles to
ensure that there was a pixel in each quadrant that was fully covered.

With Vulkan "llvmpipe (LLVM 14.0.0, 256 bits)", the unmodified test passes.

This is probably limited to Linux, but
`wgpu_tests::common::TestParameters` doesn't have a slot for that.

Fixes gfx-rs#2751.
@jimblandy jimblandy force-pushed the expect-fail-primitive-index-amd branch from eb05749 to 2409853 Compare June 9, 2022 06:46
@jimblandy jimblandy requested a review from cwfitzgerald June 9, 2022 06:47
@cwfitzgerald cwfitzgerald enabled auto-merge (squash) June 10, 2022 19:44
@cwfitzgerald cwfitzgerald merged commit 498842f into gfx-rs:master Jun 10, 2022
@jimblandy jimblandy deleted the expect-fail-primitive-index-amd branch June 28, 2022 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: vulkan Issues with Vulkan external: driver-bug A driver is causing the bug, though we may still want to work around it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New shader_primitive_index::draw and shader_primitive_index::draw_indexed tests fail on Linux Vulkan back end
2 participants