<div dir="ltr">Thanks for the patch, Praveen. Mateusz' earlier patch provides a more complete fix and has been pushed.</div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, May 30, 2016 at 6:48 PM, Praveen Tiwari <span dir="ltr"><<a href="mailto:praveen@multicorewareinc.com" target="_blank">praveen@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I remember some numa functionality requires minimum window 7, they are not supported on previous version of window OS.   </div><div class="gmail_extra"><br clear="all"><div><div data-smartmail="gmail_signature"><div dir="ltr">Regards,<div>Praveen</div></div></div></div><div><div class="h5">
<br><div class="gmail_quote">On Mon, May 30, 2016 at 6:43 PM, Mateusz <span dir="ltr"><<a href="mailto:mateuszb@poczta.onet.pl" target="_blank">mateuszb@poczta.onet.pl</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">There is a serious bug in threadpool code that prevent working in Windows XP/Vista.<br>
VS 2015 error when compiling for 32-bit Windows XP:<br>
(ClCompile target) -><br>
  I:\vs\x265\source\common\threadpool.cpp(590): error C3861: 'GetNumaNodeProcessorMaskEx': identifier not found [I:\vs\x265\ma\<br>
8-b\common\common.vcxproj]<br>
<br>
Did you see patch <a href="https://patches.videolan.org/patch/13495/" rel="noreferrer" target="_blank">https://patches.videolan.org/patch/13495/</a> (it fixes also this warning)?<br>
<br>
<br>
W dniu 2016-05-30 o 14:45, <a href="mailto:praveen@multicorewareinc.com" target="_blank">praveen@multicorewareinc.com</a> pisze:<br>
<div><div>> # HG changeset patch<br>
> # User Praveen Tiwari <<a href="mailto:praveen@multicorewareinc.com" target="_blank">praveen@multicorewareinc.com</a>><br>
> # Date 1464585837 -19800<br>
> #      Mon May 30 10:53:57 2016 +0530<br>
> # Node ID b8dbe8d7c09e7fc0b7cce236569fc5df2eb70b1e<br>
> # Parent  aeade2e8d8688ebffb8455b8948d89d6a72e2c38<br>
> threadpool: fix warning: ‘int popCount(uint64_t)’ defined but not used [-Wunused-function]<br>
>  static int popCount(uint64_t x)<br>
><br>
> diff -r aeade2e8d868 -r b8dbe8d7c09e source/common/threadpool.cpp<br>
> --- a/source/common/threadpool.cpp    Thu May 26 16:45:09 2016 +0530<br>
> +++ b/source/common/threadpool.cpp    Mon May 30 10:53:57 2016 +0530<br>
> @@ -68,6 +68,7 @@<br>
>  # define strcasecmp _stricmp<br>
>  #endif<br>
><br>
> +#if defined(_WIN32_WINNT) && _WIN32_WINNT >= _WIN32_WINNT_WIN7<br>
>  const uint64_t m1 = 0x5555555555555555; //binary: 0101...<br>
>  const uint64_t m2 = 0x3333333333333333; //binary: 00110011..<br>
>  const uint64_t m3 = 0x0f0f0f0f0f0f0f0f; //binary:  4 zeros,  4 ones ...<br>
> @@ -80,6 +81,7 @@<br>
>      x = (x + (x >> 4)) & m3;<br>
>      return (x * h01) >> 56;<br>
>  }<br>
> +#endif<br>
><br>
>  namespace X265_NS {<br>
>  // x265 private namespace<br>
><br>
><br>
><br>
</div></div>> _______________________________________________<br>
> x265-devel mailing list<br>
> <a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
> <a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
><br>
<br>
<br>
_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org" target="_blank">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
</blockquote></div><br></div></div></div>
<br>_______________________________________________<br>
x265-devel mailing list<br>
<a href="mailto:x265-devel@videolan.org">x265-devel@videolan.org</a><br>
<a href="https://mailman.videolan.org/listinfo/x265-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/x265-devel</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div>Deepthi Nandakumar<br></div>Engineering Manager, x265<br></div>Multicoreware, Inc<br></div></div>
</div>