From 069f60a4e7d1e351fca7e9140f66da2e12ce390b Mon Sep 17 00:00:00 2001 From: Samuel Pitoiset Date: Tue, 2 Dec 2025 13:47:46 +0100 Subject: [PATCH] radv: enable conservativeRasterizationPostDepthCoverage on GFX10+ when possible This seems to work just fine, except on NAVI21, NAVI22 and VANGOGH. It might be the same issue as has_vrs_ds_export_bug but it's not documented anywhere. That being said, the CTS tests that fail don't even export depth or stencil from fragment shaders. Do not enable this feature on these GPUs to be conservative. Signed-off-by: Samuel Pitoiset Part-of: --- src/amd/vulkan/radv_physical_device.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/src/amd/vulkan/radv_physical_device.c b/src/amd/vulkan/radv_physical_device.c index e52fdde10c4..fa3000d201d 100644 --- a/src/amd/vulkan/radv_physical_device.c +++ b/src/amd/vulkan/radv_physical_device.c @@ -1919,7 +1919,13 @@ radv_get_physical_device_properties(struct radv_physical_device *pdev) .degenerateTrianglesRasterized = true, .degenerateLinesRasterized = false, .fullyCoveredFragmentShaderInputVariable = true, - .conservativeRasterizationPostDepthCoverage = false, + .conservativeRasterizationPostDepthCoverage = + pdev->info.gfx_level >= GFX10 && + /* The combination of VRS + underestimate conservative rasterization + post depth coverage + * seem to not behave correctly on few RDNA2 GPUs. It might be similar to + * has_vrs_ds_export_bug but this isn't documented anywhere. Be conservative here. + */ + pdev->info.family != CHIP_NAVI21 && pdev->info.family != CHIP_NAVI22 && pdev->info.family != CHIP_VANGOGH, #ifndef _WIN32 /* VK_EXT_pci_bus_info */