]> Git Repo - linux.git/commit
net: openvswitch: fix unwanted error log on timeout policy probing
authorIlya Maximets <[email protected]>
Wed, 3 Apr 2024 20:38:01 +0000 (22:38 +0200)
committerJakub Kicinski <[email protected]>
Sat, 6 Apr 2024 05:16:06 +0000 (22:16 -0700)
commit4539f91f2a801c0c028c252bffae56030cfb2cae
treebe4e8f65653b89637e4facc5116bf1e9eaf38285
parentc88b9b4cde17aec34fb9bfaf69f9f72a1c44f511
net: openvswitch: fix unwanted error log on timeout policy probing

On startup, ovs-vswitchd probes different datapath features including
support for timeout policies.  While probing, it tries to execute
certain operations with OVS_PACKET_ATTR_PROBE or OVS_FLOW_ATTR_PROBE
attributes set.  These attributes tell the openvswitch module to not
log any errors when they occur as it is expected that some of the
probes will fail.

For some reason, setting the timeout policy ignores the PROBE attribute
and logs a failure anyway.  This is causing the following kernel log
on each re-start of ovs-vswitchd:

  kernel: Failed to associated timeout policy `ovs_test_tp'

Fix that by using the same logging macro that all other messages are
using.  The message will still be printed at info level when needed
and will be rate limited, but with a net rate limiter instead of
generic printk one.

The nf_ct_set_timeout() itself will still print some info messages,
but at least this change makes logging in openvswitch module more
consistent.

Fixes: 06bd2bdf19d2 ("openvswitch: Add timeout support to ct action")
Signed-off-by: Ilya Maximets <[email protected]>
Acked-by: Eelco Chaudron <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Jakub Kicinski <[email protected]>
net/openvswitch/conntrack.c
This page took 0.054017 seconds and 4 git commands to generate.