Fix failure order, should be relaxed not release.

dougbinks · dougbinks · commit 49d2de8a9290 · 2020-01-21T14:42:35.000Z
diff --git a/src/LockLessMultiReadPipe.h b/src/LockLessMultiReadPipe.h
@@ -125,7 +125,7 @@ namespace enki
             // Multiple potential readers mean we should check if the data is valid,
             // using an atomic compare exchange
             uint32_t previous = FLAG_CAN_READ;
-            bool bSuccess = m_Flags[  actualReadIndex ].compare_exchange_strong( previous, FLAG_INVALID, std::memory_order_acq_rel, std::memory_order_release );
+            bool bSuccess = m_Flags[  actualReadIndex ].compare_exchange_strong( previous, FLAG_INVALID, std::memory_order_acq_rel, std::memory_order_relaxed );
             if( bSuccess )
             {
                 break;

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ namespace enki`
`125`	`125`	`// Multiple potential readers mean we should check if the data is valid,`
`126`	`126`	`// using an atomic compare exchange`
`127`	`127`	`uint32_t previous = FLAG_CAN_READ;`
`128`		`- bool bSuccess = m_Flags[ actualReadIndex ].compare_exchange_strong( previous, FLAG_INVALID, std::memory_order_acq_rel, std::memory_order_release );`
	`128`	`+ bool bSuccess = m_Flags[ actualReadIndex ].compare_exchange_strong( previous, FLAG_INVALID, std::memory_order_acq_rel, std::memory_order_relaxed );`
`129`	`129`	`if( bSuccess )`
`130`	`130`	`{`
`131`	`131`	`break;`