[Mesa-dev] [PATCH 1/5] r600g/compute: Fix input buffer size calculation

Alex Deucher alexdeucher at gmail.com
Tue Apr 23 06:18:10 PDT 2013


On Mon, Apr 22, 2013 at 11:24 PM, Tom Stellard <tom at stellard.net> wrote:
> From: Tom Stellard <thomas.stellard at amd.com>
>
> Buffer size should be in bytes not dwords.

For the series:

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

> ---
>  src/gallium/drivers/r600/evergreen_compute.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/r600/evergreen_compute.c b/src/gallium/drivers/r600/evergreen_compute.c
> index 189ffac..58dc36d 100644
> --- a/src/gallium/drivers/r600/evergreen_compute.c
> +++ b/src/gallium/drivers/r600/evergreen_compute.c
> @@ -203,7 +203,7 @@ void evergreen_compute_upload_input(
>                 unsigned buffer_size = shader->input_size;
>
>                 /* Add space for the grid dimensions */
> -               buffer_size += kernel_parameters_offset_bytes * sizeof(uint);
> +               buffer_size += kernel_parameters_offset_bytes;
>                 shader->kernel_param = r600_compute_buffer_alloc_vram(
>                                                 ctx->screen, buffer_size);
>         }
> --
> 1.7.11.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list