From 813cdd3cbc35a86ebf33aaf028937e70a15ddba0 Mon Sep 17 00:00:00 2001 From: Eric Wong Date: Mon, 25 Jan 2010 18:07:52 -0800 Subject: KNOWN_ISSUES: background threads + Logger monkey patch Thanks to Michael Guterl for informing us of the issues and testing the monkey patch. --- examples/logger_mp_safe.rb | 25 +++++++++++++++++++++++++ 1 file changed, 25 insertions(+) create mode 100644 examples/logger_mp_safe.rb (limited to 'examples') diff --git a/examples/logger_mp_safe.rb b/examples/logger_mp_safe.rb new file mode 100644 index 0000000..05ad3fa --- /dev/null +++ b/examples/logger_mp_safe.rb @@ -0,0 +1,25 @@ +# Multi-Processing-safe monkey patch for Logger +# +# This monkey patch fixes the case where "preload_app true" is used and +# the application spawns a background thread upon being loaded. +# +# This removes all lock from the Logger code and solely relies on the +# underlying filesystem to handle write(2) system calls atomically when +# O_APPEND is used. This is safe in the presence of both multiple +# threads (native or green) and multiple processes when writing to +# a filesystem with POSIX O_APPEND semantics. +# +# It should be noted that the original locking on Logger could _never_ be +# considered reliable on non-POSIX filesystems with multiple processes, +# either, so nothing is lost in that case. + +require 'logger' +class Logger::LogDevice + def write(message) + @dev.syswrite(message) + end + + def close + @dev.close + end +end -- cgit v1.2.3-24-ge0c7