diff mbox

[ovs-dev,1/2] netlink-conntrack: Change unsupported IPPROTO log to debug.

Message ID 1490733829-95917-1-git-send-email-jarno@ovn.org
State Accepted
Delegated to: Joe Stringer
Headers show

Commit Message

Jarno Rajahalme March 28, 2017, 8:43 p.m. UTC 
No point littering the logs with messages on an unsupported protocol,
so change the log to debug level.

Signed-off-by: Jarno Rajahalme <jarno@ovn.org>
---
 lib/netlink-conntrack.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Joe Stringer March 28, 2017, 10:16 p.m. UTC | #1 
On 28 March 2017 at 13:43, Jarno Rajahalme <jarno@ovn.org> wrote:
> No point littering the logs with messages on an unsupported protocol,
> so change the log to debug level.
>
> Signed-off-by: Jarno Rajahalme <jarno@ovn.org>

Acked-by: Joe Stringer <joe@ovn.org>
diff mbox

Patch

diff --git a/lib/netlink-conntrack.c b/lib/netlink-conntrack.c
index 8b82db2..f0e2aea 100644
--- a/lib/netlink-conntrack.c
+++ b/lib/netlink-conntrack.c
@@ -463,8 +463,8 @@  nl_ct_parse_tuple_proto(struct nlattr *nla, struct ct_dpif_tuple *tuple)
             tuple->dst_port = nl_attr_get_be16(attrs[CTA_PROTO_DST_PORT]);
         } else {
             /* Unsupported IPPROTO and no ports, leave them zeroed.
-             * We have parsed the ip_proto, so this is not a total failure. */
-            VLOG_INFO_RL(&rl, "Unsupported L4 protocol: %u.", tuple->ip_proto);
+             * We have parsed the ip_proto, so this is not a failure. */
+            VLOG_DBG_RL(&rl, "Unsupported L4 protocol: %u.", tuple->ip_proto);
         }
     } else {
         VLOG_ERR_RL(&rl, "Could not parse nested tuple protocol options. "