FAQ
On 2012/09/20 19:22:20, dvyukov wrote:
https://codereview.appspot.com/6496054/diff/23001/src/pkg/net/fd_unix.go#newcode303
src/pkg/net/fd_unix.go:303: pollN := runtime.GOMAXPROCS(0)
Please move this out of newFD().
Done.

https://codereview.appspot.com/6496054/

Search Discussions

  • Sebastien Paolacci at Sep 21, 2012 at 4:43 pm

    On 2012/09/21 14:36:13, rsc wrote:
    I would consider that anything beyond NumCPU is useless, that
    anything
    smaller is not really damaging, and that anything greater than 8
    currently doesn't bring any improvements.

    Min(NumCPU, 8) + lazy instantiation as suggested by Dave/Russ ?
    This is okay with me but I would like dvyukov to agree.
    Russ
    Hello Russ,

    Just for the record, last patch round is stabilizing on `Min(GOMAXPROCS,
    NumCPU, 8)' with lazy instantiation. CL's description also mentions (but
    does not explain) that choice.

    Sebastien

    http://codereview.appspot.com/6496054/
  • Dvyukov at Sep 21, 2012 at 8:16 pm

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupgolang-dev @
categoriesgo
postedSep 21, '12 at 11:13a
activeSep 21, '12 at 8:16p
posts3
users2
websitegolang.org

2 users in discussion

Sebastien Paolacci: 2 posts Dvyukov: 1 post

People

Translate

site design / logo © 2022 Grokbase