diff options
| author | Justin Pettit <jpettit@nicira.com> | 2012-05-02 10:20:07 -0700 |
|---|---|---|
| committer | Justin Pettit <jpettit@nicira.com> | 2012-05-02 10:20:07 -0700 |
| commit | b0a5c43bd76c248408326f51823e67dcd87d0994 (patch) | |
| tree | bf41b1c4fa516aa9c716a678dce75ae0c50b23d6 | |
| parent | f66b87de59564e70bf6325e6bd211e45243187c8 (diff) | |
| download | openvswitch-b0a5c43bd76c248408326f51823e67dcd87d0994.tar.gz | |
ofproto: Rate-limit STP status errors on non-existent ports.
As part of the bridge's main loop, it queries the STP status of all
ports. If the port doesn't exist, log files can become filled with
warning messages. This situation is very unusual, since system devices
do not normally disappear, but it's easy enough to rate-limit these
messages.
Bug #10936
Reported-by: Reid Price <reid@nicira.com>
Signed-off-by: Justin Pettit <jpettit@nicira.com>
| -rw-r--r-- | ofproto/ofproto.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ofproto/ofproto.c b/ofproto/ofproto.c index cb46d26df..6ef2a5d72 100644 --- a/ofproto/ofproto.c +++ b/ofproto/ofproto.c @@ -664,8 +664,8 @@ ofproto_port_get_stp_status(struct ofproto *ofproto, uint16_t ofp_port, { struct ofport *ofport = ofproto_get_port(ofproto, ofp_port); if (!ofport) { - VLOG_WARN("%s: cannot get STP status on nonexistent port %"PRIu16, - ofproto->name, ofp_port); + VLOG_WARN_RL(&rl, "%s: cannot get STP status on nonexistent " + "port %"PRIu16, ofproto->name, ofp_port); return ENODEV; } |