[Spice-devel] [PATCH spice-streaming-agent v2 3/4] Always log statistics

Frediano Ziglio fziglio at redhat.com
Mon May 21 10:23:07 UTC 2018


Allow to see frame size even if frame data is disabled.
Useful to compute frame statistics not spending huge amount of space
for frame data.

Signed-off-by: Frediano Ziglio <fziglio at redhat.com>
---
 src/spice-streaming-agent.cpp | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/spice-streaming-agent.cpp b/src/spice-streaming-agent.cpp
index 9ccc9da..d694eab 100644
--- a/src/spice-streaming-agent.cpp
+++ b/src/spice-streaming-agent.cpp
@@ -329,7 +329,7 @@ static void cursor_changes(StreamPort *stream_port, Display *display, int event_
 }
 
 #define STAT_LOG(format, ...) do { \
-    if (f_log && !log_binary) { \
+    if (f_log) { \
         fprintf(f_log, "%" PRIu64 ": " format "\n", get_time(), ## __VA_ARGS__); \
     } \
 } while(0)
@@ -387,12 +387,11 @@ do_capture(StreamPort &stream_port, FILE *f_log)
 
                 spice_stream_send_format(stream_port, width, height, codec);
             }
+            STAT_LOG("Frame of %zu bytes:", frame.buffer_size);
             if (f_log) {
                 if (log_binary) {
                     fwrite(frame.buffer, frame.buffer_size, 1, f_log);
                 } else {
-                    fprintf(f_log, "%" PRIu64 ": Frame of %zu bytes:\n",
-                            get_time(), frame.buffer_size);
                     hexdump(frame.buffer, frame.buffer_size, f_log);
                 }
             }
-- 
2.17.0



More information about the Spice-devel mailing list