[Mesa-dev] [PATCH] radeonsi: add RW_BUFFERS only once in si_ce_needed_cs_space

Bas Nieuwenhuizen bas at basnieuwenhuizen.nl
Fri Apr 22 10:39:19 UTC 2016


On Fri, Apr 22, 2016 at 10:34 AM, Marek Olšák <maraeo at gmail.com> wrote:
> From: Marek Olšák <marek.olsak at amd.com>
>
> ---
>  src/gallium/drivers/radeonsi/si_hw_context.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_hw_context.c b/src/gallium/drivers/radeonsi/si_hw_context.c
> index e6018f3..5ab20d8 100644
> --- a/src/gallium/drivers/radeonsi/si_hw_context.c
> +++ b/src/gallium/drivers/radeonsi/si_hw_context.c
> @@ -43,13 +43,13 @@ static unsigned si_ce_needed_cs_space(void)
>         unsigned space = 0;
>
>         space += si_descriptor_list_cs_space(SI_NUM_CONST_BUFFERS, 4);
> -       space += si_descriptor_list_cs_space(SI_NUM_RW_BUFFERS, 4);
>         space += si_descriptor_list_cs_space(SI_NUM_SHADER_BUFFERS, 4);
>         space += si_descriptor_list_cs_space(SI_NUM_SAMPLERS, 16);
>         space += si_descriptor_list_cs_space(SI_NUM_IMAGES, 8);
> -

Is this whitespace change intended? Either way,

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

- Bas

>         space *= SI_NUM_SHADERS;
>
> +       space += si_descriptor_list_cs_space(SI_NUM_RW_BUFFERS, 4);
> +
>         /* Increment CE counter packet */
>         space += 2;
>
> --
> 2.5.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