Please consider the following formatting changes to #12745

Framework/CHANGELOG.md

@@ -1,3 +1,12 @@
+# 2024-02-22: Drop Tracy support
+
+Tracy support never took off, so I am dropping it. This was mostly because people do not know about it and having a per process profile GUI was way unpractical. Moreover, needing an extra compile time flag meant one most likely did not have the support compiled in when needed.
+
+I have therefore decided to replace it with signposts, which hopefully will see better adoption thanks
+to the integration with Instruments on mac and the easy way they can be enabled dynamically.
+
+We could then reintroduce Tracy support as a hook on top of signposts, if really needed.
+
 # 2024-02-16: Improved Signposts.
 
 In particular:

Framework/Core/include/Framework/DataRelayer.h

@@ -194,7 +194,7 @@ class DataRelayer
   std::vector<PruneOp> mPruneOps;
   size_t mMaxLanes;
 
-  TracyLockableN(std::recursive_mutex, mMutex, "data relayer mutex");
+  O2_LOCKABLE_NAMED(std::recursive_mutex, mMutex, "data relayer mutex");
 };
 
 } // namespace o2::framework

Framework/Core/include/Framework/DataSender.h

@@ -61,7 +61,7 @@ class DataSender
   std::vector<bool> mPresent;
   std::vector<bool> mPresentDefaults;
 
-  TracyLockableN(std::recursive_mutex, mMutex, "data relayer mutex");
+  O2_LOCKABLE_NAMED(std::recursive_mutex, mMutex, "data relayer mutex");
 };
 
 } // namespace o2::framework

Framework/Core/src/CommonMessageBackendsHelpers.h

@@ -38,7 +38,6 @@ struct CommonMessageBackendsHelpers {
   static ServiceProcessingCallback sendCallback()
   {
     return [](ProcessingContext& ctx, void* service) {
-      ZoneScopedN("send message callback");
       T* context = reinterpret_cast<T*>(service);
       DataProcessor::doSend(ctx.services().get<DataSender>(), *context, ctx.services());
     };

Framework/Core/src/CommonServices.cxx

@@ -741,7 +741,6 @@ auto sendRelayerMetrics(ServiceRegistryRef registry, DataProcessingStats& stats)
     stats.updateStats({static_cast<unsigned short>(static_cast<int>(ProcessingStatsId::AVAILABLE_MANAGED_SHM_BASE) + (runningWorkflow.shmSegmentId % 512)), DataProcessingStats::Op::SetIfPositive, freeMemory});
   }
 
-  ZoneScopedN("send metrics");
   auto device = registry.get<RawDeviceService>().device();
 
   int64_t totalBytesIn = 0;

Framework/Core/src/DataAllocator.cxx

@@ -35,6 +35,7 @@
 #include <utility>
 
 O2_DECLARE_DYNAMIC_LOG(stream_context);
+O2_DECLARE_DYNAMIC_LOG(parts);
 
 namespace o2::framework
 {
@@ -129,6 +130,8 @@ void DataAllocator::addPartToContext(RouteIndex routeIndex, fair::mq::MessagePtr
                                      o2::header::SerializationMethod serializationMethod)
 {
   auto headerMessage = headerMessageFromOutput(spec, routeIndex, serializationMethod, 0);
+  O2_SIGNPOST_ID_FROM_POINTER(pid, parts, headerMessage->GetData());
+  O2_SIGNPOST_START(parts, pid, "parts", "addPartToContext %p", headerMessage->GetData());
 
   // FIXME: this is kind of ugly, we know that we can change the content of the
   // header message because we have just created it, but the API declares it const
@@ -150,6 +153,8 @@ void DataAllocator::adopt(const Output& spec, std::string* ptr)
   // the correct payload size is set later when sending the
   // StringContext, see DataProcessor::doSend
   auto header = headerMessageFromOutput(spec, routeIndex, o2::header::gSerializationMethodNone, 0);
+  O2_SIGNPOST_ID_FROM_POINTER(pid, parts, header->GetData());
+  O2_SIGNPOST_START(parts, pid, "parts", "addPartToContext %p", header->GetData());
   mRegistry.get<StringContext>().addString(std::move(header), std::move(payload), routeIndex);
   assert(payload.get() == nullptr);
 }
@@ -206,6 +211,8 @@ void DataAllocator::adopt(const Output& spec, LifetimeHolder<TableBuilder>& tb)
   auto& timingInfo = mRegistry.get<TimingInfo>();
   RouteIndex routeIndex = matchDataHeader(spec, timingInfo.timeslice);
   auto header = headerMessageFromOutput(spec, routeIndex, o2::header::gSerializationMethodArrow, 0);
+  O2_SIGNPOST_ID_FROM_POINTER(pid, parts, header->GetData());
+  O2_SIGNPOST_START(parts, pid, "parts", "adopt %p", header->GetData());
   auto& context = mRegistry.get<ArrowContext>();
 
   auto creator = [transport = context.proxy().getOutputTransport(routeIndex)](size_t s) -> std::unique_ptr<fair::mq::Message> {