Question

我是并行计算的新手，因此我正在使用pthread和互斥体。运行以下代码会导致死锁。任何人都可以解释我为什么，以及如何以正确的方式解决这个问题？我的目标是保护成员变量不被交叉写入。非常感谢提前！

#include <pthread.h>
#include <iostream>

class mutex_class
{

    public:
        mutex_class();
        void print_stars(int count);
        void print_dots(int count);

    private:
        pthread_mutex_t mutex;

};

mutex_class::mutex_class()
{
    pthread_mutex_t mutex;
    pthread_mutex_init (&mutex, NULL);
}

void mutex_class::print_stars(int count)
{
    pthread_mutex_lock(&this->mutex);
    for(int i=0; i<count; i++)
    {
        std::cout << "*" << std::flush;
    }
    pthread_mutex_unlock(&this->mutex);
}

void mutex_class::print_dots(int count)
{
    pthread_mutex_lock(&this->mutex);
    for(int i=0; i<count; i++)
    {
        std::cout << "." << std::flush;
    }
    pthread_mutex_unlock(&this->mutex);
}

void* call_dots(void* m)
{
    (*(mutex_class*)m).print_dots(20);
}

void* call_stars(void* m)
{
    (*(mutex_class*)m).print_stars(20);
}

int main()
{
    mutex_class m1 = mutex_class();
    pthread_t t1, t2;
    pthread_create(&t1, NULL, call_dots, (void*)&m1);
    pthread_create(&t2, NULL, call_stars, (void*)&m1);
    pthread_join(t1, NULL);
    pthread_join(t2, NULL);
    return(0);
}

Answer 1

问题在于：

mutex_class::mutex_class()
{
    pthread_mutex_t mutex;             // <<< remove this line
    pthread_mutex_init (&mutex, NULL);
}

您正在初始化本地互斥锁，并保留成员变量未初始化。

除非您因某些原因而陷入困境，否则请考虑使用标准C ++线程库;与使用C风格的API相比，它更具可移植性，并且不易出错。这是你用更多C ++习语的程序：

#include <thread>
#include <mutex>
#include <iostream>

class mutex_class
{
    public:
        // No need for a special constructor

        void print_stars(int count);
        void print_dots(int count);

    private:
        std::mutex mutex;
};

void mutex_class::print_stars(int count)
{
    std::lock_guard<std::mutex> lock(mutex);  // unlocks automatically
    for(int i=0; i<count; i++)
    {
        std::cout << "*" << std::flush;
    }
}

void mutex_class::print_dots(int count)
{
    std::lock_guard<std::mutex> lock(mutex);  // unlocks automatically
    for(int i=0; i<count; i++)
    {
        std::cout << "." << std::flush;
    }
}

// No need to muck around with C-style 'void*' functions

int main()
{
    mutex_class m1;
    std::thread t1([&]{m1.print_dots(20);});
    std::thread t2([&]{m1.print_stars(20);});
    t1.join();
    t2.join();
}

pthread_mutex_t作为类成员导致死锁

1 个答案: