{"push_id":602115721,"size":323,"distinct_size":323,"ref":"refs/heads/master","head":"66dabb5f9e0745ca35eed5ae4352c6025671dd13","before":"7c7126cfeb82894229b9c3d5109e4b04e6cfde0c","commits":[{"sha":"3dd029aa7e5b3e770e21bd8b03981c3ef0b59c23","author":{"email":"e2f4d2eec1d9a1f781345cdd7260d7d52e6468ae@gmail.com","name":"Mikio Hara"},"message":"net: enable TestTCPReadWriteAllocs in short mode\n\nThe change 2096 removed unwanted allocations and a few noises in test\nusing AllocsPerRun. Now it's safe to enable this canary test on netpoll\nhotpaths.\n\nChange-Id: Icdbee813d81c1410a48ea9960d46447042976905\nReviewed-on: https://go-review.googlesource.com/5713\nReviewed-by: Dmitry Vyukov <dvyukov@google.com>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/3dd029aa7e5b3e770e21bd8b03981c3ef0b59c23"},{"sha":"edcad8639a902741dc49f77d000ed62b0cc6956f","author":{"email":"05e8181be3150e47da8fe2e1f18d381c75496ff4@google.com","name":"Dmitry Vyukov"},"message":"sync: add active spinning to Mutex\n\nCurrently sync.Mutex is fully cooperative. That is, once contention is discovered,\nthe goroutine calls into scheduler. This is suboptimal as the resource can become\nfree soon after (especially if critical sections are short). Server software\nusually runs at ~~50% CPU utilization, that is, switching to other goroutines\nis not necessary profitable.\n\nThis change adds limited active spinning to sync.Mutex if:\n1. running on a multicore machine and\n2. GOMAXPROCS>1 and\n3. there is at least one other running P and\n4. local runq is empty.\nAs opposed to runtime mutex we don't do passive spinning,\nbecause there can be work on global runq on on other Ps.\n\nbenchmark                   old ns/op     new ns/op     delta\nBenchmarkMutexNoSpin        1271          1272          +0.08%\nBenchmarkMutexNoSpin-2      702           683           -2.71%\nBenchmarkMutexNoSpin-4      377           372           -1.33%\nBenchmarkMutexNoSpin-8      197           190           -3.55%\nBenchmarkMutexNoSpin-16     131           122           -6.87%\nBenchmarkMutexNoSpin-32     170           164           -3.53%\nBenchmarkMutexSpin          4724          4728          +0.08%\nBenchmarkMutexSpin-2        2501          2491          -0.40%\nBenchmarkMutexSpin-4        1330          1325          -0.38%\nBenchmarkMutexSpin-8        684           684           +0.00%\nBenchmarkMutexSpin-16       414           372           -10.14%\nBenchmarkMutexSpin-32       559           469           -16.10%\n\nBenchmarkMutex                 19.1          19.1          +0.00%\nBenchmarkMutex-2               81.6          54.3          -33.46%\nBenchmarkMutex-4               143           100           -30.07%\nBenchmarkMutex-8               154           156           +1.30%\nBenchmarkMutex-16              140           159           +13.57%\nBenchmarkMutex-32              141           163           +15.60%\nBenchmarkMutexSlack            33.3          31.2          -6.31%\nBenchmarkMutexSlack-2          122           97.7          -19.92%\nBenchmarkMutexSlack-4          168           158           -5.95%\nBenchmarkMutexSlack-8          152           158           +3.95%\nBenchmarkMutexSlack-16         140           159           +13.57%\nBenchmarkMutexSlack-32         146           162           +10.96%\nBenchmarkMutexWork             154           154           +0.00%\nBenchmarkMutexWork-2           89.2          89.9          +0.78%\nBenchmarkMutexWork-4           139           86.1          -38.06%\nBenchmarkMutexWork-8           177           162           -8.47%\nBenchmarkMutexWork-16          170           173           +1.76%\nBenchmarkMutexWork-32          176           176           +0.00%\nBenchmarkMutexWorkSlack        160           160           +0.00%\nBenchmarkMutexWorkSlack-2      103           99.1          -3.79%\nBenchmarkMutexWorkSlack-4      155           148           -4.52%\nBenchmarkMutexWorkSlack-8      176           170           -3.41%\nBenchmarkMutexWorkSlack-16     170           173           +1.76%\nBenchmarkMutexWorkSlack-32     175           176           +0.57%\n\n\"No work\" benchmarks are not very interesting (BenchmarkMutex and\nBenchmarkMutexSlack), as they are absolutely not realistic.\n\nFixes #8889\n\nChange-Id: I6f14f42af1fa48f73a776fdd11f0af6dd2bb428b\nReviewed-on: https://go-review.googlesource.com/5430\nReviewed-by: Rick Hudson <rlh@golang.org>\nRun-TryBot: Dmitry Vyukov <dvyukov@google.com>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/edcad8639a902741dc49f77d000ed62b0cc6956f"},{"sha":"291bf1f03feece8360e080856e835bcc2cd1f521","author":{"email":"a2f0bf29257797dd66800b4fbb9945c3e216c004@golang.org","name":"David Crawshaw"},"message":"cmd/go: limit default darwin/arm parallelism\n\niOS devices can only run tests serially.\n\nChange-Id: I3f4e7abddf812a186895d9d5138999c8bded698f\nReviewed-on: https://go-review.googlesource.com/5751\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>\nReviewed-by: Hyang-Ah Hana Kim <hyangah@gmail.com>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/291bf1f03feece8360e080856e835bcc2cd1f521"},{"sha":"d9859ad40438cc27415ec294e9a06bb58fa24e9a","author":{"email":"a6752097faebf1f02d80107d77a9be009089afe3@golang.org","name":"Robert Griesemer"},"message":"math/big: fix several issues with string->Float conversion\n\nChange-Id: I7bf7154e2d8d779fdf7f1d2bb561a06ad174f3b0\nReviewed-on: https://go-review.googlesource.com/4883\nReviewed-by: Alan Donovan <adonovan@google.com>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/d9859ad40438cc27415ec294e9a06bb58fa24e9a"},{"sha":"c2317db2f9bf9f097f0bc297004a8f581b944206","author":{"email":"db68fba232b2c49ed3e9d3081df349dde7b61dfd@golang.org","name":"Adam Langley"},"message":"crypto/x509: don't reject certs with critical policy extensions.\n\nThere was a missing continue that caused certificates with critical\ncertificate-policy extensions to be rejected. Additionally, that code\nstructure in general was prone to exactly that bug so I changed it\naround to hopefully be more robust in the future.\n\nFixes #9964.\n\nChange-Id: I58fc6ef3a84c1bd292a35b8b700f44ef312ec1c1\nReviewed-on: https://go-review.googlesource.com/5670\nReviewed-by: Andrew Gerrand <adg@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/c2317db2f9bf9f097f0bc297004a8f581b944206"},{"sha":"8f8d066bfd86c736e35f1c0e36ae01fb0bdd7190","author":{"email":"db68fba232b2c49ed3e9d3081df349dde7b61dfd@golang.org","name":"Adam Langley"},"message":"crypto/tls: allow larger initial records.\n\nSome servers which misunderstood the point of the CertificateRequest\nmessage send huge reply records. These records are large enough that\nthey were considered “insane” by the TLS code and rejected.\n\nThis change removes the sanity test for record lengths. Although the\nmaxCiphertext test still remains, just above, which (roughly) enforces\nthe 16KB protocol limit on record sizes:\nhttps://tools.ietf.org/html/rfc5246#section-6.2.1\n\nFixes #8928.\n\nChange-Id: Idf89a2561b1947325b7ddc2613dc2da638d7d1c9\nReviewed-on: https://go-review.googlesource.com/5690\nReviewed-by: Andrew Gerrand <adg@golang.org>\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/8f8d066bfd86c736e35f1c0e36ae01fb0bdd7190"},{"sha":"e7fae6854008c163ba83e2eac52ae9f12003a9a9","author":{"email":"db68fba232b2c49ed3e9d3081df349dde7b61dfd@golang.org","name":"Adam Langley"},"message":"crypto/x509: allow wildcards only as the first label.\n\nRFC 6125 now specifies that wildcards are only allowed for the leftmost\nlabel in a pattern: https://tools.ietf.org/html/rfc6125#section-6.4.3.\n\nThis change updates Go to match the behaviour of major browsers in this\nrespect.\n\nFixes #9834.\n\nChange-Id: I37c10a35177133624568f2e0cf2767533926b04a\nReviewed-on: https://go-review.googlesource.com/5691\nReviewed-by: Andrew Gerrand <adg@golang.org>\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/e7fae6854008c163ba83e2eac52ae9f12003a9a9"},{"sha":"abf8bbb709137c7212704dd0fb777b9c549a5fe1","author":{"email":"db68fba232b2c49ed3e9d3081df349dde7b61dfd@golang.org","name":"Adam Langley"},"message":"crypto/x509: make behaviour of absolute DNS names match Chromium.\n\nPreviously, we didn't handle absolute DNS names in certificates the same\nway as Chromium, and we probably shouldn't diverge from major browsers.\n\nChange-Id: I56a3962ad1002f68b5dbd65ae90991b82c2f5629\nReviewed-on: https://go-review.googlesource.com/5692\nReviewed-by: Andrew Gerrand <adg@golang.org>\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/abf8bbb709137c7212704dd0fb777b9c549a5fe1"},{"sha":"90590a0cd210686a34bd4b645e0ef3ae468795e4","author":{"email":"4dc7c9ec434ed06502767136789763ec11d2c4b7@golang.org","name":"Rob Pike"},"message":"cmd/asm: add alias for MOVOA=MOVO on amd64\n\nThe alias should exist for both 386 and amd64.\nThere were a few others missing as well. Add them.\n\nChange-Id: Ia0c3e71abc79f67a7a66941c0d932a8d5d6e9989\nReviewed-on: https://go-review.googlesource.com/5800\nReviewed-by: Russ Cox <rsc@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/90590a0cd210686a34bd4b645e0ef3ae468795e4"},{"sha":"aff84b171c4188cc79bc23bffa3b284b6c02c2d7","author":{"email":"4dc7c9ec434ed06502767136789763ec11d2c4b7@golang.org","name":"Rob Pike"},"message":"cmd/asm: fix build for new aliases\n\nMissing leading A on names.\n\nChange-Id: I6f3a66bdd3a21220f45a898f0822930b6a7bfa38\nReviewed-on: https://go-review.googlesource.com/5801\nReviewed-by: Rob Pike <r@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/aff84b171c4188cc79bc23bffa3b284b6c02c2d7"},{"sha":"c651fdc0cf2ba7986ce0a0a23b4dbb44b6ecdae3","author":{"email":"a6752097faebf1f02d80107d77a9be009089afe3@golang.org","name":"Robert Griesemer"},"message":"math/big: change Float.SetMantExp to always multiply mant by 2**exp\n\nChange-Id: If840e647376a2141f8c17729f7ef251bfff13f5f\nReviewed-on: https://go-review.googlesource.com/5810\nReviewed-by: Rob Pike <r@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/c651fdc0cf2ba7986ce0a0a23b4dbb44b6ecdae3"},{"sha":"5ce9fde8b64a07a7cbfbe43c9451e2d1d536c972","author":{"email":"a6752097faebf1f02d80107d77a9be009089afe3@golang.org","name":"Robert Griesemer"},"message":"go/ast, go/parser: correct End() position for *ast.EmptyStmt\n\n- added a new field ast.EmptyStmt.Implicit to indicate explicit\n  or implicit semicolon\n- fix ast.EmptyStmt.End() accordingly\n- adjusted parser and added test case\n\nFixes #9979.\n\nChange-Id: I72b0983b3a0cabea085598e1bf6c8df629776b57\nReviewed-on: https://go-review.googlesource.com/5720\nReviewed-by: Russ Cox <rsc@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/5ce9fde8b64a07a7cbfbe43c9451e2d1d536c972"},{"sha":"c4b8f9127a26680432ddf329b409b8ec595ac9ae","author":{"email":"a6752097faebf1f02d80107d77a9be009089afe3@golang.org","name":"Robert Griesemer"},"message":"math/big: fix test for 32bit platforms (fix build)\n\nChange-Id: I73509cd0e0866cfe2b2ae46a26fb4d043dd050c4\nReviewed-on: https://go-review.googlesource.com/5840\nReviewed-by: Alan Donovan <adonovan@google.com>\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/c4b8f9127a26680432ddf329b409b8ec595ac9ae"},{"sha":"42289a4f394a31e88d7288b91887c614f218808e","author":{"email":"bfcdf3e6ca6cef45543bfbb57509c92aec9a39fb@cheney.net","name":"Dave Cheney"},"message":"syscall: rebuild zsyscall_linux_*.go\n\nRebuild the zsyscall_linux_*.go files in preperation for #9974\n\nThe only change is the ppc64/ppc64le files which were not rebuilt when\nsyscall.use was added.\n\nChange-Id: I804c63731e4900c782025de04ea3585d99688958\nReviewed-on: https://go-review.googlesource.com/5831\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/42289a4f394a31e88d7288b91887c614f218808e"},{"sha":"e31e35a0dea212bba08d85449554dad2969a2d91","author":{"email":"838bd060584c511f21c75c69ea04b7b120d86af0@golang.org","name":"Rick Hudson"},"message":"runtime: reset gcscanvalid and gcworkdone when GODEBUG=gctrace=2\n\nWhen GODEBUG=gctrace=2 two gcs are preformed. During the first gc\nthe stack scan sets the g's gcscanvalid and gcworkdone flags to true\nindicating that the stacks have to be scanned and do not need to\nbe rescanned. These need to be reset to false for the second GC so the\nstacks are rescanned, otherwise if the only pointer to an object is\non the stack it will not be discovered and the object will be freed.\nTypically this will include the object that was just allocated in\nthe mallocgc call that initiated the GC.\n\nChange-Id: Ic25163f4689905fd810c90abfca777324005c02f\nReviewed-on: https://go-review.googlesource.com/5861\nReviewed-by: Russ Cox <rsc@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/e31e35a0dea212bba08d85449554dad2969a2d91"},{"sha":"7ce02613870a67f26055836ded66591be148b82b","author":{"email":"bfcdf3e6ca6cef45543bfbb57509c92aec9a39fb@cheney.net","name":"Dave Cheney"},"message":"cmd/dist: respect runtime.NumCPU when bootstrapping arm hosts\n\nThis is a reproposal of CL 2957. This reproposal restricts the\nscope of this change to just arm systems.\n\nWith respect to rsc's comments on 2957, on all my arm hosts they perform\nthe build significantly faster with this change in place.\n\nChange-Id: Ie09be1a73d5bb777ec5bca3ba93ba73d5612d141\nReviewed-on: https://go-review.googlesource.com/5834\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/7ce02613870a67f26055836ded66591be148b82b"},{"sha":"9b3ccc082f6bda01727fc98096f7d197bba830db","author":{"email":"bfcdf3e6ca6cef45543bfbb57509c92aec9a39fb@cheney.net","name":"Dave Cheney"},"message":"syscall: split implementation of Pipe/Pipe2 per GOOS\n\nUpdates #9974\n\nThis proposal moves the definition of Pipe an Pipe2 from the generic\nsyscall_linux.go to the GOOS specific variants. This is in preparation\nfor the arm64 port.\n\nFor platforms where pipe2(2) is not supported in the minimum 2.6.23 kernel,\namd64 and 386, we retain pipe(2). For all other platforms pipe(2) is removed\nand Pipe forwards to pipe2(2).\n\nBecause mksycall.pl does not sort symbols before generating the output file\nthe diff includes some unavoidable code moves as Pipe and Pipe2 are processed\nlatter in the run.\n\nDiscussion: https://groups.google.com/forum/#!topic/golang-dev/zpeFtN2z5Fc\n\nChange-Id: Ie26d6761eeb9760dbaff974ee8bc0d57a9ceaee4\nReviewed-on: https://go-review.googlesource.com/5833\nReviewed-by: Brad Fitzpatrick <bradfitz@golang.org>\nRun-TryBot: Brad Fitzpatrick <bradfitz@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/9b3ccc082f6bda01727fc98096f7d197bba830db"},{"sha":"18fd940620708813dd33a2d5169c9861278102af","author":{"email":"a6752097faebf1f02d80107d77a9be009089afe3@golang.org","name":"Robert Griesemer"},"message":"math/big: permit passing of (possibly nil) *Float to MantExp to avoid allocation\n\nChange-Id: Ia92eea833283f8b16fa09d4ca1c9cb3bc0eb18a2\nReviewed-on: https://go-review.googlesource.com/5870\nReviewed-by: Rob Pike <r@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/18fd940620708813dd33a2d5169c9861278102af"},{"sha":"b0d8cb6b3f6ff2d0df66f96e93e33747acc36882","author":{"email":"db68fba232b2c49ed3e9d3081df349dde7b61dfd@golang.org","name":"Adam Langley"},"message":"doc/go1.5: mention the change in certificate wildcard handling.\n\nhttps://go-review.googlesource.com/#/c/5691/ shouldn't affect anyone\nbecause it reflects the RFC and behaviour of modern browsers, but it's\nprobably worth mentioning.\n\nChange-Id: I8a8eedaac5226b1d8018dc56de6b2d41bd82a642\nReviewed-on: https://go-review.googlesource.com/5862\nReviewed-by: Rob Pike <r@golang.org>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/b0d8cb6b3f6ff2d0df66f96e93e33747acc36882"},{"sha":"1fda57ba725044e61148258ea9e98080b0a35260","author":{"email":"bfcdf3e6ca6cef45543bfbb57509c92aec9a39fb@cheney.net","name":"Dave Cheney"},"message":"syscall: split implementation of Dup2 per linux GOOS\n\nUpdates #9974\n\nThis proposal moves the definition of Dup2 from the generic syscall_linux.go\nto the GOOS specific variants. This is in preparation for the arm64 port.\n\nFor all existing platforms Dup2 is not affected. When arm64 is added we'll use\neither a forwarding method to Dup3 or\n\n\t//sysnb        Dup2(oldfd int, newfd int) (err error) = SYS_DUP3\n\nBecause mksycall.pl does not sort symbols before generating the output file\nthe diff includes some unavoidable code moves as Dup2 is processed latter in\nthe run.\n\nDiscussion: https://groups.google.com/forum/#!topic/golang-dev/zpeFtN2z5Fc\n\nChange-Id: Icdedf55bb29e749c4230e1ee371bf9d0bd0cfb38\nReviewed-on: https://go-review.googlesource.com/5835\nReviewed-by: Minux Ma <minux@golang.org>\nRun-TryBot: Dave Cheney <dave@cheney.net>","distinct":true,"url":"https://api.github.com/repos/curtiszimmerman/go/commits/1fda57ba725044e61148258ea9e98080b0a35260"}]}