Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(52)

Issue 5687076: code review 5687076: runtime: goroutine profile, stack dumps (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 2 months ago by rsc
Modified:
12 years, 2 months ago
Reviewers:
CC:
golang-dev, r, r2
Visibility:
Public.

Description

runtime: goroutine profile, stack dumps

Patch Set 1 #

Patch Set 2 : diff -r f426a254673e https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r f426a254673e https://go.googlecode.com/hg/ #

Total comments: 1

Patch Set 4 : diff -r 884309ac4a2e https://go.googlecode.com/hg/ #

Patch Set 5 : diff -r 98061205e2d4 https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+207 lines, -59 lines) Patch
M src/pkg/runtime/debug.go View 1 4 chunks +23 lines, -10 lines 0 comments Download
M src/pkg/runtime/malloc.h View 1 2 1 chunk +1 line, -9 lines 0 comments Download
M src/pkg/runtime/mgc0.c View 1 5 chunks +23 lines, -12 lines 0 comments Download
M src/pkg/runtime/mprof.goc View 1 8 chunks +112 lines, -8 lines 0 comments Download
M src/pkg/runtime/print.c View 1 2 3 4 10 chunks +38 lines, -18 lines 0 comments Download
M src/pkg/runtime/proc.c View 1 1 chunk +6 lines, -0 lines 0 comments Download
M src/pkg/runtime/runtime.h View 1 3 chunks +4 lines, -2 lines 0 comments Download

Messages

Total messages: 6
rsc
Hello golang-dev@googlegroups.com, I'd like you to review this change to https://go.googlecode.com/hg/
12 years, 2 months ago (2012-02-22 06:59:52 UTC) #1
r
LGTM but please change the name of write http://codereview.appspot.com/5687076/diff/3001/src/pkg/runtime/print.c File src/pkg/runtime/print.c (right): http://codereview.appspot.com/5687076/diff/3001/src/pkg/runtime/print.c#newcode15 src/pkg/runtime/print.c:15: write(void ...
12 years, 2 months ago (2012-02-22 21:38:31 UTC) #2
rsc
On 2012/02/22 21:38:31, r wrote: > LGTM but please change the name of write it's ...
12 years, 2 months ago (2012-02-22 21:56:18 UTC) #3
r2
On 23/02/2012, at 8:56 AM, rsc@golang.org wrote: > On 2012/02/22 21:38:31, r wrote: >> LGTM ...
12 years, 2 months ago (2012-02-22 23:44:12 UTC) #4
rsc
changed to gwrite
12 years, 2 months ago (2012-02-23 02:34:37 UTC) #5
rsc
12 years, 2 months ago (2012-02-23 02:45:04 UTC) #6
*** Submitted as http://code.google.com/p/go/source/detail?r=4e38cad33649 ***

runtime: goroutine profile, stack dumps

R=golang-dev, r, r
CC=golang-dev
http://codereview.appspot.com/5687076
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b