code review 131190043: runtime: fix plan9 HeapSys accounting

runtime: fix plan9 HeapSys accounting

[0intro, 2014-08-25 13:00:48]

https://codereview.appspot.com/131190043/diff/20001/src/pkg/runtime/mem_plan9.c
File src/pkg/runtime/mem_plan9.c (right):

https://codereview.appspot.com/131190043/diff/20001/src/pkg/runtime/mem_plan9...
src/pkg/runtime/mem_plan9.c:20: static void(
static void*

[0intro, 2014-08-25 13:28:35]

LGTM with fixes.

https://codereview.appspot.com/131190043/diff/20001/src/pkg/runtime/mem_plan9.c
File src/pkg/runtime/mem_plan9.c (right):

https://codereview.appspot.com/131190043/diff/20001/src/pkg/runtime/mem_plan9...
src/pkg/runtime/mem_plan9.c:78: USED(v, reserved, stat);
USED(v, reserved);

https://codereview.appspot.com/131190043/diff/20001/src/pkg/runtime/mem_plan9...
src/pkg/runtime/mem_plan9.c:79: runtime·xadd64(&mstats.heap_sys, nbytes);
s/&mstats.heap_sys/stat/

[dvyukov, 2014-08-25 13:32:14]

Hello 0intro@gmail.com (cc: golang-codereviews@googlegroups.com),

I'd like you to review this change to
https://dvyukov%40google.com@code.google.com/p/go/

[dvyukov, 2014-08-25 13:32:59]

Please take another look and LGTM again after I mailed it.

[0intro, 2014-08-25 13:56:57]

LGTM

Thanks!

[dvyukov, 2014-08-25 14:02:34]

*** Submitted as https://code.google.com/p/go/source/detail?r=a25aa660e247 ***

runtime: fix plan9 HeapSys accounting

LGTM=0intro
R=0intro
CC=golang-codereviews
https://codereview.appspot.com/131190043

[gobot, 2014-08-25 14:15:48]

This CL appears to have broken the openbsd-386-rootbsd builder.
See http://build.golang.org/log/2a485cfebe206c733dd731cbb6d6ad756169c7d9