[Spice-devel] [spice 1/2] server: Don't reset the latency before showing it in the invalid net test error message.

Francois Gouget fgouget at codeweavers.com
Wed Jul 15 07:34:00 PDT 2015


Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
 server/main_channel.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/server/main_channel.c b/server/main_channel.c
index e7df3f9..f1b38af 100644
--- a/server/main_channel.c
+++ b/server/main_channel.c
@@ -969,11 +969,11 @@ static int main_channel_handle_parsed(RedChannelClient *rcc, uint32_t size, uint
                 mcc->net_test_id = 0;
                 if (roundtrip <= mcc->latency) {
                     // probably high load on client or server result with incorrect values
+                    spice_printerr("net test: invalid values, latency %" PRIu64
+                                   " roundtrip %" PRIu64 ". assuming high"
+                                   " bandwidth", mcc->latency, roundtrip);
                     mcc->latency = 0;
                     mcc->net_test_stage = NET_TEST_STAGE_INVALID;
-                    spice_printerr("net test: invalid values, latency %" PRIu64
-                               " roundtrip %" PRIu64 ". assuming high"
-                               "bandwidth", mcc->latency, roundtrip);
                     red_channel_client_start_connectivity_monitoring(&mcc->base,
                                                                      CLIENT_CONNECTIVITY_TIMEOUT);
                     break;
-- 
2.1.4



More information about the Spice-devel mailing list