[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

[Sergey Sharybin]

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
>



-- 
With best regards, Sergey Sharybin