code review 97550043: runtime: fix freeOSMemory to free memory immediately

runtime: fix freeOSMemory to free memory immediately
Currently freeOSMemory makes only marking phase of GC, but not sweeping phase.
So recently memory is not released after freeOSMemory.
Do both marking and sweeping during freeOSMemory.
Fixes issue 8019.

[dvyukov, 2014-05-18 07:43:13]

Hello golang-codereviews@googlegroups.com (cc: khr@golang.org, rsc@golang.org),

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

[khr, 2014-05-18 08:16:44]

LGTM

https://codereview.appspot.com/97550043/diff/60001/src/pkg/runtime/mgc0.c
File src/pkg/runtime/mgc0.c (right):

https://codereview.appspot.com/97550043/diff/60001/src/pkg/runtime/mgc0.c#new...
src/pkg/runtime/mgc0.c:2298: if(!force && mstats.heap_alloc < mstats.next_gc) {
change !force to force==0.  It's no longer a boolean.

[dvyukov, 2014-05-19 08:06:14]

https://codereview.appspot.com/97550043/diff/60001/src/pkg/runtime/mgc0.c
File src/pkg/runtime/mgc0.c (right):

https://codereview.appspot.com/97550043/diff/60001/src/pkg/runtime/mgc0.c#new...
src/pkg/runtime/mgc0.c:2298: if(!force && mstats.heap_alloc < mstats.next_gc) {
On 2014/05/18 08:16:44, khr wrote:
> change !force to force==0.  It's no longer a boolean.

Done.

[dvyukov, 2014-05-19 08:06:34]

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

runtime: fix freeOSMemory to free memory immediately
Currently freeOSMemory makes only marking phase of GC, but not sweeping phase.
So recently memory is not released after freeOSMemory.
Do both marking and sweeping during freeOSMemory.
Fixes issue 8019.

LGTM=khr
R=golang-codereviews, khr
CC=golang-codereviews, rsc
https://codereview.appspot.com/97550043

[gobot, 2014-05-19 08:16:31]

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