[Mesa-dev] [PATCH] radeonsi: fix undefined shift in macro definition

Bas Nieuwenhuizen bas at basnieuwenhuizen.nl
Tue Jun 18 21:40:28 UTC 2019


Reviewed-by: Bas Nieuwenhuizen <bas at basnieuwenhuizen.nl>

On Tue, Jun 18, 2019 at 11:20 PM Dave Airlie <airlied at gmail.com> wrote:
>
> From: Dave Airlie <airlied at redhat.com>
>
> Pointed out by coverity
> ---
>  src/gallium/drivers/radeonsi/si_perfcounter.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_perfcounter.c b/src/gallium/drivers/radeonsi/si_perfcounter.c
> index ad7f1eb3d99..75bd61730da 100644
> --- a/src/gallium/drivers/radeonsi/si_perfcounter.c
> +++ b/src/gallium/drivers/radeonsi/si_perfcounter.c
> @@ -126,7 +126,7 @@ static const unsigned si_pc_shader_type_bits[] = {
>  /* Max counters per HW block */
>  #define SI_QUERY_MAX_COUNTERS 16
>
> -#define SI_PC_SHADERS_WINDOWING (1 << 31)
> +#define SI_PC_SHADERS_WINDOWING (1u << 31)
>
>  struct si_query_group {
>         struct si_query_group *next;
> --
> 2.21.0
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list