code review 11694043: [release-branch.go1.1] runtime: properly set G status a...

[release-branch.go1.1] runtime: properly set G status after syscall

««« CL 9307045 / fab6ba2a2d10
runtime: properly set G status after syscall

R=golang-dev, r, dave
CC=golang-dev
https://codereview.appspot.com/9307045
»»»

[adg, 2013-07-22 22:02:55]

Hello golang-dev@googlegroups.com,

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

[bradfitz, 2013-07-22 22:10:13]

LGTM


On Mon, Jul 22, 2013 at 3:02 PM, <adg@golang.org> wrote:

> Reviewers: golang-dev1,
>
> Message:
> Hello golang-dev@googlegroups.com,
>
> I'd like you to review this change to
> https://code.google.com/p/go
>
>
> Description:
> [release-branch.go1.1] runtime: properly set G status after syscall
>
> ««« CL 9307045 / fab6ba2a2d10
> runtime: properly set G status after syscall
>
> R=golang-dev, r, dave
> CC=golang-dev
>
https://codereview.appspot.**com/9307045<https://codereview.appspot.com/9307045>
> »»»
>
>
> Please review this at
https://codereview.appspot.**com/11694043/<https://codereview.appspot.com/116...
>
> Affected files:
>   M src/pkg/runtime/proc.c
>
>
> Index: src/pkg/runtime/proc.c
> ==============================**==============================**=======
> --- a/src/pkg/runtime/proc.c
> +++ b/src/pkg/runtime/proc.c
> @@ -1371,6 +1371,8 @@
>                 runtime·unlock(&runtime·sched)**;
>                 if(p) {
>                         acquirep(p);
> +                       m->p->tick++;
> +                       g->status = Grunning;
>                         g->gcstack = (uintptr)nil;
>                         g->gcsp = (uintptr)nil;
>                         return;
>
>
> --
>
> ---You received this message because you are subscribed to the Google
> Groups "golang-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to
golang-dev+unsubscribe@**googlegroups.com<golang-dev%2Bunsubscribe@googlegrou...
> .
> For more options, visit
https://groups.google.com/**groups/opt_out<https://groups.google.com/groups/o...
> .
>
>
>

[adg, 2013-07-22 23:31:38]

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

[release-branch.go1.1] runtime: properly set G status after syscall

««« CL 9307045 / fab6ba2a2d10
runtime: properly set G status after syscall

R=golang-dev, r, dave
CC=golang-dev
https://codereview.appspot.com/9307045
»»»

R=golang-dev, bradfitz
CC=golang-dev
https://codereview.appspot.com/11694043