From: Ben Pfaff <blp@nicira.com>
Date: Thu, 15 Apr 2010 23:36:14 +0000 (-0700)
Subject: bridge: Don't log warnings when revalidating.
X-Git-Url: https://pintos-os.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=f925807dc8ab649834b880bf34e985c2149ff5d2;p=openvswitch

bridge: Don't log warnings when revalidating.

The rest of the warnings along this path follow this rule, but this one
didn't.  Make it consistent.
---

diff --git a/vswitchd/bridge.c b/vswitchd/bridge.c
index 620689fc..baa889b4 100644
--- a/vswitchd/bridge.c
+++ b/vswitchd/bridge.c
@@ -2223,10 +2223,12 @@ process_flow(struct bridge *br, const flow_t *flow,
 
     /* Drop frames on ports reserved for mirroring. */
     if (in_port->is_mirror_output_port) {
-        static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
-        VLOG_WARN_RL(&rl, "bridge %s: dropping packet received on port %s, "
-                     "which is reserved exclusively for mirroring",
-                     br->name, in_port->name);
+        if (packet) {
+            static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
+            VLOG_WARN_RL(&rl, "bridge %s: dropping packet received on port "
+                         "%s, which is reserved exclusively for mirroring",
+                         br->name, in_port->name);
+        }
         goto done;
     }