diff options
author | Ben Pfaff <blp@nicira.com> | 2010-04-15 16:36:14 -0700 |
---|---|---|
committer | Ben Pfaff <blp@nicira.com> | 2010-04-16 09:53:08 -0700 |
commit | f925807dc8ab649834b880bf34e985c2149ff5d2 (patch) | |
tree | 28c817de08c3f259810bc653c30456549045fe1e /vswitchd | |
parent | 50b2ad57cb4e2478baf7b71f0c31433eb6a37981 (diff) |
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.
Diffstat (limited to 'vswitchd')
-rw-r--r-- | vswitchd/bridge.c | 10 |
1 files changed, 6 insertions, 4 deletions
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; } |