- 11 Sep, 2013 7 commits
-
-
Russ Cox authored
This allows us to make two changes: 1. Force the argument type to be size_t, even on broken systems that declare malloc to take a ulong. 2. Call runtime.throw if malloc fails. (That is, the program crashes; it does not panic.) Fixes #3403. Fixes #5926. R=golang-dev, iant CC=golang-dev https://golang.org/cl/13413047
-
Mikio Hara authored
This CL adds minimal support of Happy Eyeballs-like TCP connection setup to Dialer API. Happy Eyeballs and derivation techniques are described in the following: - Happy Eyeballs: Success with Dual-Stack Hosts http://tools.ietf.org/html/rfc6555 - Analysing Dual Stack Behaviour and IPv6 Quality http://www.potaroo.net/presentations/2012-04-17-dual-stack-quality.pdf Usually, the techniques consist of three components below. - DNS query racers, that run A and AAAA queries in parallel or series - A short list of destination addresses - TCP SYN racers, that run IPv4 and IPv6 transport in parallel or series This CL implements only the latter two. The existing DNS query component gathers together A and AAAA records in series, so we don't touch it here. This CL just uses extended resolveInternetAddr and makes it possible to run multiple Dial racers in parallel. For example, when the given destination is a DNS name and the name has multiple address family A and AAAA records, and it happens on the TCP wildcard network "tcp" with DualStack=true like the following: (&net.Dialer{DualStack: true}).Dial("tcp", "www.example.com:80") The function will return a first established connection either TCP over IPv4 or TCP over IPv6, and close the other connection internally. Fixes #3610. Fixes #5267. Benchmark results on freebsd/amd64 virtual machine, tip vs. tip+12416043: benchmark old ns/op new ns/op delta BenchmarkTCP4OneShot 50696 52141 +2.85% BenchmarkTCP4OneShotTimeout 65775 66426 +0.99% BenchmarkTCP4Persistent 10986 10457 -4.82% BenchmarkTCP4PersistentTimeout 11207 10445 -6.80% BenchmarkTCP6OneShot 62009 63718 +2.76% BenchmarkTCP6OneShotTimeout 78351 79138 +1.00% BenchmarkTCP6Persistent 14695 14659 -0.24% BenchmarkTCP6PersistentTimeout 15032 14646 -2.57% BenchmarkTCP4ConcurrentReadWrite 7215 6217 -13.83% BenchmarkTCP6ConcurrentReadWrite 7528 7493 -0.46% benchmark old allocs new allocs delta BenchmarkTCP4OneShot 36 36 0.00% BenchmarkTCP4OneShotTimeout 36 36 0.00% BenchmarkTCP4Persistent 0 0 n/a% BenchmarkTCP4PersistentTimeout 0 0 n/a% BenchmarkTCP6OneShot 37 37 0.00% BenchmarkTCP6OneShotTimeout 37 37 0.00% BenchmarkTCP6Persistent 0 0 n/a% BenchmarkTCP6PersistentTimeout 0 0 n/a% BenchmarkTCP4ConcurrentReadWrite 0 0 n/a% BenchmarkTCP6ConcurrentReadWrite 0 0 n/a% benchmark old bytes new bytes delta BenchmarkTCP4OneShot 2500 2503 0.12% BenchmarkTCP4OneShotTimeout 2508 2505 -0.12% BenchmarkTCP4Persistent 0 0 n/a% BenchmarkTCP4PersistentTimeout 0 0 n/a% BenchmarkTCP6OneShot 2713 2707 -0.22% BenchmarkTCP6OneShotTimeout 2722 2720 -0.07% BenchmarkTCP6Persistent 0 0 n/a% BenchmarkTCP6PersistentTimeout 0 0 n/a% BenchmarkTCP4ConcurrentReadWrite 0 0 n/a% BenchmarkTCP6ConcurrentReadWrite 0 0 n/a% R=golang-dev, bradfitz, nightlyone, rsc CC=golang-dev https://golang.org/cl/12416043
-
Russ Cox authored
For example, if the pattern is m... there is no need to look in directories not beginning with m. Fixes #5214. R=golang-dev, adg CC=golang-dev https://golang.org/cl/13253049
-
Russ Cox authored
R=golang-dev, iant CC=golang-dev https://golang.org/cl/13420048
-
Russ Cox authored
This is not quite what that issue reports, because this does not involve a DLL. But I wanted to make sure this much was working. Update #4339 R=golang-dev, minux.ma CC=golang-dev https://golang.org/cl/13653043
-
Dave Cheney authored
Add coverage for some uncovered bytes methods. The increase in actual coverage is disapointing small. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13651044
-
Robert Daniel Kortschak authored
Fixes #6353. R=golang-dev, bradfitz, alex.brainman CC=golang-dev https://golang.org/cl/13652043
-
- 10 Sep, 2013 23 commits
-
-
Russ Cox authored
The scan starts at the directory we care about and works backward to the GOPATH root. The error should say the original directory name, not the name of the GOPATH root. Fixes #6175. R=golang-dev, minux.ma CC=golang-dev https://golang.org/cl/13366050
-
Russ Cox authored
Fixes #5764. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13441051
-
Russ Cox authored
The entry for LEAL/LEAQ in these optabs was listed as having two data bytes in the y array. In fact they had and expect no data bytes. However, the general loop expects to be able to look at at least one data byte, to make sure it is not 0x0f. So give them each a single data byte set to 0 (not 0x0f). Since the MOV instructions have the largest optab cases, this requires growing the size of the data array. Clang found this bug because the general o->op[z] == 0x0f test was using z == 22, which was out of bounds. In practice the next byte in memory was probably not 0x0f so it wasn't truly broken. But might as well be clean. Update #5764 R=ken2 CC=golang-dev https://golang.org/cl/13241050
-
Arnaud Ysmal authored
Fixes cpu subtype check when using external linker which sets the CPU_SUBTYPE_LIB64 bit (1<<31). Fixes #6197. R=golang-dev, minux.ma, rsc CC=golang-dev https://golang.org/cl/13248046
-
Brad Fitzpatrick authored
Generated by addca. R=gobot CC=golang-dev https://golang.org/cl/13265044
-
Russ Cox authored
For example, if an x_test.go file contains a syntax error, b.test fails with an error message. But it wasn't printing the same FAIL line that a build failure later would print. This makes all the test failures that happen (once we decide to start running tests) consistently say FAIL. Fixes #4701. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13431044
-
Russ Cox authored
A package main binary (that is, a command) being installed does not mean we can skip the build of the package archive during a test. Fixes #3417. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13462046
-
Russ Cox authored
Fixes #5662. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13650043
-
Russ Cox authored
It was lost when the generic "Notes" support went in. Had to change the test setup, because it precluded even being able test multi-line comments, much less multi-paragraph comments. Now 'godoc sync/atomic' works correctly again. Fixes #6135. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13427045
-
Russ Cox authored
Fixes #6191. Fixes #5426. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13234052
-
Ian Lance Taylor authored
Fixes #5054. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/13609043
-
Ian Lance Taylor authored
Remove test of whether SWIG shared library is older than sources--should be covered by test of package file anyhow. Fixes #5739. R=golang-dev, rsc CC=golang-dev https://golang.org/cl/13352046
-
Russ Cox authored
Fixes #5774. R=golang-dev, adg, r, bradfitz CC=golang-dev https://golang.org/cl/9164043
-
Alexis Imperial-Legrand authored
If using other gdb python scripts loaded before Go's gdb-runtime.py and that have a different init prototype: Traceback (most recent call last): File "/usr/lib/go/src/pkg/runtime/runtime-gdb.py", line 446, in <module> k() TypeError: __init__() takes exactly 3 arguments (1 given) The problem is that gdb keeps all python scripts in the same namespace, so vars() contains them. To avoid that, load helpers one by one. R=iant, rsc CC=gobot, golang-dev https://golang.org/cl/9752044
-
Russ Cox authored
R=golang-dev, bradfitz CC=ail, golang-dev https://golang.org/cl/13441050
-
Russ Cox authored
$INTBITS will not be defined, of course, but that's the best we can do. Fixes #5978. R=golang-dev, iant CC=golang-dev https://golang.org/cl/13253048
-
Russ Cox authored
Fixes #5676. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13250047
-
Russ Cox authored
Fixes #6038. R=iant CC=golang-dev https://golang.org/cl/13649043
-
Russ Cox authored
Various compilers complain about the macro expansion not being used. I fixed a few yesterday. More today. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13643044
-
Keith Randall authored
R=golang-dev, r CC=golang-dev https://golang.org/cl/13336046
-
Mikio Hara authored
R=golang-dev, dave CC=golang-dev https://golang.org/cl/13396051
-
Rob Pike authored
R=golang-dev, remyoudompheng, dominik.honnef, adg CC=golang-dev https://golang.org/cl/13341049
-
Alex Brainman authored
Fixes #6224 R=golang-dev, hcwfrichter, bradfitz CC=golang-dev https://golang.org/cl/13410045
-
- 09 Sep, 2013 10 commits
-
-
Russ Cox authored
Fixes #4900. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13400044
-
Richard Eric Gavaletz authored
Before CL 7065067 calling Next on an element returned either the next/prev element or nil was returned. After the CL if an element was not part of a list e.Next() and e.Prev() will panic. This CL returns to the documented behavior, that Next/Prev returns the next/prev list element or nil. Fixes #6349. R=golang-dev, gri CC=golang-dev https://golang.org/cl/13234051
-
Russ Cox authored
Bug3486 tried to walk the entire file tree, but other tests might be creating and removing files in that tree. In particular, package os creates and removes files in the os directory, and issue 5863 reports failures due to seeing those files appear and then disappear. Change the test to walk just the test tree, which should not be changing. Fixes #5863. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13467045
-
Russ Cox authored
Fixes #6341. R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13512048
-
Russ Cox authored
Fixes #5894. R=golang-dev, iant CC=golang-dev https://golang.org/cl/13303051
-
Russ Cox authored
Fixes #5803. Is it correct behavior? Who knows. R=golang-dev, bradfitz, jgc CC=golang-dev, jgc https://golang.org/cl/13248048
-
Ian Lance Taylor authored
This will bring in the C++ standard library without requiring any special #cgo LDFLAGS options. When using gccgo, just add -lstdc++ to link line; this should do no harm if it is not needed. No tests, since we don't want to assume a C++ compiler. Update #5629 R=golang-dev, minux.ma, rsc CC=golang-dev https://golang.org/cl/13394045
-
Russ Cox authored
If you thought gcc -ansi -pedantic was pedantic, just wait until you meet clang -fsanitize=undefined. I think this addresses all the reported "errors", but we'll need another run to be sure. all.bash still passes. Update #5764 Dave, can you please try again? R=golang-dev, bradfitz CC=golang-dev https://golang.org/cl/13334049
-
Russ Cox authored
The code in question is trying to print a nice error message when a Go EABI binary runs on an OABI machine. Unfortunately, the only way to do that is to use ARM Thumb instructions, which we otherwise don't use. There exist ARM EABI machines that do not support Thumb. We could run on them if not for this OABI check, so disable it. Fixes #5685. R=golang-dev, minux.ma CC=golang-dev https://golang.org/cl/13234050
-
Rémy Oudompheng authored
Fixes #6036. R=golang-dev, bradfitz, rsc CC=golang-dev https://golang.org/cl/12992043
-