[Mesa-dev] [PATCH] radeon/compute: Always report at least 1 compute unit

Bruno Jimenez brunojimen at gmail.com
Fri Jun 13 12:33:48 PDT 2014


Hi,

Looks good to me.

-Bruno

On Fri, 2014-06-13 at 12:58 -0400, Tom Stellard wrote:
> Some apps will abort if they detect 0 compute units.  This fixes
> crashes in some OpenCV tests.
> ---
>  src/gallium/drivers/radeon/r600_pipe_common.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/gallium/drivers/radeon/r600_pipe_common.c b/src/gallium/drivers/radeon/r600_pipe_common.c
> index 4b9d31c..1911cde 100644
> --- a/src/gallium/drivers/radeon/r600_pipe_common.c
> +++ b/src/gallium/drivers/radeon/r600_pipe_common.c
> @@ -523,7 +523,7 @@ static int r600_get_compute_param(struct pipe_screen *screen,
>  	case PIPE_COMPUTE_CAP_MAX_COMPUTE_UNITS:
>  		if (ret) {
>  			uint32_t *max_compute_units = ret;
> -			*max_compute_units = rscreen->info.max_compute_units;
> +			*max_compute_units = MAX2(rscreen->info.max_compute_units, 1);
>  		}
>  		return sizeof(uint32_t);
>  




More information about the mesa-dev mailing list