- 13 Jan, 2020 3 commits
-
-
Constantin Konstantinidis authored
Fixes #36375 Change-Id: I407a1db23868880b83e73bc136d274659483fb69 Reviewed-on: https://go-review.googlesource.com/c/go/+/214437 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Keith Randall authored
Change the type printer to take a map of types that we're currently printing. When we happen upon a type that we're already in the middle of printing, print a reference to it instead. A reference to another type is built using the offset of the first byte of that type's string representation in the result. To facilitate that computation (and it's probably more efficient, regardless), we print the type to a buffer as we go, and build the string at the end. It would be nice to use string.Builder instead of bytes.Buffer, but string.Builder wasn't around in Go 1.4, and we'd like to bootstrap from that version. Fixes #29312 Change-Id: I49d788c1fa20f770df7b2bae3b9979d990d54803 Reviewed-on: https://go-review.googlesource.com/c/go/+/214239Reviewed-by: Robert Griesemer <gri@golang.org>
-
fujimoto kyosuke authored
The comment contained a link that had a file name and ID that no longer existed, so change to the URL of the corresponding part of the latest page. Change-Id: I74e0885aabf470facc39b84035f7a83fef9c6a8e GitHub-Last-Rev: 5681c84d9f1029449da6860c65a1d9a128296e85 GitHub-Pull-Request: golang/go#36514 Reviewed-on: https://go-review.googlesource.com/c/go/+/214181 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
- 10 Jan, 2020 10 commits
-
-
Ian Lance Taylor authored
Whenever more than 1/4 of the timers on a P's heap are deleted, remove them from the heap. Change-Id: Iff63ed3d04e6f33ffc5c834f77f645c52c007e52 Reviewed-on: https://go-review.googlesource.com/c/go/+/214299 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Michael Knyszek <mknyszek@google.com>
-
Bryan C. Mills authored
Updates #36489 Change-Id: I1ca215ba0a64a31d662134385b8be46bb4ba4434 Reviewed-on: https://go-review.googlesource.com/c/go/+/214282 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Matthew Dempsky authored
We had a few test cases to make sure checkptr didn't have certain false positives, but none to test for any true positives. This CL fixes that. Updates #22218. Change-Id: I24c02e469a4af43b1748829a9df325ce510f7cc4 Reviewed-on: https://go-review.googlesource.com/c/go/+/214238 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Austin Clements <austin@google.com>
-
Jeremy Faller authored
Make file names consistent, using only forward slashes in the path. Fixes #36495 Change-Id: I337119d6dff233ab9d4bfe757147ec25961ae021 Reviewed-on: https://go-review.googlesource.com/c/go/+/214286 Run-TryBot: Jeremy Faller <jeremy@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Jay Conrod authored
This pulls in two new commits: a fix for a test broken on plan9 and a correction to a comment. Change-Id: I31cf046a5073872714c758894732b2fcc800ab6b Reviewed-on: https://go-review.googlesource.com/c/go/+/214289 Run-TryBot: Jay Conrod <jayconrod@google.com> Reviewed-by: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Ian Lance Taylor authored
When adjustTimers sees a timerModifiedEarlier or timerModifiedLater, it removes it from the heap, leaving a new timer at that position in the heap. We were accidentally skipping that new timer in our loop. In some unlikely cases this could cause adjustTimers to look at more timers than necessary. Change-Id: Ic71e54c175ab7d86a7fa46f1497aca71ed1c43cc Reviewed-on: https://go-review.googlesource.com/c/go/+/214338 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Michael Knyszek <mknyszek@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Filippo Valsorda authored
This imports CL 213880. Fixes #35511 Change-Id: I55d18713bdac8fa556ba5a2aced922f80d1ac970 Reviewed-on: https://go-review.googlesource.com/c/go/+/214078Reviewed-by: Cherry Zhang <cherryyz@google.com> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
bxq2011hust authored
Change-Id: Ic7997d1f747152afec78e8e439770166029f34ec GitHub-Last-Rev: 6a07f25056c960dc8684cd6eac22cd3405a936c8 GitHub-Pull-Request: golang/go#36491 Reviewed-on: https://go-review.googlesource.com/c/go/+/214298 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Tobias Klauser authored
Go 1.14 will drop support for macOS 10.10, see #23011 This reverts CL 125304 Updates #26475 Updates #26513 Change-Id: Ia13eef30f22d67103f7ae45424124fbb116e1261 Reviewed-on: https://go-review.googlesource.com/c/go/+/214057 Run-TryBot: Tobias Klauser <tobias.klauser@gmail.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Ian Lance Taylor authored
Go 1.14 will be the last to support macOS 10.11. Go 1.15 will require macOS 10.12 (Sierra). Updates #23011 Change-Id: I8fff555e5b8fffe088e7e960e77fac9558cb74e4 Reviewed-on: https://go-review.googlesource.com/c/go/+/210137Reviewed-by: Andrew Bonventre <andybons@golang.org>
-
- 09 Jan, 2020 10 commits
-
-
Matthew Dempsky authored
Updates #34964. Change-Id: I5afb2c1e77a9a47358a1d0d108c4a787d7172b94 Reviewed-on: https://go-review.googlesource.com/c/go/+/214217 Run-TryBot: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Austin Clements <austin@google.com>
-
Jay Conrod authored
In "Module support" section, there were two mentions of "Go 1.13", assuming that's the latest version. Rather than update these to 1.14, this CL changes those to "The go command". Also, a minor change in wording for finding go.mod files. Change-Id: Id194be9405b540f221464814e71c361a22cc0f55 Reviewed-on: https://go-review.googlesource.com/c/go/+/214140 Run-TryBot: Jay Conrod <jayconrod@google.com> Reviewed-by: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Austin Clements authored
Hopefully we'll have the remaining safety violations in the standard library ironed out by 1.15. We also fix a minor (but important) typo while we're here. Updates #34964, #34972. Change-Id: Ic72fd4d9411b749f8c0cea87e95ab68347009893 Reviewed-on: https://go-review.googlesource.com/c/go/+/214118Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Dmitri Shuralyov authored
This update was automatically generated using the updatecontrib command: cd gotip go run golang.org/x/build/cmd/updatecontrib With minor manual changes based on publicly available information to canonicalize letter case and formatting for a few names. Actions taken (relative to CONTRIBUTORS at origin/master): Added Aaron Beitch <aaronb@arista.com> Added Adam Shelton <aashelt90@gmail.com> Added Aditya Harindar <aditya.harindar@gmail.com> Added Alec Benzer <alec.benzer@gmail.com> Added Alex Buchanan <buchanae@gmail.com> Added Alex Gaynor <alex@alloy.us> Added Alex Harford <alex.harford@saucelabs.com> Added Alex Zhirov <azhirov@google.com> Added Alexandr Mayorskiy <a.mayorskiy@corp.mail.ru> Added An Xiao <hac@zju.edu.cn> Added Andrei Enshin <b1os@bk.ru> Added Andrew Medvedev <andrew.y.medvedev@gmail.com> Added Andrew Stormont <astormont@racktopsystems.com> Added Antonio Huete Jimenez <tuxillo@quantumachine.net> Added Arda Güçlü <ardaguclu@gmail.com> Added Artem Alekseev <artem.alekseev@intel.com> Added Ayan George <ayan@ayan.net> Added Barnaby Keene <accounts@southcla.ws> Added Ben Toews <mastahyeti@gmail.com> Added Benjamin Wuethrich <benjamin.wuethrich@gmail.com> Added Brandon Philips <brandon@ifup.org> Added Carlos Amedee <carlos@golang.org> Added Changkun Ou <hi@changkun.us> Added Chauncy Cullitan <chauncyc@google.com> Added Clint J. Edwards <clint.j.edwards@gmail.com> Added Daisuke Suzuki <daisuzu@gmail.com> Added Dan Scales <danscales@google.com> Added David Bond <davidsbond93@gmail.com> Added Denis Isaev <idenx@yandex.com> Added Domas Tamašauskas <puerdomus@gmail.com> Added Duco van Amstel <duco.vanamstel@gmail.com> Added Eduardo Villaseñor <evillasrmx@gmail.com> Added Eric Biggers <ebiggers@google.com> Added Eric Brown <browne@vmware.com> Added Eric Rutherford <erutherford@gmail.com> Added Gabriel Rosenhouse <rosenhouse@gmail.com> Added George Hartzell <hartzell@alerce.com> Added Gert Cuykens <gert.cuykens@gmail.com> Added Ghazni Nattarshah <ghazni.nattarshah@gmail.com> Added GitHub User DQNEO (188741) <dqneoo@gmail.com> Added GitHub User Matts966 (28551465) <Matts966@users.noreply.github.com> Added GitHub User ajz01 (4744634) <ajzdenek@gmail.com> Added GitHub User andig (184815) <cpuidle@gmx.de> Added GitHub User jinmiaoluo (39730824) <jinmiaoluo@icloud.com> Added GitHub User maltalex (10195391) <code@bit48.net> Added GitHub User po3rin (29445112) <abctail30@gmail.com> Added GitHub User pokutuna (57545) <popopopopokutuna@gmail.com> Added GitHub User ramenjuniti (32011829) <ramenjuniti@gmail.com> Added GitHub User skanehira (7888591) <sho19921005@gmail.com> Added GitHub User witchard (4994659) <witchard@hotmail.co.uk> Added GitHub User zikaeroh (48577114) <zikaeroh@gmail.com> Added Günther Noack <gnoack@google.com> Added Hasit Bhatt <hasit.p.bhatt@gmail.com> Added Howard Zhang <howard.zhang@arm.com> Added Huan Du <i@huandu.me> Added Ignacio Hagopian <jsign.uy@gmail.com> Added Ilya Sinelnikov <sidhmangh@gmail.com> Added Irbe Krumina <irbekrm@gmail.com> Added Isfan Azhabil <isfan.azhabil@tokopedia.com> Added Ivan Trubach <mr.trubach@icloud.com> Added Jaap Aarts <jaap.aarts1@gmail.com> Added Jeremy Faller <jeremy@golang.org> Added Johan Jansson <johan.jansson@iki.fi> Added John Papandriopoulos <jpap.code@gmail.com> Added Jorge L. Fatta <jorge.fatta@auth0.com> Added Josa Gesell <josa@gesell.me> Added Julian Tibble <julian.tibble@gmail.com> Added Jun Zhang <jim.zoumo@gmail.com> Added Jędrzej Szczepaniak <jbszczepaniak@gmail.com> Added Kalman Bekesi <kalmanb@google.com> Added Katharine Berry <ktbry@google.com> Added Keisuke Kishimoto <keisuke.kishimoto@gmail.com> Added Kevan Swanberg <kevswanberg@gmail.com> Added Koki Tomoshige <tomocy.dev@gmail.com> Added Liz Rice <liz@lizrice.com> Added Lorenz Brun <lorenz@brun.one> Added Luke Young <bored-engineer@users.noreply.github.com> Added Marc Sanmiquel <marcsanmiquel@gmail.com> Added Marko Kungla <marko.kungla@gmail.com> Added Mikhail Fesenko <proggga@gmail.com> Added Mohit Verma <vmohit.93@gmail.com> Added Moritz Fain <moritz@fain.io> Added Nathan Dias <nathan.dias@orijtech.com> Added Nikita Vanyasin <nikita.vanyasin@gmail.com> Added Nuno Cruces <ncruces@users.noreply.github.com> Added Oliver Powell <oliverpowell84@gmail.com> Added Panos Georgiadis <pgeorgiadis@suse.de> Added Pantelis Sampaziotis <psampaz@gmail.com> Added Paulo Gomes <paulo.gomes.uk@gmail.com> Added Prashant Agrawal <prashant.a.vjti@gmail.com> Added Ricardo Seriani <ricardo.seriani@gmail.com> Added Rob Findley <rfindley@google.com> Added Robin Zhong <robin@robinzhong.co> Added Rohan Challa <rohan@golang.org> Added Roman Kollár <roman.kollar.0@gmail.com> Added Ruixin Bao <ruixin.bao@ibm.com> Added Sardorbek Pulatov <sardorbek.pulatov@outlook.com> Added Scott Ragan <ragansa@fb.com> Added Sean Liao <seankhliao@gmail.com> Added Sebastian Chlopecki <sebsebmc@gmail.com> Added Sebastian Kinne <skinne@google.com> Added Sergei Lemeshkin <sergeilem@gmail.com> Added Serhat Giydiren <serhatgiydiren@gmail.com> Added Shivashis Padhi <shivashispadhi@gmail.com> Added Simarpreet Singh <simar@linux.com> Added Simon Ferquel <simon.ferquel@docker.com> Added Simon Rozman <simon@rozman.si> Added Sjoerd Siebinga <sjoerd.siebinga@gmail.com> Added Spencer Kocot <spencerkocot@gmail.com> Added Srinidhi Kaushik <shrinidhi.kaushik@gmail.com> Added Sven Taute <sven.taute@gmail.com> Added Tao Qingyun <qingyunha@gmail.com> Added Tianji Wu <the729@gmail.com> Added Tomas Dabasinskas <tomas@dabasinskas.net> Added Ville Skyttä <ville.skytta@iki.fi> Added Vitaly Zdanevich <zdanevich.vitaly@ya.ru> Added Vladimir Evgrafov <evgrafov.vladimir@gmail.com> Added Vojtech Bocek <vbocek@gmail.com> Added Wang Xuerui <git@xen0n.name> Added William Poussier <william.poussier@gmail.com> Added Xiangdong Ji <xiangdong.ji@arm.com> Added Yuichi Nishiwaki <yuichi.nishiwaki@gmail.com> Added Ziheng Liu <lzhfromustc@gmail.com> Used GitHub User DQNEO (188741) form for DQNEO <dqneoo@gmail.com> https://github.com/golang/go/commit/f07059d949 [go] Used GitHub User Matts966 (28551465) form for Matts966 <Matts966@users.noreply.github.com> https://github.com/golang/tools/commit/d89860af [tools] Used GitHub User ajz01 (4744634) form for ajz01 <ajzdenek@gmail.com> https://github.com/golang/go/commit/57ad6ef15d [go] Used GitHub User andig (184815) form for andig <cpuidle@gmx.de> https://github.com/golang/go/commit/cf630586ca [go sys] Used GitHub User jinmiaoluo (39730824) form for jinmiaoluo <jinmiaoluo@icloud.com> https://github.com/golang/go/commit/c7e73ef60a [go] Used GitHub User maltalex (10195391) form for maltalex <code@bit48.net> https://github.com/golang/sys/commit/c709ea0 [sys] Used GitHub User po3rin (29445112) form for po3rin <abctail30@gmail.com> https://github.com/golang/go/commit/bf865823ba [go] Used GitHub User pokutuna (57545) form for pokutuna <popopopopokutuna@gmail.com> https://github.com/golang/go/commit/46e0d724b3 [go] Used GitHub User ramenjuniti (32011829) form for ramenjuniti <ramenjuniti@gmail.com> https://github.com/golang/go/commit/f9dd99cae3 [go] Used GitHub User skanehira (7888591) form for skanehira <sho19921005@gmail.com> https://github.com/golang/go/commit/87805c92fd [go] Used GitHub User utkarsh-extc (53217283) form for utkarsh-extc <53217283+utkarsh-extc@users.noreply.github.com> https://github.com/golang/sys/commit/51ab0e2 [sys] Used GitHub User witchard (4994659) form for witchard <witchard@hotmail.co.uk> https://github.com/golang/go/commit/42db1da8e9 [go] Used GitHub User zikaeroh (48577114) form for zikaeroh <zikaeroh@gmail.com> https://github.com/golang/tools/commit/e84277c2 [tools] Used GitHub name "Adam Shelton" for Adam <aashelt90@gmail.com> https://github.com/golang/exp/commit/c286b88 [exp] Used GitHub name "Andrew Bonventre" for Andrew <andybons@golang.org> https://github.com/golang/go/commit/8bbfc51d9a [arch blog build crypto debug exp gddo go image lint mobile net oauth2 perf playground proposal.git review sync sys talks term text time tools tour website] Used GitHub name "David Bond" for davidsbond <davidsbond93@gmail.com> https://github.com/golang/go/commit/b421b85841 [go] Used GitHub name "Eduardo Villaseñor" for galaxy-designer <evillasrmx@gmail.com> https://github.com/golang/tools/commit/db047d72 [tools] Used GitHub name "George Hartzell" for hartzell <hartzell@alerce.com> https://github.com/golang/tools/commit/5eefd052 [tools] Used GitHub name "Ignacio Hagopian" for jsign <jsign.uy@gmail.com> https://github.com/golang/go/commit/4d4ddd862d [go] Used GitHub name "Ivan Markin" for nogoegst <nogoegst@users.noreply.github.com> https://github.com/golang/go/commit/a1addf15df [go] Used GitHub name "Jun Zhang" for zoumo <jim.zoumo@gmail.com> https://github.com/golang/tools/commit/81ca6dc7 [tools] Used GitHub name "Keiji Yoshida" for yosssi <yoshida.keiji.84@gmail.com> https://github.com/golang/lint/commit/ac6833c [lint] Used GitHub name "Koki Tomoshige" for tomocy <tomocy.dev@gmail.com> https://github.com/golang/go/commit/2f04903fec [go] Used GitHub name "Michalis Kargakis" for kargakis <mkargaki@redhat.com> https://github.com/golang/go/commit/e243d242d7 [go] Used GitHub name "Nikita Vanyasin" for nikita-vanyasin <nikita.vanyasin@gmail.com> https://github.com/golang/go/commit/c3e8a20a65 [go] Used GitHub name "Roberto Clapis" for Roberto <empijei@users.noreply.github.com> https://github.com/golang/go/commit/963776e689 [go] Used GitHub name "Robin Eklind" for mewmew <rnd0x00@gmail.com> https://github.com/golang/go/commit/b8620afb8d [arch blog go proposal.git] Used GitHub name "Sergei Lemeshkin" for sergeilem <sergeilem@gmail.com> https://github.com/golang/go/commit/a3a1bdff79 [go] Used GitHub name "Vladimir Evgrafov" for vovapi <evgrafov.vladimir@gmail.com> https://github.com/golang/go/commit/207a0b7933 [go] Updates #12042 Change-Id: I1ba3c17108491316255f914c3a71f69fc6f4341a Reviewed-on: https://go-review.googlesource.com/c/go/+/213824Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> Reviewed-by: Carlos Amedee <carlos@golang.org>
-
Michael Anthony Knyszek authored
Currently, scavenging information is printed if the gctrace debug variable is >0. Scavenging information is also printed naively, for every page scavenged, resulting in a lot of noise when the typical expectation for GC trace is one line per GC. This change adds a new GODEBUG flag called scavtrace which prints scavenge information roughly once per GC cycle and removes any scavenge information from gctrace. The exception is debug.FreeOSMemory, which may force an additional line to be printed. Fixes #32952. Change-Id: I4177dcb85fe3f9653fd74297ea93c97c389c1811 Reviewed-on: https://go-review.googlesource.com/c/go/+/212640 Run-TryBot: Michael Knyszek <mknyszek@google.com> Reviewed-by: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Bryan C. Mills authored
The information requested by these flags is not available from the vendor directory. Noticed while diagnosing #36478. Updates #33848 Change-Id: I2b181ba5c27f01fdd6277d8d0ab1003c05774ff7 Reviewed-on: https://go-review.googlesource.com/c/go/+/214081 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Bryan C. Mills authored
'go list -m' allows explicit module@version arguments, which it resolves (using Query) but does not add to the build list. Similarly, 'go list -u' resolves versions without modifying the build list. These explicit operations should be allowed even when '-mod=readonly' is set. Updates #36478 'go list' and 'go mod download' do not Change-Id: I5d2735729ad573635b9c1902d5d3a8bd960b8a76 Reviewed-on: https://go-review.googlesource.com/c/go/+/214077 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Austin Clements authored
On Windows, we implement asynchronous preemption using SuspendThread to suspend other threads in our process. However, SuspendThread is itself actually asynchronous (it enqueues a kernel "asynchronous procedure call" and returns). Unfortunately, Windows' ExitProcess API kills all threads except the calling one and then runs APCs. As a result, if SuspendThread and ExitProcess are called simultaneously, the exiting thread can be suspended and the suspending thread can be exited, leaving behind a ghost process consisting of a single thread that's suspended. We've already protected against the runtime's own calls to ExitProcess, but if Go code calls external code, there's nothing stopping that code from calling ExitProcess. For example, in #35775, our own call to racefini leads to C code calling ExitProcess and occasionally causing a deadlock. This CL fixes this by introducing synchronization between calling external code on Windows and preemption. It adds an atomic field to the M that participates in a simple CAS-based synchronization protocol to prevent suspending a thread running external code. We use this to protect cgocall (which is used for both cgo calls and system calls on Windows) and racefini. Tested by running the flag package's TestParse test compiled in race mode in a loop. Before this change, this would reliably deadlock after a few minutes. Fixes #35775. Updates #10958, #24543. Change-Id: I50d847abcdc2688b4f71eee6a75eca0f2fee892c Reviewed-on: https://go-review.googlesource.com/c/go/+/213837 Run-TryBot: Austin Clements <austin@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com> Reviewed-by: David Chase <drchase@google.com>
-
Bryan C. Mills authored
In CL 204777, I made the "finding" messages in cachingRepo only print after a “longish” delay, on the theory that they would help diagnose slow or stuck fetches. However, as I've been testing Go 1.14 beta 1, I've found that these messages are mostly just noise, and the fact that they are so nondeterministic causes both confusion and test flakes (#35539). Moreover, it currently triggers once for each candidate module, when what we're usually after is actually a specific package within the module. So let's log the package operation unconditionally instead of the module fetches nondeterministically. Fixes #35539 Updates #26152 Change-Id: I41a1c772465b2f0b357d3402bc372b6907773741 Reviewed-on: https://go-review.googlesource.com/c/go/+/213679 Run-TryBot: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Cherry Zhang authored
On Windows, it might be possible that SuspendThread suspends a thread right between when an exception happens and when the exception handler runs. (This is my guess. I don't know the implementation detail of Windows exceptions to be sure.) In this case, we may inject a call to asyncPreempt before the exception handler runs. The exception handler will inject a sigpanic call, which will make the stack trace looks like sigpanic asyncPreempt actual panicking function i.e. it appears asyncPreempt panicked. Instead, just overwrite the PC, without pushing another frame. Fixes #35773. Change-Id: Ief4e964dcb7f45670b5f93c4dcf285cc1c737514 Reviewed-on: https://go-review.googlesource.com/c/go/+/213879 Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Austin Clements <austin@google.com>
-
- 08 Jan, 2020 4 commits
-
-
Bryan C. Mills authored
We know of at least one module (github.com/stripe/stripe-go) that has a run of +incompatible versions, followed by a run of versions with go.mod files, followed by another run of +incompatible versions. We want the heuristics for showing +incompatible versions to reflect the authors' current intent, and it seems clear that the current intent of the authors of that module is for users of the unversioned import path to still be on +incompatible versions. To respect that intent, we need to keep checking for +incompatible versions even after we have seen a lower major version with an explicit go.mod file. However, we still don't want to download every single version of the module to check it. A given major version should have a consistent, canonical import path, so the path (as inferred by the presence or absence of a go.mod file) should be the same for every release across that major version. To avoid unnecessary overhead — and to allow module authors to correct accidental changes to a major version's import path — we check only the most recent release of each major version. If a release accidentally changes the import path in either direction (by deleting or adding a go.mod file), it can be corrected by issuing a single subsequent release of that major version to restore the correct path. I manually verified that, with this change, github.com/stripe/stripe-go@latest reverts to v68.7.0+incompatible as it was in Go 1.13. The other regression tests for #34165 continue to pass. Updates #34165 Change-Id: I5daff3cd2123f94c7c49519babf4eecd509f169e Reviewed-on: https://go-review.googlesource.com/c/go/+/212317Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Keith Randall authored
This avoids the security problem in #29312 where two very deep, but distinct, types are given the same name. They both make it to the linker which chooses one, and the use of the other is now type unsafe. Instead, give every very deep type its own name. This errs on the other side, in that very deep types that should be convertible to each other might now not be. But at least that's not a security hole. Update #29312. Change-Id: Iac0ebe73fdc50594fd6fbf7432eef65f9a053126 Reviewed-on: https://go-review.googlesource.com/c/go/+/213517 Run-TryBot: Keith Randall <khr@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com> Reviewed-by: Robert Griesemer <gri@golang.org>
-
Luka Zitnik authored
Fixes #35620 Change-Id: I71bc56ec7a7507d14b4f013177b4b816bb1a2094 Reviewed-on: https://go-review.googlesource.com/c/go/+/212458Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
-
Joel Sing authored
This is more readable and less error-prone than using RSP offsets. Suggested during review of CL 212765. Change-Id: I070190abeeac8eae5dbd414407602619d9d57422 Reviewed-on: https://go-review.googlesource.com/c/go/+/213577 Run-TryBot: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
-
- 07 Jan, 2020 13 commits
-
-
Michael Matloob authored
The TMPDIR environment variable isn't always available. Use $WORK/tmp instead, to fix this test on Windows. Change-Id: I521c470430c6eea4d1f2fc7db749d38178a03a13 Reviewed-on: https://go-review.googlesource.com/c/go/+/213680 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Bryan C. Mills <bcmills@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Michael Matloob authored
I accidentally used // instead of #. Change-Id: I2c9b9d40dd83994ce80fc837e8d992d3807f3e24 Reviewed-on: https://go-review.googlesource.com/c/go/+/213659 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Dmitri Shuralyov <dmitshur@golang.org>
-
Michael Matloob authored
Part of converting all tests to script framework to improve test parallelism. Updates #36320 Updates #17751 Change-Id: I30230ca3b4d8b037ea861db952b89e706ed8706d Reviewed-on: https://go-review.googlesource.com/c/go/+/213425 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Michael Matloob authored
This test already calls tg.Parallel Part of converting all tests to script framework to improve test parallelism. Updates #36320 Updates #17751 Change-Id: I4cdb2464cd3e51f1369558c238925f036ce8d828 Reviewed-on: https://go-review.googlesource.com/c/go/+/213426 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Michael Matloob authored
Part of converting all tests to script framework to improve test parallelism. Updates #36320 Updates #17751 Change-Id: Id4c2c58167d5cfc80b0d81ca9ce3db678242c06c Reviewed-on: https://go-review.googlesource.com/c/go/+/213128 Run-TryBot: Michael Matloob <matloob@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Bryan C. Mills authored
Previously, we accidentally wrote the Proxy-Authorization header for the initial CONNECT request to the shared ProxyConnectHeader map when it was non-nil. Fixes #36431 Change-Id: I5cb414f391dddf8c23d85427eb6973f14c949025 Reviewed-on: https://go-review.googlesource.com/c/go/+/213638 Run-TryBot: Bryan C. Mills <bcmills@google.com> Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Michael Matloob authored
It's hard to convert this one exactly because I don't think we can guarantee that the grep command exists to filter stdout, so I've tried to replicate the intent of the test. Part of converting all tests to script framework to improve test parallelism. Updates #36320 Updates #17751 Change-Id: Ib593799ef7634ce12efb3ff357eb34475e2ea321 Reviewed-on: https://go-review.googlesource.com/c/go/+/213130 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Michael Matloob authored
Convert these two tests to the script framework. Updates #17751 Updates #36320 Change-Id: I9cc14360fab949dab91f9e5e667be85002ef5926 Reviewed-on: https://go-review.googlesource.com/c/go/+/212811 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Jay Conrod <jayconrod@google.com>
-
Michael Matloob authored
Part of converting all tests to script framework to improve test parallelism. Updates #36320 Updates #17751 Change-Id: I6573185cf14f298c51f76265f18a75e4960ce791 Reviewed-on: https://go-review.googlesource.com/c/go/+/213220 Run-TryBot: Michael Matloob <matloob@golang.org> Reviewed-by: Bryan C. Mills <bcmills@google.com>
-
Joel Sing authored
Correct the pipe and pipe2 implementations by using the correct RSP offsets, used to store and return the file descriptor array. Fix setNonblock by using the correct immediate value for O_NONBLOCK and replace EOR (exclusive OR) with ORR. Also correct the write1 implementation, which has a uintptr value for the fd argument. Change-Id: Ibca77af44b649e8bb330ca54f9c36a7a8b0f9cea Reviewed-on: https://go-review.googlesource.com/c/go/+/212765 Run-TryBot: Cherry Zhang <cherryyz@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com>
-
Julian Tibble authored
Change-Id: I96db053184e5e72864514d5421a97774545cc2dd GitHub-Last-Rev: f1451ab626563f82f1703a559e4cb6d66665a7b6 GitHub-Pull-Request: golang/go#36425 Reviewed-on: https://go-review.googlesource.com/c/go/+/213597Reviewed-by: Ian Lance Taylor <iant@golang.org>
-
Joel Sing authored
The current code uses EOR (exclusive OR), which will result in the O_NONBLOCK flag being toggled rather than being set. Other implementations use OR, hence this is likely a bug. Change-Id: I5dafa9c572452070bd37789c8a731ad6d04a86cd Reviewed-on: https://go-review.googlesource.com/c/go/+/212766 Run-TryBot: Ian Lance Taylor <iant@golang.org> Reviewed-by: Cherry Zhang <cherryyz@google.com> Reviewed-by: Ian Lance Taylor <iant@golang.org> TryBot-Result: Gobot Gobot <gobot@golang.org>
-
Dan Scales authored
In reclaimChunk, the runtime is calling traceGCSweepDone() while holding the mheap lock. traceGCSweepDone() can call traceEvent() and traceFlush(). These functions not only can get various trace locks, but they may also do memory allocations (runtime.newobject) that may end up getting the mheap lock. So, there may be either a self-deadlock or a possible deadlock between multiple threads. It seems better to release the mheap lock before calling traceGCSweepDone(). It is fine to release the lock, since the operations to get the index of the chunk of work to do are atomic. We already release the lock to call sweep, so there is no new behavior for any of the callers of reclaimChunk. With this change, mheap is a leaf lock (no other lock is ever acquired while it is held). Testing: besides normal all.bash, also ran all.bash with --long enabled, since it does longer tests of runtime/trace. Change-Id: I4f8cb66c24bb8d424f24d6c2305b4b8387409248 Reviewed-on: https://go-review.googlesource.com/c/go/+/207846Reviewed-by: Austin Clements <austin@google.com> Reviewed-by: Michael Knyszek <mknyszek@google.com>
-