* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
@ 2012-08-04 17:31 ` rguenth at gcc dot gnu.org
2012-08-05 19:36 ` redi at gcc dot gnu.org
` (20 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-08-04 17:31 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.7.2
Summary|[4.7 Regression] |[4.7/4.8 Regression]
|__cxa_guard_acquire |__cxa_guard_acquire
|thread-safety issue |thread-safety issue
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
2012-08-04 17:31 ` [Bug libstdc++/54172] [4.7/4.8 " rguenth at gcc dot gnu.org
@ 2012-08-05 19:36 ` redi at gcc dot gnu.org
2012-08-10 15:07 ` jakub at gcc dot gnu.org
` (19 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: redi at gcc dot gnu.org @ 2012-08-05 19:36 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Jonathan Wakely <redi at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |NEW
Last reconfirmed| |2012-08-05
Ever Confirmed|0 |1
--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-08-05 19:36:24 UTC ---
The analysis looks correct, and I think the patch is actually an improvement on
the old behaviour, correctly setting 'expected' to 0 for the first CAS but also
avoiding unnecessarily looping when the second CAS fails: we don't need to loop
and do the first CAS again to decide what to do, examining the result of that
second CAS tells you what needs to be done.
I don't particularly like the variable name 'expected' because it serves two
purposes: it holds the expected value and after the CAS it holds the actual
value, the name 'expected' only suits the first usage and makes comparisons
such as (expected == guard_bit) look odd. It's not the expected value that's
being compared, it's the actual value which was *not* what was expected. I
preferred the original name "old", or maybe better yet "current".
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
2012-08-04 17:31 ` [Bug libstdc++/54172] [4.7/4.8 " rguenth at gcc dot gnu.org
2012-08-05 19:36 ` redi at gcc dot gnu.org
@ 2012-08-10 15:07 ` jakub at gcc dot gnu.org
2012-08-30 2:40 ` foom at fuhm dot net
` (18 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-08-10 15:07 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |jakub at gcc dot gnu.org
--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-08-10 15:07:33 UTC ---
The patch looks good to me, though IMHO only the first hunk is strictly
necessary (and pretty much obvious and matches what the code was doing before
r184110).
The second hunk is just an optimisation, so supposedly it should go only to
4.8?
Can you please post it to gcc-patches and libstdc++ mailing lists where review
should happen? Thanks.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (2 preceding siblings ...)
2012-08-10 15:07 ` jakub at gcc dot gnu.org
@ 2012-08-30 2:40 ` foom at fuhm dot net
2012-08-30 7:53 ` thiago at kde dot org
` (17 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: foom at fuhm dot net @ 2012-08-30 2:40 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
James Y Knight <foom at fuhm dot net> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |foom at fuhm dot net
--- Comment #3 from James Y Knight <foom at fuhm dot net> 2012-08-30 02:39:31 UTC ---
Ping; this seems like a serious regression in 4.7 with a patch.
I guess fixing it is blocked on Thiago posting the patch to gcc-patches?
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (3 preceding siblings ...)
2012-08-30 2:40 ` foom at fuhm dot net
@ 2012-08-30 7:53 ` thiago at kde dot org
2012-08-30 8:21 ` jakub at gcc dot gnu.org
` (16 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: thiago at kde dot org @ 2012-08-30 7:53 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #4 from Thiago Macieira <thiago at kde dot org> 2012-08-30 07:52:31 UTC ---
I'll post today.
I haven't yet looked up which mailing list you're even talking about.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (4 preceding siblings ...)
2012-08-30 7:53 ` thiago at kde dot org
@ 2012-08-30 8:21 ` jakub at gcc dot gnu.org
2012-08-30 9:57 ` redi at gcc dot gnu.org
` (15 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-08-30 8:21 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-08-30 08:19:56 UTC ---
Send to both gcc-patches at gcc.gnu.org and libstdc++ at gcc.gnu.org.
I think it would be better to send both hunks separately, as the first hunk is
shorter for backporting and the only necessary thing to fix, and you should
include ChangeLog entries alongside with the patches, something like:
2012-08-30 Thiago Macieira <whatever@whatever.whatever>
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Describe your change here.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (5 preceding siblings ...)
2012-08-30 8:21 ` jakub at gcc dot gnu.org
@ 2012-08-30 9:57 ` redi at gcc dot gnu.org
2012-09-01 8:27 ` thiago at kde dot org
` (14 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: redi at gcc dot gnu.org @ 2012-08-30 9:57 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #6 from Jonathan Wakely <redi at gcc dot gnu.org> 2012-08-30 09:56:44 UTC ---
This is just the usual submission process, documented at
http://gcc.gnu.org/contribute.html#patches
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (6 preceding siblings ...)
2012-08-30 9:57 ` redi at gcc dot gnu.org
@ 2012-09-01 8:27 ` thiago at kde dot org
2012-09-04 17:20 ` rth at gcc dot gnu.org
` (13 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: thiago at kde dot org @ 2012-09-01 8:27 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #7 from Thiago Macieira <thiago at kde dot org> 2012-09-01 08:26:05 UTC ---
I posted the patches on Thursday, three patches because I found one more issue,
to both lists.
Will they be picked up from there and applied to the source tree?
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (7 preceding siblings ...)
2012-09-01 8:27 ` thiago at kde dot org
@ 2012-09-04 17:20 ` rth at gcc dot gnu.org
2012-09-04 17:48 ` thiago at kde dot org
` (12 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: rth at gcc dot gnu.org @ 2012-09-04 17:20 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Richard Henderson <rth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |rth at gcc dot gnu.org
--- Comment #8 from Richard Henderson <rth at gcc dot gnu.org> 2012-09-04 17:19:41 UTC ---
(In reply to comment #7)
> I posted the patches on Thursday, three patches because I found one more issue,
> to both lists.
I havn't seen anything from you arrive on gcc-patches.
But I will say that the patch attached here looks good.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (8 preceding siblings ...)
2012-09-04 17:20 ` rth at gcc dot gnu.org
@ 2012-09-04 17:48 ` thiago at kde dot org
2012-09-05 16:23 ` paolo.carlini at oracle dot com
` (11 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: thiago at kde dot org @ 2012-09-04 17:48 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #9 from Thiago Macieira <thiago at kde dot org> 2012-09-04 17:47:08 UTC ---
(In reply to comment #8)
> (In reply to comment #7)
> > I posted the patches on Thursday, three patches because I found one more issue,
> > to both lists.
>
> I havn't seen anything from you arrive on gcc-patches.
> But I will say that the patch attached here looks good.
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg02026.html
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg02027.html
http://gcc.gnu.org/ml/gcc-patches/2012-08/msg02028.html
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (9 preceding siblings ...)
2012-09-04 17:48 ` thiago at kde dot org
@ 2012-09-05 16:23 ` paolo.carlini at oracle dot com
2012-09-06 18:54 ` rth at gcc dot gnu.org
` (10 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: paolo.carlini at oracle dot com @ 2012-09-05 16:23 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Paolo Carlini <paolo.carlini at oracle dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |paolo.carlini at oracle dot
| |com
--- Comment #10 from Paolo Carlini <paolo.carlini at oracle dot com> 2012-09-05 16:22:14 UTC ---
Richard, could you please review the 3 patches as sent to the mailing list?
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (10 preceding siblings ...)
2012-09-05 16:23 ` paolo.carlini at oracle dot com
@ 2012-09-06 18:54 ` rth at gcc dot gnu.org
2012-09-06 20:32 ` bkoz at gcc dot gnu.org
` (9 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: rth at gcc dot gnu.org @ 2012-09-06 18:54 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #11 from Richard Henderson <rth at gcc dot gnu.org> 2012-09-06 18:53:25 UTC ---
The combined patch set also looks ok.
Talking with bkoz, he would like to see more comments added,
since a reasonable testcase seems unlikely.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7/4.8 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (11 preceding siblings ...)
2012-09-06 18:54 ` rth at gcc dot gnu.org
@ 2012-09-06 20:32 ` bkoz at gcc dot gnu.org
2012-09-07 10:24 ` [Bug libstdc++/54172] [4.7 " rguenth at gcc dot gnu.org
` (8 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-09-06 20:32 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #12 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-06 20:31:13 UTC ---
Author: bkoz
Date: Thu Sep 6 20:31:08 2012
New Revision: 191042
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191042
Log:
2012-09-06 Thiago Macieira <thiago.macieira@intel.com>
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Exit the loop earlier if
we detect that another thread has had success. Don't compare_exchange
from a finished state back to a waiting state. Comment.
Modified:
trunk/libstdc++-v3/ChangeLog
trunk/libstdc++-v3/libsupc++/guard.cc
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (12 preceding siblings ...)
2012-09-06 20:32 ` bkoz at gcc dot gnu.org
@ 2012-09-07 10:24 ` rguenth at gcc dot gnu.org
2012-09-10 5:09 ` bkoz at gcc dot gnu.org
` (7 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-09-07 10:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Richard Guenther <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Keywords| |wrong-code
Priority|P3 |P2
Summary|[4.7/4.8 Regression] |[4.7 Regression]
|__cxa_guard_acquire |__cxa_guard_acquire
|thread-safety issue |thread-safety issue
--- Comment #13 from Richard Guenther <rguenth at gcc dot gnu.org> 2012-09-07 10:23:00 UTC ---
Fixed on trunk sofar.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (13 preceding siblings ...)
2012-09-07 10:24 ` [Bug libstdc++/54172] [4.7 " rguenth at gcc dot gnu.org
@ 2012-09-10 5:09 ` bkoz at gcc dot gnu.org
2012-09-10 5:10 ` bkoz at gcc dot gnu.org
` (6 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-09-10 5:09 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #14 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-10 05:08:15 UTC ---
Author: bkoz
Date: Mon Sep 10 05:08:07 2012
New Revision: 191125
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191125
Log:
2012-09-09 Thiago Macieira <thiago.macieira@intel.com>
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Exit the loop earlier if
we detect that another thread has had success. Don't compare_exchange
from a finished state back to a waiting state. Comment.
Modified:
branches/gcc-4_7-branch/libstdc++-v3/ChangeLog
branches/gcc-4_7-branch/libstdc++-v3/libsupc++/guard.cc
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (14 preceding siblings ...)
2012-09-10 5:09 ` bkoz at gcc dot gnu.org
@ 2012-09-10 5:10 ` bkoz at gcc dot gnu.org
2012-09-10 5:11 ` bkoz at gcc dot gnu.org
` (5 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-09-10 5:10 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #14 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-10 05:08:15 UTC ---
Author: bkoz
Date: Mon Sep 10 05:08:07 2012
New Revision: 191125
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191125
Log:
2012-09-09 Thiago Macieira <thiago.macieira@intel.com>
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Exit the loop earlier if
we detect that another thread has had success. Don't compare_exchange
from a finished state back to a waiting state. Comment.
Modified:
branches/gcc-4_7-branch/libstdc++-v3/ChangeLog
branches/gcc-4_7-branch/libstdc++-v3/libsupc++/guard.cc
--- Comment #15 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-10 05:08:51 UTC ---
In 4.7-branch
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (15 preceding siblings ...)
2012-09-10 5:10 ` bkoz at gcc dot gnu.org
@ 2012-09-10 5:11 ` bkoz at gcc dot gnu.org
2012-09-11 15:24 ` jakub at gcc dot gnu.org
` (4 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-09-10 5:11 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #15 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-10 05:08:51 UTC ---
In 4.7-branch
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (16 preceding siblings ...)
2012-09-10 5:11 ` bkoz at gcc dot gnu.org
@ 2012-09-11 15:24 ` jakub at gcc dot gnu.org
2012-09-11 15:25 ` jakub at gcc dot gnu.org
` (3 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-09-11 15:24 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #16 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-09-11 15:23:01 UTC ---
Author: jakub
Date: Tue Sep 11 15:22:54 2012
New Revision: 191190
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191190
Log:
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Fix up the last
argument of the first __atomic_compare_exchange_n.
Modified:
trunk/libstdc++-v3/ChangeLog
trunk/libstdc++-v3/libsupc++/guard.cc
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (17 preceding siblings ...)
2012-09-11 15:24 ` jakub at gcc dot gnu.org
@ 2012-09-11 15:25 ` jakub at gcc dot gnu.org
2012-09-11 15:26 ` jakub at gcc dot gnu.org
` (2 subsequent siblings)
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-09-11 15:25 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #16 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-09-11 15:23:01 UTC ---
Author: jakub
Date: Tue Sep 11 15:22:54 2012
New Revision: 191190
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191190
Log:
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Fix up the last
argument of the first __atomic_compare_exchange_n.
Modified:
trunk/libstdc++-v3/ChangeLog
trunk/libstdc++-v3/libsupc++/guard.cc
--- Comment #17 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-09-11 15:24:12 UTC ---
Author: jakub
Date: Tue Sep 11 15:24:06 2012
New Revision: 191191
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191191
Log:
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Fix up the last
argument of the first __atomic_compare_exchange_n.
Modified:
branches/gcc-4_7-branch/libstdc++-v3/ChangeLog
branches/gcc-4_7-branch/libstdc++-v3/libsupc++/guard.cc
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (18 preceding siblings ...)
2012-09-11 15:25 ` jakub at gcc dot gnu.org
@ 2012-09-11 15:26 ` jakub at gcc dot gnu.org
2012-09-20 10:20 ` jakub at gcc dot gnu.org
2012-09-26 18:41 ` bkoz at gcc dot gnu.org
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-09-11 15:26 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
--- Comment #17 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-09-11 15:24:12 UTC ---
Author: jakub
Date: Tue Sep 11 15:24:06 2012
New Revision: 191191
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=191191
Log:
PR libstdc++/54172
* libsupc++/guard.cc (__cxa_guard_acquire): Fix up the last
argument of the first __atomic_compare_exchange_n.
Modified:
branches/gcc-4_7-branch/libstdc++-v3/ChangeLog
branches/gcc-4_7-branch/libstdc++-v3/libsupc++/guard.cc
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (19 preceding siblings ...)
2012-09-11 15:26 ` jakub at gcc dot gnu.org
@ 2012-09-20 10:20 ` jakub at gcc dot gnu.org
2012-09-26 18:41 ` bkoz at gcc dot gnu.org
21 siblings, 0 replies; 23+ messages in thread
From: jakub at gcc dot gnu.org @ 2012-09-20 10:20 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Jakub Jelinek <jakub at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|4.7.2 |4.7.3
--- Comment #18 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-09-20 10:17:37 UTC ---
GCC 4.7.2 has been released.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [Bug libstdc++/54172] [4.7 Regression] __cxa_guard_acquire thread-safety issue
2012-08-04 11:37 [Bug libstdc++/54172] New: [4.7 Regression] __cxa_guard_acquire thread-safety issue thiago at kde dot org
` (20 preceding siblings ...)
2012-09-20 10:20 ` jakub at gcc dot gnu.org
@ 2012-09-26 18:41 ` bkoz at gcc dot gnu.org
21 siblings, 0 replies; 23+ messages in thread
From: bkoz at gcc dot gnu.org @ 2012-09-26 18:41 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54172
Benjamin Kosnik <bkoz at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
CC| |bkoz at gcc dot gnu.org
Resolution| |FIXED
Target Milestone|4.7.3 |4.7.2
--- Comment #19 from Benjamin Kosnik <bkoz at gcc dot gnu.org> 2012-09-26 18:40:33 UTC ---
Fixed in trunk and 4.7.2.
^ permalink raw reply [flat|nested] 23+ messages in thread