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-Tag: v1.0.0~145
X-Git-Url: http://git.onelab.eu/?a=commitdiff_plain;h=f925807dc8ab649834b880bf34e985c2149ff5d2;p=sliver-openvswitch.git

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 620689fcf..baa889b45 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;
     }