[Mesa-dev] [PATCH] winsys/amdgpu: add an assertion to cik_get_num_tile_pipes (v2)

[Alex Deucher]

On Thu, Feb 4, 2016 at 11:15 AM, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> v2: print an error to stderr

Reviewed-by: Alex Deucher <alexander.deucher at amd.com>

> ---
>  src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
> index d1ba259..2c4de75 100644
> --- a/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
> +++ b/src/gallium/winsys/amdgpu/drm/amdgpu_winsys.c
> @@ -68,7 +68,6 @@ static unsigned cik_get_num_tile_pipes(struct amdgpu_gpu_info *info)
>
>     switch (CIK__GB_TILE_MODE__PIPE_CONFIG(mode2d)) {
>     case CIK__PIPE_CONFIG__ADDR_SURF_P2:
> -   default:
>         return 2;
>     case CIK__PIPE_CONFIG__ADDR_SURF_P4_8x16:
>     case CIK__PIPE_CONFIG__ADDR_SURF_P4_16x16:
> @@ -86,6 +85,10 @@ static unsigned cik_get_num_tile_pipes(struct amdgpu_gpu_info *info)
>     case CIK__PIPE_CONFIG__ADDR_SURF_P16_32X32_8X16:
>     case CIK__PIPE_CONFIG__ADDR_SURF_P16_32X32_16X16:
>         return 16;
> +   default:
> +       fprintf(stderr, "Invalid CIK pipe configuration, assuming P2\n");
> +       assert(!"this should never occur");
> +       return 2;
>     }
>  }
>
> --
> 2.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev