about summary refs log tree commit homepage
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2015-02-18 09:22:56 +0000
committerEric Wong <e@80x24.org>2015-02-18 09:33:45 +0000
commit6aaa6f67e66e478fa039d01a91b89d6eaf881de5 (patch)
tree6906d4df153ae53dbbc3b5bd56ed9ea021bdd6ed
parent3efcb739dd558bbc94075b4dee9d0dc3c528d16a (diff)
downloadunicorn-6aaa6f67e66e478fa039d01a91b89d6eaf881de5.tar.gz
Oops, this should've been explained long ago but apparently not.

In response to a comment on
http://www.sitepoint.com/the-self-pipe-trick-explained/

> Does anybody know why both unicorn and foreman read 11 bytes from
> self-pipe?

Unfortunately I couldn't find a way to comment on the site on a
JavaScript-free browser nor does it seem possible without
registering.

Again, anybody can send plain-text mail to:
unicorn-public@bogomips.org

No registration, no real name policy, no terms-of-service, just
plain-text.  Feel free to use Tor, mixmaster or any anonymity
service, too.
-rw-r--r--lib/unicorn/http_server.rb4
1 files changed, 4 insertions, 0 deletions
diff --git a/lib/unicorn/http_server.rb b/lib/unicorn/http_server.rb
index 015bd94..683eb82 100644
--- a/lib/unicorn/http_server.rb
+++ b/lib/unicorn/http_server.rb
@@ -370,6 +370,10 @@ class Unicorn::HttpServer
   # wait for a signal hander to wake us up and then consume the pipe
   def master_sleep(sec)
     IO.select([ @self_pipe[0] ], nil, nil, sec) or return
+    # 11 bytes is the maximum string length which can be embedded within
+    # the Ruby itself and not require a separate malloc (on 32-bit MRI 1.9+).
+    # Most reads are only one byte here and uncommon, so it's not worth a
+    # persistent buffer, either:
     @self_pipe[0].kgio_tryread(11)
   end