From 4bb97934558b6b6a1aafdfe3bc56c38dc3b79c87 Mon Sep 17 00:00:00 2001 From: Denys Vlasenko Date: Mon, 22 Nov 2010 14:42:49 +0100 Subject: Improve some comments. No code changes Signed-off-by: Denys Vlasenko --- src/plugins/CCpp.cpp | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) (limited to 'src/plugins/CCpp.cpp') diff --git a/src/plugins/CCpp.cpp b/src/plugins/CCpp.cpp index fad9cf62..44550ed6 100644 --- a/src/plugins/CCpp.cpp +++ b/src/plugins/CCpp.cpp @@ -31,13 +31,15 @@ using namespace std; #define CORE_PATTERN "|"CCPP_HOOK_PATH" "DEBUG_DUMPS_DIR" %p %s %u %c" #define CORE_PIPE_LIMIT_IFACE "/proc/sys/kernel/core_pipe_limit" /* core_pipe_limit specifies how many dump_helpers might run at the same time -0 - means unlimited, but the it's not guaranteed that /proc/ of crashing -process might not be available for dump_helper -4 - means that 4 dump_helpers can run at the same time, which should be enough -for ABRT, we can miss some crashes, but what are the odds that more processes -crash at the same time? This value has been recommended by nhorman -*/ -#define CORE_PIPE_LIMIT "4" + * 0 - means unlimited, but it's not guaranteed that /proc/ of crashing + * process will be available for dump_helper + * 4 - means that 4 dump_helpers can run at the same time (the rest will also + * run, but they will fail to read /proc/) + * This should be enough for ABRT, we can miss some crashes, but what are + * the odds that more processes crash at the same time? + * The value of 4 has been recommended by nhorman. + */ +#define CORE_PIPE_LIMIT "4" #define DEBUGINFO_CACHE_DIR LOCALSTATEDIR"/cache/abrt-di" -- cgit