From f925807dc8ab649834b880bf34e985c2149ff5d2 Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Thu, 15 Apr 2010 16:36:14 -0700 Subject: [PATCH] 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. --- vswitchd/bridge.c | 10 ++++++---- 1 file 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; } -- 2.30.2