Skip to content

G
golang
Switch branch/tag
  1. 12 Jun, 2013 11 commits
    • Dmitriy Vyukov's avatar
      runtime: more flexible heap memory mapping on 64-bits · a8ad859c
      Dmitriy Vyukov authored 
      Fixes #5641.

R=golang-dev, dave, daniel.morsing, iant
CC=golang-dev, kcc
https://golang.org/cl/10126044
      a8ad859c
    • Dmitriy Vyukov's avatar
      runtime: fix scheduler race condition · dbcfed93
      Dmitriy Vyukov authored 
      In starttheworld() we assume that P's with local work
are situated in the beginning of idle P list.
However, once we start the first M, it can execute all local G's
and steal G's from other P's.
That breaks the assumption above. Thus starttheworld() will fail
to start some P's with local work.
It seems that it can not lead to very bad things, but still
it's wrong and breaks other assumtions
(e.g. we can have a spinning M with local work).
The fix is to collect all P's with local work first,
and only then start them.

R=golang-dev, rsc
CC=golang-dev
https://golang.org/cl/10051045
      dbcfed93
    • Russ Cox's avatar
      runtime: fix plan9, windows builds · e5cbebc1
      Russ Cox authored 
      TBR=dvyukov
CC=golang-dev
https://golang.org/cl/10229043
      e5cbebc1
    • Russ Cox's avatar
      runtime: fix build · 80efeff2
      Russ Cox authored 
      TBR=dvyukov
CC=golang-dev
https://golang.org/cl/10227044
      80efeff2
    • Russ Cox's avatar
      runtime: adjust traceback / garbage collector boundary · e58f798c
      Russ Cox authored 
      The garbage collection routine addframeroots is duplicating
logic in the traceback routine that calls it, sometimes correctly,
sometimes incorrectly, sometimes incompletely.
Pass necessary information to addframeroots instead of
deriving it anew.

Should make addframeroots significantly more robust.
It's certainly smaller.

Also try to standardize on uintptr for saved pc, sp values.

Will make CL 10036044 trivial.

R=golang-dev, dave, dvyukov
CC=golang-dev
https://golang.org/cl/10169045
      e58f798c
    • Russ Cox's avatar
      cmd/go: diagnose invalid coverage runs · 7ea75a5f
      Russ Cox authored 
      # bufio
coverage analysis cannot handle package (bufio_test imports testing imports bufio)
# bytes
coverage analysis cannot handle package (bytes_test imports encoding/base64 imports bytes)
# crypto/cipher
coverage analysis cannot handle package (cipher_test imports crypto/aes imports crypto/cipher)
# debug/dwarf
coverage analysis cannot handle package (dwarf_test imports debug/elf imports debug/dwarf)
# errors
coverage analysis cannot handle package (errors_test imports fmt imports errors)
# flag
coverage analysis cannot handle package (flag_test imports testing imports flag)
# fmt
coverage analysis cannot handle package (fmt_test imports testing imports fmt)
# go/ast
coverage analysis cannot handle package (ast_test imports go/format imports go/ast)
# image
coverage analysis cannot handle package (image_test imports image/gif imports image)
# io
coverage analysis cannot handle package (io_test imports bytes imports io)
# math
coverage analysis cannot handle package (math_test imports fmt imports math)
# net/http
coverage analysis cannot handle package (http_test imports net/http/httptest imports net/http)
# os
coverage analysis cannot handle package (os_test imports flag imports os)
# path/filepath
coverage analysis cannot handle package (filepath_test imports io/ioutil imports path/filepath)
# reflect
coverage analysis cannot handle package (reflect_test imports flag imports reflect)
# runtime
coverage analysis cannot handle package (runtime_test imports fmt imports runtime)
# runtime/pprof
coverage analysis cannot handle package (pprof_test imports testing imports runtime/pprof)
# sort
coverage analysis cannot handle package (sort_test imports testing imports sort)
# strconv
coverage analysis cannot handle package (strconv_test imports fmt imports strconv)
# strings
coverage analysis cannot handle package (strings_test imports testing imports strings)
# sync
coverage analysis cannot handle package (sync_test imports fmt imports sync)
# sync/atomic
coverage analysis cannot handle package (atomic_test imports testing imports sync/atomic)
# syscall
coverage analysis cannot handle package (syscall_test imports flag imports syscall)
# text/tabwriter
coverage analysis cannot handle package (tabwriter_test imports testing imports text/tabwriter)
# time
coverage analysis cannot handle package (time_test imports encoding/gob imports time)
# unicode
coverage analysis cannot handle package (unicode_test imports testing imports unicode)
# unicode/utf8
coverage analysis cannot handle package (utf8_test imports bytes imports unicode/utf8)

R=r
CC=golang-dev
https://golang.org/cl/10216043
      7ea75a5f
    • Shenghou Ma's avatar
      cmd/go: clarify test filenames in help messages · da634dd7
      Shenghou Ma authored 
      Fixes #5655.

R=golang-dev, r
CC=golang-dev
https://golang.org/cl/9944044
      da634dd7
    • David Symonds's avatar
      C: add Josh Hoak (Google CLA). · e49bda65
      David Symonds authored 
      R=golang-dev, rsc
CC=golang-dev, jhoak
https://golang.org/cl/10217043
      e49bda65
    • Rob Pike's avatar
      cmd/go: use -o option of cover tool · bc7e2662
      Rob Pike authored 
      Separates correct from erroneous output so errors running the tool will appear
in the log.

R=rsc
CC=golang-dev
https://golang.org/cl/10191043
      bc7e2662
    • Ian Lance Taylor's avatar
      cmd/gc: save local var list before inlining · ae5e791e
      Ian Lance Taylor authored 
      This avoids problems with inlining in genwrappers, which
occurs after functions have been compiled.  Compiling a
function may cause some unused local vars to be removed from
the list.  Since a local var may be unused due to
optimization, it is possible that a removed local var winds up
beingused in the inlined version, in which case hilarity
ensues.

Fixes #5515.

R=golang-dev, khr, dave
CC=golang-dev
https://golang.org/cl/10210043
      ae5e791e
    • Russ Cox's avatar
      cmd/gc: turn race detector off for tail-call method wrapper functions · e440354c
      Russ Cox authored 
      It was off in the old implementation (because there was no high-level
description of the function at all). Maybe some day the race detector
should be fixed to handle the wrapper and then enabled for it, but there's
no reason that has to be today.

R=golang-dev
TBR=dvyukov
CC=golang-dev
https://golang.org/cl/10037045
      e440354c
  3. 11 Jun, 2013 9 commits
  5. 10 Jun, 2013 17 commits
  7. 09 Jun, 2013 3 commits