]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
CODING_STYLE: split out section about deadlocks
authorLennart Poettering <lennart@poettering.net>
Fri, 12 Apr 2019 14:50:24 +0000 (16:50 +0200)
committerLennart Poettering <lennart@poettering.net>
Fri, 12 Apr 2019 14:50:24 +0000 (16:50 +0200)
docs/CODING_STYLE.md

index 6a743fb493306c221d62422ad6f61ecc06f70d1d..017a26c69681f227f9a7cf0db7a341f6b4ed1afd 100644 (file)
@@ -73,14 +73,6 @@ title: Coding Style
 
 - To minimize strict aliasing violations, we prefer unions over casting.
 
-- Do not issue NSS requests (that includes user name and host name
-  lookups) from PID 1 as this might trigger deadlocks when those
-  lookups involve synchronously talking to services that we would need
-  to start up.
-
-- Do not synchronously talk to any other service from PID 1, due to
-  risk of deadlocks.
-
 - Be exceptionally careful when formatting and parsing floating point
   numbers. Their syntax is locale dependent (i.e. `5.000` in en_US is
   generally understood as 5, while in de_DE as 5000.).
@@ -419,6 +411,15 @@ title: Coding Style
   headers (i.e those in `src/systemd/sd-*.h`) use integers after all, as `bool`
   is C99 and in our public APIs we try to stick to C89 (with a few extension).
 
+## Deadlocks
+
+- Do not issue NSS requests (that includes user name and host name lookups)
+  from PID 1 as this might trigger deadlocks when those lookups involve
+  synchronously talking to services that we would need to start up.
+
+- Do not synchronously talk to any other service from PID 1, due to risk of
+  deadlocks.
+
 ## File Descriptors
 
 - When you allocate a file descriptor, it should be made `O_CLOEXEC` right from