Correct.

Ted Unangst <[email protected]> wrote:

> I was looking at the chrome profiler the other day and noticed it seems to
> start more threads than I have CPUs online (since hw.smt is disabled). This is
> not great for performance.
> 
> Fortunately, I did not have to download and read the chromium source, because
> I found this snippet in the patches directory. :)
> 
>  int SysInfo::NumberOfProcessors() {
>     int mib[] = {CTL_HW, HW_NCPU};
> 
> As with other programs using this systl, this will overcount CPUs and create
> too many threads. It should be HW_NCPUONLINE.
> 

Reply via email to