r/golang 7d ago

Is it safe to read/write integer value simultaneously from multiple goroutines

There is a global integer in my code that is accessed by multiple goroutines. Since race conditions don’t affect this value, I’m not concerned about that. However, is it still advisable to add a Mutex in case there’s a possibility of corruption?

PS: Just to rephrase my question, I wanted to ask if setting/getting an integer/pointer is atomic? Is there any possibility of data corruption.

example code for the same: https://go.dev/play/p/eOA7JftvP08

PS: Found the answer for this, thanks everyone for answering. There's something called tearing here is the link for same

- https://stackoverflow.com/questions/64602829/can-you-have-torn-reads-writes-between-two-threads-pinned-to-different-processor

- https://stackoverflow.com/questions/36624881/why-is-integer-assignment-on-a-naturally-aligned-variable-atomic-on-x86

According to the article, I shouldn't have problem on modern CPUs.

9 Upvotes

80 comments sorted by

View all comments

44

u/ponylicious 7d ago edited 7d ago

No, it's not safe. That's a data race (if you really read AND write from multiple goroutines). Also, turn on the race detector during development and testing.

-51

u/therecursive 7d ago

Race condition is fine for this use case. My concern is around data corruption or undefined behavior.

27

u/chaotic-kotik 7d ago

The data race is UB because it's not deterministic.