Don't use sync/atomic

1 files changed, 12 insertions, 1 deletions

diff --git a/STYLE.md b/STYLE.md
index 3a6ae2bad..d72775073 100644
--- a/STYLE.md
+++ b/STYLE.md
@@ -107,4 +107,15 @@ instead of:
 func (server) GetBlob(_ *pb.GetBlobRequest, _ pb.BlobService_GetBlobServer) error {
     return helper.Unimplemented
 }
-```
\ No newline at end of file
+```
+
+## Concurrency
+
+The [documentation of "sync/atomic"](https://golang.org/pkg/sync/atomic/) says:
+
+> These functions require great care to be used correctly. Except for
+special, low-level applications, synchronization is better done with
+channels or the facilities of the sync package.
+
+Gitaly is not a low-level application so we should avoid "sync/atomic".
+We use channels and [package "sync"](https://golang.org/pkg/sync/).