yeahuh / rpms / qemu-kvm

Forked from rpms/qemu-kvm 2 years ago
Clone

Blame SOURCES/kvm-virtiofsd-Add-timestamp-to-the-log-with-FUSE_LOG_DEB.patch

902636
From 52e93f2dc499ead339bf808dac3480b369dfadd1 Mon Sep 17 00:00:00 2001
902636
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
902636
Date: Mon, 27 Jan 2020 19:01:39 +0100
902636
Subject: [PATCH 068/116] virtiofsd: Add timestamp to the log with
902636
 FUSE_LOG_DEBUG level
902636
MIME-Version: 1.0
902636
Content-Type: text/plain; charset=UTF-8
902636
Content-Transfer-Encoding: 8bit
902636
902636
RH-Author: Dr. David Alan Gilbert <dgilbert@redhat.com>
902636
Message-id: <20200127190227.40942-65-dgilbert@redhat.com>
902636
Patchwork-id: 93517
902636
O-Subject: [RHEL-AV-8.2 qemu-kvm PATCH 064/112] virtiofsd: Add timestamp to the log with FUSE_LOG_DEBUG level
902636
Bugzilla: 1694164
902636
RH-Acked-by: Philippe Mathieu-Daudé <philmd@redhat.com>
902636
RH-Acked-by: Stefan Hajnoczi <stefanha@redhat.com>
902636
RH-Acked-by: Sergio Lopez Pascual <slp@redhat.com>
902636
902636
From: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>
902636
902636
virtiofsd has some threads, so we see a lot of logs with debug option.
902636
It would be useful for debugging if we can see the timestamp.
902636
902636
Add nano second timestamp, which got by get_clock(), to the log with
902636
FUSE_LOG_DEBUG level if the syslog option isn't set.
902636
902636
The log is like as:
902636
902636
  # ./virtiofsd -d -o vhost_user_socket=/tmp/vhostqemu0 -o source=/tmp/share0 -o cache=auto
902636
  ...
902636
  [5365943125463727] [ID: 00000002] fv_queue_thread: Start for queue 0 kick_fd 9
902636
  [5365943125568644] [ID: 00000002] fv_queue_thread: Waiting for Queue 0 event
902636
  [5365943125573561] [ID: 00000002] fv_queue_thread: Got queue event on Queue 0
902636
902636
Signed-off-by: Masayoshi Mizuma <m.mizuma@jp.fujitsu.com>
902636
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
902636
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
902636
(cherry picked from commit 50fb955aa0e6ede929422146936cf68bf1ca876f)
902636
Signed-off-by: Miroslav Rezanina <mrezanin@redhat.com>
902636
---
902636
 tools/virtiofsd/passthrough_ll.c | 9 ++++++++-
902636
 1 file changed, 8 insertions(+), 1 deletion(-)
902636
902636
diff --git a/tools/virtiofsd/passthrough_ll.c b/tools/virtiofsd/passthrough_ll.c
902636
index f08324f..98114a3 100644
902636
--- a/tools/virtiofsd/passthrough_ll.c
902636
+++ b/tools/virtiofsd/passthrough_ll.c
902636
@@ -36,6 +36,7 @@
902636
  */
902636
 
902636
 #include "qemu/osdep.h"
902636
+#include "qemu/timer.h"
902636
 #include "fuse_virtio.h"
902636
 #include "fuse_log.h"
902636
 #include "fuse_lowlevel.h"
902636
@@ -2276,7 +2277,13 @@ static void log_func(enum fuse_log_level level, const char *fmt, va_list ap)
902636
     }
902636
 
902636
     if (current_log_level == FUSE_LOG_DEBUG) {
902636
-        localfmt = g_strdup_printf("[ID: %08ld] %s", syscall(__NR_gettid), fmt);
902636
+        if (!use_syslog) {
902636
+            localfmt = g_strdup_printf("[%" PRId64 "] [ID: %08ld] %s",
902636
+                                       get_clock(), syscall(__NR_gettid), fmt);
902636
+        } else {
902636
+            localfmt = g_strdup_printf("[ID: %08ld] %s", syscall(__NR_gettid),
902636
+                                       fmt);
902636
+        }
902636
         fmt = localfmt;
902636
     }
902636
 
902636
-- 
902636
1.8.3.1
902636