From: Benjamin Kaduk <kaduk@mit.edu>
Date: Mon, 21 Dec 2015 04:11:23 +0000 (-0600)
Subject: util: Remove undocumented magic of mrafs-style logs
X-Git-Tag: upstream/1.8.0_pre1^2~117
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=ac5346b3612e4f7dc3f97a57990959794b18c3a7;p=packages%2Fo%2Fopenafs.git

util: Remove undocumented magic of mrafs-style logs

The MR-AFS-style logs would always include the thread number in
log entries with the timestamp; now that we are trying to rebrand
this feature as "timestampped logs", having this bonus feature
is unexpected.

Thread ids are still used at higher log levels, as enabled by SIGTSTP.

Change-Id: Ie8c276e47a34d729ccce685ddf27bfa9e7a8f9f1
Reviewed-on: https://gerrit.openafs.org/12136
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
---

diff --git a/src/util/serverLog.c b/src/util/serverLog.c
index 4ce846068..4461880df 100644
--- a/src/util/serverLog.c
+++ b/src/util/serverLog.c
@@ -116,7 +116,7 @@ vFSLog(const char *format, va_list args)
 		   localtime_r(&currenttime, &tm));
     info = &tbuffer[len];
 
-    if (mrafsStyleLogs || threadIdLogs) {
+    if (threadIdLogs) {
 	num = (*threadNumProgram) ();
         if (num > -1) {
 	    snprintf(info, (sizeof tbuffer) - strlen(tbuffer), "[%d] ",