[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [60166] trunk/blender/build_files/ build_environment/install_deps.sh: Making install_deps. sh correctly see hyperthreading

[Bastien Montagne]

Not at all! Much simpler than previous solutions, in addition! :D

On 17/09/2013 09:50, Sergey Sharybin wrote:
> Hi,
>
> Hate to bring it to you guys, but this is still not fully correct ;) The
> code is still unaware of multi-cpu setup.
>
> So, if we already want to use all HT threads, why not to use this: cat
> /proc/cpuinfo  | grep processor | wc -l
>
> Imo, would be nice change while we're on track of CPU usage. Any objections
> on this?
>
>
> On Mon, Sep 16, 2013 at 8:38 PM, Daniel Salazar<zanqdo at gmail.com>  wrote:
>
>> Revision: 60166
>>
>> http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=60166
>> Author:   zanqdo
>> Date:     2013-09-16 14:38:33 +0000 (Mon, 16 Sep 2013)
>> Log Message:
>> -----------
>> Making install_deps.sh correctly see hyperthreading
>>
>> Modified Paths:
>> --------------
>>      trunk/blender/build_files/build_environment/install_deps.sh
>>
>> Modified: trunk/blender/build_files/build_environment/install_deps.sh
>> ===================================================================
>> --- trunk/blender/build_files/build_environment/install_deps.sh 2013-09-16
>> 13:24:28 UTC (rev 60165)
>> +++ trunk/blender/build_files/build_environment/install_deps.sh 2013-09-16
>> 14:38:33 UTC (rev 60166)
>> @@ -48,7 +48,7 @@
>>   # Try to link everything statically. Use this to produce portable
>> versions of blender.
>>   ALL_STATIC=false
>>
>> -THREADS=`cat /proc/cpuinfo | grep cores | uniq | sed -e "s/.*:
>> *\(.*\)/\\1/"`
>> +THREADS=`cat /proc/cpuinfo | grep siblings | uniq | sed -e "s/.*:
>> *\(.*\)/\\1/"`
>>   if [ -z "$THREADS" ]; then
>>     THREADS=1
>>   fi
>>
>> _______________________________________________
>> Bf-blender-cvs mailing list
>> Bf-blender-cvs at blender.org
>> http://lists.blender.org/mailman/listinfo/bf-blender-cvs
>>
>
>