From: Paul Durrant <paul.durrant@citrix.com>
Date: Fri, 19 Dec 2014 11:59:05 +0000 (+0000)
Subject: Squash buffer overlow error messages in transmit
X-Git-Tag: 8.1.0-rc1~76
X-Git-Url: http://xenbits.xensource.com/gitweb?a=commitdiff_plain;h=9dd9878fef92fb282d5a7f3241abcc28843776f5;p=people%2Fpauldu%2Fxenvif.git

Squash buffer overlow error messages in transmit

Occasionally packets are too fragmented to fit into the maximum number of
requests allowed by the netif protocol. In this case the packet preparation
function will fail with STATUS_BUFFER_OVERFLOW and the alternative copy-
coalesce prepation function will be called. These error messages are
therefore un-interesting (since the number of packets directly granted vs.
those that were copied is available in the statistics dump initiated by the
debug VIRQ).
This patch squashes the error messages to avoid filling logs.

Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
---

diff --git a/src/xenvif/transmitter.c b/src/xenvif/transmitter.c
index 8125a5a..29d17fa 100644
--- a/src/xenvif/transmitter.c
+++ b/src/xenvif/transmitter.c
@@ -787,7 +787,8 @@ __TransmitterRingGrantPayload(
 
 fail3:
 fail2:
-    Error("fail2\n");
+    if (status != STATUS_BUFFER_OVERFLOW)
+        Error("fail2\n");
 
     if (Fragment != NULL) {
         ASSERT3P(Fragment->Context, ==, Packet);
@@ -800,7 +801,8 @@ fail2:
     }
 
 fail1:
-    Error("fail1 (%08x)\n", status);
+    if (status != STATUS_BUFFER_OVERFLOW)
+        Error("fail1 (%08x)\n", status);
 
     ASSERT3P(Fragment, ==, NULL);