Fix/fifo race (#1591) (#1595)

* fix race condition in Fifo::FreeAddress

(cherry picked from commit dd5b2360d6)

slsReceiverSoftware/src/Fifo.cpp

@@ -81,7 +81,10 @@ void Fifo::DestroyFifos() {
     fifoStream = nullptr;
 }
 
-void Fifo::FreeAddress(char *&address) { fifoFree->push(address); }
+void Fifo::FreeAddress(char *&address) {
+    std::lock_guard<std::mutex> lock(fifoFreeMutex);
+    fifoFree->push(address);
+}
 
 void Fifo::GetNewAddress(char *&address) {
     int temp = fifoFree->getDataValue();

slsReceiverSoftware/src/Fifo.h

@@ -16,6 +16,7 @@
 #include "sls/sls_detector_defs.h"
 
 #include <atomic>
+#include <mutex>
 
 namespace sls {
 
@@ -52,6 +53,9 @@ class Fifo : private virtual slsDetectorDefs {
     int fifoDepth;
     std::atomic<int> status_fifoBound;
     std::atomic<int> status_fifoFree;
+    /** fifoFree has several producers, CircularFifo allows only one.
+     * Aligned to avoid false sharing with the members above */
+    alignas(128) std::mutex fifoFreeMutex;
 };
 
 } // namespace sls