code review 122130043: runtime: shrink stacks in parallel

runtime: shrink stacks in parallel
Shrinkstack does not touch normal heap anymore,
so we can shink stacks concurrently with marking.

[dvyukov, 2014-08-05 21:04:40]

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/

[rsc, 2014-08-05 21:15:16]

R=khr

[khr1, 2014-08-06 20:31:17]

LGTM.



On Tue, Aug 5, 2014 at 2:15 PM, Russ Cox <rsc@golang.org> wrote:

> R=khr
>
>

[dvyukov, 2014-08-07 08:55:34]

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

runtime: shrink stacks in parallel
Shrinkstack does not touch normal heap anymore,
so we can shink stacks concurrently with marking.

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

[rlh, 2014-08-07 13:46:48]

Stacks get shrink, the old one is freed, and the new one scanned. This
would seem to foreclose the option of concurrent scans of the same stack
which I'm not objecting too, I'm just noting it.
LGTM


On Thu, Aug 7, 2014 at 4:55 AM, <dvyukov@google.com> wrote:

> *** Submitted as
> https://code.google.com/p/go/source/detail?r=739dc7f55e3f ***
>
> runtime: shrink stacks in parallel
> Shrinkstack does not touch normal heap anymore,
> so we can shink stacks concurrently with marking.
>
> LGTM=khr
> R=golang-codereviews, khr
> CC=golang-codereviews, khr, rlh, rsc
> https://codereview.appspot.com/122130043
>
>
> https://codereview.appspot.com/122130043/
>