bridge: Don't log warnings when revalidating.
authorBen Pfaff <blp@nicira.com>
Thu, 15 Apr 2010 23:36:14 +0000 (16:36 -0700)
committerBen Pfaff <blp@nicira.com>
Fri, 16 Apr 2010 16:53:08 +0000 (09:53 -0700)
The rest of the warnings along this path follow this rule, but this one
didn't.  Make it consistent.

vswitchd/bridge.c

index 620689f..baa889b 100644 (file)
@@ -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;
     }