简单的读写锁

Question

我发现互联网上的许多读写自旋锁实现都不必要地复杂。我用c++写了一个简单的读写锁。

如果我遗漏了什么，谁能告诉我？

int r = 0;
int w = 0;

read_lock(void) 
{
     atomic_inc(r); //increment value atomically
     while( w != 0);    
}

read_unlock(void)
{
   atomic_dec(r); // Decrement value atomically
}
write_lock(void)
{
 while( (r != 0) && 
            ( w != 0))
  atomic_inc(w); //increment value atomically
}

write_unlock(void)
{
    atomic_dec(w); //Decrement value atomically
}

用法如下。

read_lock()
// Critical Section
read_unlock();

write_lock()
// Critical Section
write_unlock();

编辑：

感谢您的回答。 我现在将答案更改为原子等价物

Answer 1

如果线程同时访问r 和w，它们就会发生数据争用。如果 C++ 程序存在数据竞争，则程序的行为是未定义的。

C++ 标准不保证int 是原子的。即使我们假设访问int 的系统是原子的，operator++ 即使在这样的系统上也可能不是原子操作。因此，同步增量可能会“消失”。

此外，在write_lock 中的循环之后，另一个线程也可以在w 递增之前结束它们的循环，从而允许多个同时写入者——我认为这个锁应该防止这种情况。

---

最后，这似乎是一种实现自旋锁的尝试。自旋锁有优点也有缺点。它们的缺点是它们在阻塞时会消耗线程的所有 CPU 周期。这是对资源的高度低效使用，不利于电池使用时间，也不利于其他可能使用这些周期的进程。但如果等待时间短，它可能是最佳的。