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<[email protected]> 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 >> [email protected] >> http://lists.blender.org/mailman/listinfo/bf-blender-cvs >> > > _______________________________________________ Bf-committers mailing list [email protected] http://lists.blender.org/mailman/listinfo/bf-committers
