[Mesa-dev] [PATCH 2/2] gallium/u_queue: set num_threads correctly if not all threads start

Marek Olšák maraeo at gmail.com
Mon Feb 27 13:51:08 UTC 2017


Pushed the series. Thanks.

Marek

On Sun, Feb 26, 2017 at 1:44 AM, Grazvydas Ignotas <notasas at gmail.com> wrote:
> If i-th thread could not be created it means we have i threads,
> not i+1, because we start from 0.
>
> Fixes: 404d0d5 "gallium/u_queue: add an option to have multiple worker threads"
> Signed-off-by: Grazvydas Ignotas <notasas at gmail.com>
> ---
> no commit access
>
>  src/gallium/auxiliary/util/u_queue.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/src/gallium/auxiliary/util/u_queue.c b/src/gallium/auxiliary/util/u_queue.c
> index c51b621..8dd4cb3 100644
> --- a/src/gallium/auxiliary/util/u_queue.c
> +++ b/src/gallium/auxiliary/util/u_queue.c
> @@ -236,7 +236,7 @@ util_queue_init(struct util_queue *queue,
>              goto fail;
>           } else {
>              /* at least one thread created, so use it */
> -            queue->num_threads = i+1;
> +            queue->num_threads = i;
>              break;
>           }
>        }
> --
> 2.7.4
>
> _______________________________________________
> 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