This is OK.

Reviewed-by: Marek Olšák <marek.ol...@amd.com>

Marek

On Thu, Oct 18, 2018 at 3:43 PM Jan Vesely <jan.ves...@rutgers.edu> wrote:

> Fixes assertion failure/crash when running luxmark/luxball on clover.
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=108272
> CC: mesa-sta...@lists.freedesktop.org
> Signed-off-by: Jan Vesely <jan.ves...@rutgers.edu>
> ---
> This is just a stop gap measure. OpenCL does not limit the number of
> global buffers, but we can return CL_MEM_OBJECT_ALLOCATION_FAILURE in
> clEnqueueNDRangeKernel.
> What would be the preferred way? Add PIPE_CAP to query this? allow
> set_global_binding to fail with too many buffers?
>
> thanks,
> Jan
>
>  src/gallium/drivers/radeonsi/si_compute.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/si_compute.h
> b/src/gallium/drivers/radeonsi/si_compute.h
> index 99b501673c..57c0bde4ac 100644
> --- a/src/gallium/drivers/radeonsi/si_compute.h
> +++ b/src/gallium/drivers/radeonsi/si_compute.h
> @@ -29,7 +29,7 @@
>
>  #include "si_shader.h"
>
> -#define MAX_GLOBAL_BUFFERS 22
> +#define MAX_GLOBAL_BUFFERS 32
>
>  struct si_compute {
>         struct pipe_reference reference;
> --
> 2.17.2
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev
>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to