[x265] [PATCH] threadpool.cpp: fix default pool param behaviour, if NULL or “” (default) x265 will use all available threads on each NUMA node

[Praveen Tiwari]

Please ignore this this behaviour is not required for linux systems.
Thanks.

Regards,
Praveen

On Wed, Sep 7, 2016 at 5:19 PM, <praveen at multicorewareinc.com> wrote:

> # HG changeset patch
> # User Praveen Tiwari <praveen at multicorewareinc.com>
> # Date 1473246754 -19800
> #      Wed Sep 07 16:42:34 2016 +0530
> # Node ID 9587a394ba58a2c3a579db5fb3f7531daf49559b
> # Parent  df559450949bd085b0fc5e01332aa8458af2fa43
> threadpool.cpp: fix default pool param behaviour, if NULL or 灯 (default)
> x265 will use all available threads on each NUMA node
>
> diff -r df559450949b -r 9587a394ba58 source/common/threadpool.cpp
> --- a/source/common/threadpool.cpp      Wed Aug 10 13:26:18 2016 +0530
> +++ b/source/common/threadpool.cpp      Wed Sep 07 16:42:34 2016 +0530
> @@ -330,8 +330,8 @@
>              {
>                  for (int j = i; j < numNumaNodes; j++)
>                  {
> -                    threadsPerPool[numNumaNodes] += cpusPerNode[j];
> -                    nodeMaskPerPool[numNumaNodes] |= ((uint64_t)1 << j);
> +                    threadsPerPool[j] += cpusPerNode[j];
> +                    nodeMaskPerPool[j] |= ((uint64_t)1 << j);
>                  }
>                  break;
>              }
> @@ -366,8 +366,8 @@
>      {
>          for (int i = 0; i < numNumaNodes; i++)
>          {
> -            threadsPerPool[numNumaNodes]  += cpusPerNode[i];
> -            nodeMaskPerPool[numNumaNodes] |= ((uint64_t)1 << i);
> +            threadsPerPool[i]  += cpusPerNode[i];
> +            nodeMaskPerPool[i] |= ((uint64_t)1 << i);
>          }
>      }
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/x265-devel/attachments/20160908/2815cba4/attachment.html>