Update config generation, join fix, sampler (#129)

- update timemory submodule
  - update ring_buffer to remove all dynamic allocation
  - update sampler + sampler allocator to use ring-buffers
  - fix papi_array and papi_vector labels
- fix common::join with char delimiter
- fix config generation without --advanced

[ROCm/rocprofiler-systems commit: ac5ce00561]

projects/rocprofiler-systems/source/lib/common/join.hpp

@@ -79,16 +79,22 @@ template <typename DelimT, typename... Args>
 auto
 join(DelimT&& _delim, Args&&... _args)
 {
+    using delim_type = std::remove_cv_t<std::remove_reference_t<DelimT>>;
+
     std::stringstream _ss{};
     _ss << std::boolalpha;
-    OMNITRACE_FOLD_EXPRESSION(_ss << _delim << _args);
-    auto _ret = _ss.str();
-    if constexpr(std::is_same<DelimT, char>::value)
+
+    if constexpr(std::is_same<delim_type, char>::value)
     {
+        const char _delim_c[2] = { _delim, '\0' };
+        OMNITRACE_FOLD_EXPRESSION(_ss << _delim_c << _args);
+        auto _ret = _ss.str();
         return (_ret.length() > 1) ? _ret.substr(1) : std::string{};
     }
     else
     {
+        OMNITRACE_FOLD_EXPRESSION(_ss << _delim << _args);
+        auto   _ret = _ss.str();
         auto&& _len = std::string{ _delim }.length();
         return (_ret.length() > _len) ? _ret.substr(_len) : std::string{};
     }
@@ -101,16 +107,22 @@ template <typename DelimT, typename... Args>
 auto
 join(QuoteStrings&&, DelimT&& _delim, Args&&... _args)
 {
+    using delim_type = std::remove_cv_t<std::remove_reference_t<DelimT>>;
+
     std::stringstream _ss{};
     _ss << std::boolalpha;
-    OMNITRACE_FOLD_EXPRESSION(_ss << _delim << as_string(_args));
-    auto _ret = _ss.str();
-    if constexpr(std::is_same<DelimT, char>::value)
+
+    if constexpr(std::is_same<delim_type, char>::value)
     {
+        const char _delim_c[2] = { _delim, '\0' };
+        OMNITRACE_FOLD_EXPRESSION(_ss << _delim_c << as_string(_args));
+        auto _ret = _ss.str();
         return (_ret.length() > 1) ? _ret.substr(1) : std::string{};
     }
     else
     {
+        OMNITRACE_FOLD_EXPRESSION(_ss << _delim << as_string(_args));
+        auto   _ret = _ss.str();
         auto&& _len = std::string{ _delim }.length();
         return (_ret.length() > _len) ? _ret.substr(_len) : std::string{};
     }

projects/rocprofiler-systems/source/lib/omnitrace/library/components/backtrace.cpp

@@ -425,7 +425,6 @@ backtrace::configure(bool _setup, int64_t _tid)
         }
 
         _sampler->stop();
-        _sampler->swap_data();
         if constexpr(tim::trait::is_available<hw_counters>::value)
         {
             if(_tid == threading::get_id())
@@ -653,27 +652,29 @@ backtrace::post_process(int64_t _tid)
                                  "end_ns", _end_ns);
     };
 
-    auto _raw_data = _sampler->get_allocator().get_data();
+    _sampler->stop();
+    auto _raw_data = _sampler->get_data();
+    OMNITRACE_CI_THROW(
+        _sampler->get_sample_count() != _raw_data.size(),
+        "Error! sampler recorded %zu samples but %zu samples were returned\n",
+        _sampler->get_sample_count(), _raw_data.size());
     // single sample that is useless (backtrace to unblocking signals)
     if(_raw_data.size() == 1 && _raw_data.front().size() <= 1) _raw_data.clear();
 
     std::vector<sampling::bundle_t*> _data{};
-    for(auto& ditr : _raw_data)
+    for(auto& itr : _raw_data)
     {
-        _data.reserve(_data.size() + ditr.size());
-        for(auto& ritr : ditr)
+        _data.reserve(_data.size() + itr.size());
+        auto* _bt = itr.get<backtrace>();
+        if(!_bt)
         {
-            auto* _bt = ritr.get<backtrace>();
-            if(!_bt)
-            {
-                OMNITRACE_PRINT(
-                    "Warning! Nullptr to backtrace instance for thread %lu...\n", _tid);
-                continue;
-            }
-            if(_bt->empty()) continue;
-            if(!pthread_create_gotcha::is_valid_execution_time(_tid, _bt->m_ts)) continue;
-            _data.emplace_back(&ritr);
+            OMNITRACE_PRINT("Warning! Nullptr to backtrace instance for thread %lu...\n",
+                            _tid);
+            continue;
         }
+        if(_bt->empty()) continue;
+        if(!pthread_create_gotcha::is_valid_execution_time(_tid, _bt->m_ts)) continue;
+        _data.emplace_back(&itr);
     }
 
     if(_data.empty()) return;