code review 30460043: cgo/runtime: replace sigprocmask() with pthread_sigmask().

cgo/runtime: replace sigprocmask() with pthread_sigmask().

sigprocmask() use in a multithreaded environment is undefined so replace it with pthread_sigmask().
Fixes issue 6811.

[jsing, 2013-12-20 15:57:42]

Please 'hg mail' this for review.

[jsing, 2013-12-20 15:57:43]

Please 'hg mail' this for review.

[caglar10ur, 2013-12-21 19:33:18]

Hi,

I'd like you to review this change

Description: 
	cgo/runtime: replace sigprocmask() with pthread_sigmask().

	sigprocmask() use in a multithreaded environment is undefined so replace it
with pthread_sigmask().
	Fixes issue 6811.

Please review this at https://codereview.appspot.com/30460043


	Reviewer: jsing@google.com
	CC: golang-codereviews@googlegroups.com
	Files:
		src/pkg/runtime/cgo/gcc_dragonfly_386.c
		src/pkg/runtime/cgo/gcc_dragonfly_amd64.c
		src/pkg/runtime/cgo/gcc_freebsd_386.c
		src/pkg/runtime/cgo/gcc_freebsd_amd64.c
		src/pkg/runtime/cgo/gcc_linux_386.c
		src/pkg/runtime/cgo/gcc_linux_amd64.c
		src/pkg/runtime/cgo/gcc_netbsd_386.c
		src/pkg/runtime/cgo/gcc_netbsd_amd64.c
		src/pkg/runtime/cgo/gcc_netbsd_arm.c
		src/pkg/runtime/cgo/gcc_openbsd_386.c
		src/pkg/runtime/cgo/gcc_openbsd_amd64.c

[caglar10ur, 2013-12-21 19:40:21]

Hi,

[Looks like Rietveld web interface uses another email address to CC code
reviews. I'm re-sending this, hopefully this time to the golang-dev mailing
list]

I'd like you to review this change;

[iant, 2013-12-22 00:47:34]

LGTM

In change description, s/()//.

Have you signed the CLA?  (I haven't checked.)

Thanks.

[caglar10ur, 2013-12-22 01:10:00]

Hi Ian,

On 2013/12/22 00:47:34, iant wrote:
> LGTM
> 
> In change description, s/()//.

I'm assuming it's OK to use "Edit Issue" link on Rietveld and modify the
description there?

> Have you signed the CLA?  (I haven't checked.)

I believe I signed it online before sending this patch but please let me know if
I need to do anything else.

> Thanks.

Thanks,

[iant, 2013-12-22 16:55:31]

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

cgo/runtime: replace sigprocmask with pthread_sigmask.

sigprocmask use in a multithreaded environment is undefined so replace it with
pthread_sigmask.

Fixes issue 6811.

R=jsing, iant
CC=golang-codereviews, golang-dev
https://codereview.appspot.com/30460043

Committer: Ian Lance Taylor <iant@golang.org>