为什么单线程 FLTK 应用程序会出现死锁？

Question

我有一个所谓的单线程 FLTK 应用程序，带有一个弹出菜单，是用 Fluid 创建的。我有一个类，它是 Fl_Gl_Window 的子类并实现了一个 handle() 方法。 handle() 方法调用一个函数，该函数在右键单击时创建一个弹出窗口。我对其中一个菜单项进行了长时间的操作。我的应用程序出于其他目的创建了第二个线程。我使用锁来保护主线程和第二个线程之间的一些关键部分。特别是，doLongOperation() 使用锁。

我的问题是，我可以弹出菜单两次并运行 doLongOperation() 两次，然后它会自行死锁，挂起应用程序。 为什么第一个 doLongOperation() 不会停止 GUI 并阻止我再次启动 doLongOperation()？

我可以使用用于禁用有问题的菜单项的标志来避免此问题，但是我想首先了解为什么这是可能的。

这是代码，当然是缩写的。希望我已经包含了所有相关内容。

class MyClass {
  void doLongOperation();
};

class MyApplication : public MyClass {
  MyApplication();
  void run();
  void popup_menu();
};

void MyClass::doLongOperation()
{
   this->enterCriticalSection();
   // stuff
   // EDIT
   // @vladr I did leave out a relevant bit.
   // Inside this critical section, I was calling Fl::check().
   // That let the GUI handle a new popup and dispatch a new
   // doLongOperation() which is what lead to deadlock.
   // END EDIT
   this->leaveCriticalSection();
} 

MyApplication::MyApplication() : MyClass() 
{
  // ...
  { m_mainWindowPtr = new Fl_Double_Window(820, 935, "Title");
    m_mainWindowPtr->callback((Fl_Callback*)cb_m_mainWindowPtr, (void*)(this));
    { m_wireFrameViewPtr = new DerivedFrom_Fl_Gl_Window(10, 40, 800, 560);
      // ...
    }
    m_mainWindowPtr->end();
  } // Fl_Double_Window* m_mainWindowPtr

m_wireFrameViewPtr->setInteractive();

m_mainWindowPtr->position(7,54);
m_mainWindowPtr->show(1, &(argv[0]));

Fl::wait();
}

void MyApplication::run() {
  bool keepRunning = true;
  while(keepRunning) {

  m_wireFrameViewPtr->redraw();
  m_wireFrameView2Ptr->redraw();

  MyClass::Status result = this->runOneIteration();
  switch(result) {
  case DONE: keepRunning = false; break;
  case NONE: Fl::wait(0.001); break;
  case MORE: Fl::check(); break;
  default: keepRunning = false;
  }

}

void MyApplication::popup_menu() {
  Fl_Menu_Item *rclick_menu;


  int longOperationFlag = 0;
  // To avoid the deadlock I can set the flag when I'm "busy".
  //if (this->isBusy()) longOperationFlag = FL_MENU_INACTIVE;

  Fl_Menu_Item single_rclick_menu[] = {
     { "Do long operation", 0, 0, 0, longOperationFlag },
     // etc. ...
     { 0 }
  };

  // Define multiple_rclick_menu...

  if (this->m_selectedLandmarks.size() == 1) rclick_menu = single_rclick_menu;
  else rclick_menu = multiple_rclick_menu;

  const Fl_Menu_Item *m = rclick_menu->popup(Fl::event_x(), Fl::event_y(), 0, 0, 0);

  if (!m) return;


  if (strcmp(m->label(), "Do long operation") == 0) {
    this->doLongOperation();
    return;
  }

  // Etc.

}

Answer 1

确保您没有从多个线程调用Fl::wait(...)。我从您的代码中推断run( ) 在它自己的线程中执行？

第一次调用 Fl::wait()（例如从主线程）将捕获并处理第一次右键单击（按预期阻塞，而第一次调用 doLongOperation() 收益）；与此同时，第二个线程对例如 Fl::wait(timeout)/Fl::check() 的调用将继续刷新显示 - 并将拦截（和服务）第二个右侧 -单击，调用 handle() （在第二个线程中），而第一个长操作仍在缓慢进行。这会导致死锁的出现，尽管我预计当两个长操作完成时 UI 将恢复重绘（通过第二个线程）。

通过在 popup_menu() 中记录当前线程 ID 来验证上述内容。

您应该选择一个线程在循环中调用 Fl::wait(...)，并且不应阻塞该循环——生成任何非模态或非 UI 任务作为单独的线程。即当 popup_menu() 被调用时，在自己的线程中启动长操作；如果在长操作线程仍在运行时（再次）触发 popup_menu() ，请将弹出菜单项标记为禁用（类似于您的解决方法），或者简单地向长操作线程发出信号以重新启动一个新参数。

Answer 2

无论如何，您的 doLongOperation 是否会执行任何可能需要消息泵（或 APC，某些 Windows 的文件 API 在下面使用这些）才能运行的操作（假设您在 Windows 上看到此行为）？例如，如果doLongOperation尝试更新下面使用SendMessage的GUI，即使在单线程场景中也会遇到死锁。

另外，另一个线程是否已经声明了关键部分？您应该能够在挂起期间中断调试器，并希望看到谁在等待什么。