stable-25-3: fix thread sanitizer error in TActorSystem::IsStopped (#26166)

Author: ydbot

ydb/library/actors/core/actorsystem.cpp

@@ -372,8 +372,7 @@ namespace NActors {
 
     void TActorSystem::Start() {
         ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Start");
-        Y_ABORT_UNLESS(StartExecuted == false);
-        StartExecuted = true;
+        Y_ABORT_UNLESS(!StartExecuted.exchange(true));
 
         ScheduleQueue.Reset(new NSchedulerQueue::TQueueType());
         TVector<NSchedulerQueue::TReader*> scheduleReaders;
@@ -418,13 +417,11 @@ namespace NActors {
 
     void TActorSystem::Stop() {
         ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Stop");
-        if (StopExecuted || !StartExecuted) {
+        if (!StartExecuted.load() || StopExecuted.exchange(true)) {
             ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Stop: already stopped");
             return;
         }
 
-        StopExecuted = true;
-
         for (auto&& fn : std::exchange(DeferredPreStop, {})) {
             fn();
         }
@@ -439,11 +436,10 @@ namespace NActors {
     void TActorSystem::Cleanup() {
         ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Cleanup");
         Stop();
-        if (CleanupExecuted || !StartExecuted) {
+        if (!StartExecuted.load() || CleanupExecuted.exchange(true)) {
             ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Cleanup: already cleaned up");
             return;
         }
-        CleanupExecuted = true;
         CpuManager->Cleanup();
         Scheduler.Destroy();
         ACTORLIB_DEBUG(EDebugLevel::ActorSystem, "TActorSystem::Cleanup: cleaned up");

ydb/library/actors/core/actorsystem.h

@@ -162,9 +162,9 @@ namespace NActors {
         TMutex ProxyCreationLock;
         mutable std::vector<TActorId> DynamicProxies;
 
-        bool StartExecuted = false;
-        bool StopExecuted = false;
-        bool CleanupExecuted = false;
+        std::atomic_bool StartExecuted = false;
+        std::atomic_bool StopExecuted = false;
+        std::atomic_bool CleanupExecuted = false;
 
         std::deque<std::function<void()>> DeferredPreStop;
     public: