Question

我可以安全地执行以下代码吗？是否有可能在此处出现死锁或某些意外行为，尤其是在SIGINT到达时？

#include <atomic>
#include <condition_variable>
#include <csignal>

std::mutex m;
std::condition_variable cv;
std::atomic<bool> flag(false);

void f1(){
             std::signal(SIGTERM, [](int signal){flag.store(true, 
             std::memory_order_release); cv.notify_one();});//register for signal
             std::unique_lock<std::mutex> mtx(m);
             cv.wait(mtx, []{return flag.load(std::memory_order_consume);});//wait for signal or f2() notify
         }

void f2(){
             flag.store(true, std::memory_order_release);
             cv.notify_one();
         }

int main(){
    std::thread th1(f1);
    std::thread th2(f2);
    th1.join();
    th2.join();
    return 0;
}

Answer 1

pthread函数以及使用这些函数的std::condition_variable和std::mutex 不是异步信号安全的。请参见man signal-safety(7)中的异步信号安全功能列表。

有点题外话：如果在更新flag时未锁定互斥锁，则会导致丢失通知。想象一下场景：

Thread 1        | Thread 2
                | mutex.lock()
                | flag == false
flag = true     | 
cv.notify_one() | <--- notification is lost
                | cv.wait()

这是一个非常常见的编程错误。

在更新flag时锁定互斥锁可以解决此问题。

如果要在接收到信号时通知条件变量，请创建一个专用于信号处理的线程。示例：

#include <condition_variable>
#include <iostream>
#include <thread>
#include <signal.h>

std::mutex m;
std::condition_variable cv;
bool flag = false;

void f1(){
    std::unique_lock<std::mutex> lock(m);
    while(!flag)
        cv.wait(lock);
}

void signal_thread() {
    sigset_t sigset;
    sigfillset(&sigset);
    int signo = ::sigwaitinfo(&sigset, nullptr);
    if(-1 == signo)
        std::abort();
    std::cout << "Received signal " << signo << '\n';

    m.lock();
    flag = true;
    m.unlock();
    cv.notify_one();
}

int main(){
    sigset_t sigset;
    sigfillset(&sigset);
    ::pthread_sigmask(SIG_BLOCK, &sigset, nullptr);

    std::thread th1(f1);
    std::thread th2(signal_thread);

    th1.join();
    th2.join();
}

请注意，必须在所有线程中阻止信号，以便只有信号处理程序线程才能接收到这些信号。

std :: condition_variable :: notify_one可重入吗？

1 个答案: