Created on 2011-02-26.00:00:00 last changed 167 months ago
Proposed resolution:
Resolved 2011-03 Madrid meeting by paper N3278
Proposed resolution:
Resolved 2011-03 Madrid meeting by paper N3278
[ 2011-03-16: Jens updates wording ]
Modify [atomics.order] p.3, so that the normative part reads:
3 There shall be a single total order S on all memory_order_seq_cst operations, consistent with the "happens before" order and modification orders for all affected locations, such that each memory_order_seq_cst operation that loads a value observes either the last preceding modification according to this order S, A (if any), or the result of an operation X that
is notdoes not happen before A and that is not memory_order_seq_cst. [ Note: Although it is not explicitly required that S include locks, it can always be extended to an order that does include lock and unlock operations, since the ordering between those is already included in the "happens before" ordering. — end note ]
This violates the core intent of the memory model, as stated in the note in [intro.multithread] p. 21.
This was discovered by Mark Batty, and pointed out in their POPL 2011 paper, "Mathematizing C++ Concurrency", section 4, "Sequential consistency of SC atomics". The problem is quite technical, but well-explained in that paper.
This particular issue was not understood at the time the FCD comments were generated. But it is closely related to a number of FCD comments. It should have arisen from US-171, though that's not the actual history.
This issue has been under discussion for several months in a group that included a half dozen or so of the most interested committee members. The P/R represents a well-considered consensus among us:
History | |||
---|---|---|---|
Date | User | Action | Args |
2011-03-24 21:43:06 | admin | set | messages: + msg5712 |
2011-03-24 21:43:06 | admin | set | status: new -> resolved |
2011-03-15 23:26:43 | admin | set | messages: + msg5656 |
2011-02-26 21:55:01 | admin | set | messages: + msg5561 |
2011-02-26 00:00:00 | admin | create |