Module: Mesa
Branch: master
Commit: 01d0c5a92294e3413a8e72970e25635057674659
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=01d0c5a92294e3413a8e72970e25635057674659

Author: Dave Airlie <airl...@redhat.com>
Date:   Thu Apr 13 14:36:26 2017 +1000

radv: fix stencil regression since new addrlib import

The addrlib import meant we'd return after we attempted
to setup the no stencil bits for an S8_UINT, now we break
and use the stencil level info when creating stencil DB
info.

Reviewed-by: Bas Nieuwenhuizen <b...@basnieuwenhuizen.nl>
Signed-off-by: Dave Airlie <airl...@redhat.com>

---

 src/amd/vulkan/radv_device.c                       | 8 ++++++++
 src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/amd/vulkan/radv_device.c b/src/amd/vulkan/radv_device.c
index 5f14394196..b099ec736c 100644
--- a/src/amd/vulkan/radv_device.c
+++ b/src/amd/vulkan/radv_device.c
@@ -2732,6 +2732,7 @@ radv_initialise_ds_surface(struct radv_device *device,
        unsigned format;
        uint64_t va, s_offs, z_offs;
        const struct radeon_surf_level *level_info = 
&iview->image->surface.level[level];
+       bool stencil_only = false;
        memset(ds, 0, sizeof(*ds));
        switch (iview->vk_format) {
        case VK_FORMAT_D24_UNORM_S8_UINT:
@@ -2750,6 +2751,10 @@ radv_initialise_ds_surface(struct radv_device *device,
                        S_028B78_POLY_OFFSET_DB_IS_FLOAT_FMT(1);
                ds->offset_scale = 1.0f;
                break;
+       case VK_FORMAT_S8_UINT:
+               stencil_only = true;
+               level_info = &iview->image->surface.stencil_level[level];
+               break;
        default:
                break;
        }
@@ -2784,6 +2789,9 @@ radv_initialise_ds_surface(struct radv_device *device,
                unsigned stencil_tile_mode = 
info->si_tile_mode_array[stencil_index];
                unsigned macro_mode = 
info->cik_macrotile_mode_array[macro_index];
 
+               if (stencil_only)
+                       tile_mode = stencil_tile_mode;
+
                ds->db_depth_info |=
                        S_02803C_ARRAY_MODE(G_009910_ARRAY_MODE(tile_mode)) |
                        S_02803C_PIPE_CONFIG(G_009910_PIPE_CONFIG(tile_mode)) |
diff --git a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c 
b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c
index 0433952e74..511f464df6 100644
--- a/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c
+++ b/src/amd/vulkan/winsys/amdgpu/radv_amdgpu_surface.c
@@ -486,7 +486,7 @@ static int radv_amdgpu_winsys_surface_init(struct 
radeon_winsys *_ws,
                r = radv_compute_level(ws->addrlib, surf, false, level, type, 
compressed,
                                       &AddrSurfInfoIn, &AddrSurfInfoOut, 
&AddrDccIn, &AddrDccOut);
                if (r)
-                       return r;
+                       break;
 
                if (level == 0) {
                        surf->bo_alignment = AddrSurfInfoOut.baseAlign;

_______________________________________________
mesa-commit mailing list
mesa-commit@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to