[Lldb-commits] [PATCH] D157654: [lldb] Fix data race in PipePosix

[Jonas Devlieghere via Phabricator via lldb-commits]

JDevlieghere added a comment.

In D157654#4581578 <https://reviews.llvm.org/D157654#4581578>, @augusto2112 wrote:

> @JDevlieghere I updated this to two shared mutexes because I'm assuming you can have more than one concurrent read and more than one concurrent write. If this is wrong I can go back to two regular mutexes.

According to POSIX [1] that would be undefined:

> The behavior of multiple concurrent reads on the same pipe, FIFO, or terminal device is unspecified.

A regular mutex should be fine.

[1] https://pubs.opengroup.org/onlinepubs/9699919799/functions/read.html


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157654/new/

https://reviews.llvm.org/D157654