Added differentiated timer for waiting to data and waiting for free memory.

Only one applies at a time in read or write context, but the naming will make the code more undrestandable.

castor/tape/tapeserver/daemon/SessionStats.hpp

@@ -46,8 +46,10 @@ namespace tapeserver {
     double unloadTime;
     /** Unmount time, in seconds. */
     double unmountTime;
-    /** Cumulated time spent waiting for data blocks/free memory. */
+    /** Cumulated time spent waiting for data blocks. */
     double waitDataTime;
+    /** Cumulated time spent waiting for free memory. */
+    double waitFreeMemoryTime;
     /** Cumulated time spent by the tape thread waiting for a task. */
     double waitInstructionsTime;
     /** Cumulated time spent reporting */

castor/tape/tapeserver/daemon/TapeReadSingleThread.hpp

@@ -236,7 +236,7 @@ private:
             .add("waitInstructionsTime", m_stats.waitInstructionsTime)
             .add("checksumingTime", m_stats.checksumingTime)
             .add("transferTime", m_stats.transferTime)
-            .add("waitDataTime", m_stats.waitDataTime)
+            .add("waitFreeMemoryTime", m_stats.waitFreeMemoryTime)
             .add("waitReportingTime", m_stats.waitReportingTime)
             .add("flushTime", m_stats.flushTime)
             .add("unloadTime", m_stats.unloadTime)

castor/tape/tapeserver/daemon/TapeReadTask.hpp

@@ -96,7 +96,7 @@ public:
       while (stillReading) {
         // Get a memory block and add information to its metadata
         mb=m_mm.getFreeBlock();
-        localStats.waitDataTime += timer.secs(utils::Timer::resetCounter);
+        localStats.waitFreeMemoryTime += timer.secs(utils::Timer::resetCounter);
         
         mb->m_fSeq = m_fileToRecall->fseq();
         mb->m_fileBlock = fileBlock++;