diff options
| author | Torvald Riegel <triegel@redhat.com> | 2015-06-24 14:37:32 +0200 |
|---|---|---|
| committer | Torvald Riegel <triegel@redhat.com> | 2016-01-15 21:20:34 +0100 |
| commit | b02840bacdefde318d2ad2f920e50785b9b25d69 (patch) | |
| tree | dcf8ee01d1e4bdb42686d890c1d00bf3249fbcaf /nptl/pthread_barrierattr_setpshared.c | |
| parent | a3e5b4feeb54cb92657ec2bc6d9be1fcef9e8575 (diff) | |
| download | glibc-b02840bacdefde318d2ad2f920e50785b9b25d69.zip glibc-b02840bacdefde318d2ad2f920e50785b9b25d69.tar.gz glibc-b02840bacdefde318d2ad2f920e50785b9b25d69.tar.bz2 | |
New pthread_barrier algorithm to fulfill barrier destruction requirements.
The previous barrier implementation did not fulfill the POSIX requirements
for when a barrier can be destroyed. Specifically, it was possible that
threads that haven't noticed yet that their round is complete still access
the barrier's memory, and that those accesses can happen after the barrier
has been legally destroyed.
The new algorithm does not have this issue, and it avoids using a lock
internally.
Diffstat (limited to 'nptl/pthread_barrierattr_setpshared.c')
| -rw-r--r-- | nptl/pthread_barrierattr_setpshared.c | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/nptl/pthread_barrierattr_setpshared.c b/nptl/pthread_barrierattr_setpshared.c index 8cc1bf8..d41b041 100644 --- a/nptl/pthread_barrierattr_setpshared.c +++ b/nptl/pthread_barrierattr_setpshared.c @@ -24,15 +24,11 @@ int pthread_barrierattr_setpshared (pthread_barrierattr_t *attr, int pshared) { - struct pthread_barrierattr *iattr; - int err = futex_supports_pshared (pshared); if (err != 0) return err; - iattr = (struct pthread_barrierattr *) attr; - - iattr->pshared = pshared; + ((struct pthread_barrierattr *) attr)->pshared = pshared; return 0; } |