commit 6da0427a2d48a6471620f3cec668d2c18985b948
author Brian Silverman <brians> Sun May 18 18:47:48 2014 -0700
committer Brian Silverman <brians> Sun May 18 18:49:25 2014 -0700
tree 07b86d777c70a74fcd48c25420052b627fefbbd7
parent bd380fd4f7da279cf4547b05d5719b33f3f3ddfa

made sure everything's thread safe

Pretty much everything already was, but I tweaked a few things to make
sure they stay that way and added various comments about that fact. Also
cleaned up a few things along the way and removed SafeMessageBuilder
because it wasn't.

aos/linux_code/init.cc

diff --git a/aos/linux_code/init.cc b/aos/linux_code/init.cc
index 05004a2..d8864fe 100644
--- a/aos/linux_code/init.cc
+++ b/aos/linux_code/init.cc
@@ -44,7 +44,7 @@
   WriteCoreDumps();
 }
 
-int LockAllMemory() {
+void LockAllMemory() {
   InitStart();
   if (mlockall(MCL_CURRENT | MCL_FUTURE) == -1) {
     PDie("%s-init: mlockall failed", program_invocation_short_name);
@@ -55,8 +55,6 @@
   uint8_t data[4096 * 8];
   // Not 0 because linux might optimize that to a 0-filled page.
   memset(data, 1, sizeof(data));
-
-  return 0;
 }
 
 // Do the initialization code that is necessary for both realtime and
@@ -76,11 +74,14 @@
 void Init(int relative_priority) {
   if (getenv(kNoRealtimeEnvironmentVariable) == NULL) {  // if nobody set it
     LockAllMemory();
+
     // Only let rt processes run for 3 seconds straight.
     SetSoftRLimit(RLIMIT_RTTIME, 3000000, true);
+
     // Allow rt processes up to priority 40.
     SetSoftRLimit(RLIMIT_RTPRIO, 40, false);
-    // Set our process to priority 40.
+
+    // Set our process to the appropriate priority.
     struct sched_param param;
     param.sched_priority = 30 + relative_priority;
     if (sched_setscheduler(0, SCHED_FIFO, &param) != 0) {