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 4ce846068bf15bbcc89753d455247322d4f2a93c..4461880df6f88a0cb4242f1fbd1a0aca5b5f3bc7 100644 (file)
--- 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] ",