On 2/26/2019 10:38 AM, Ilya Maximets wrote:
It's a normal case for 'find' function, especially because this happens for every first packet of flow that was not offloaded yet. Should not warn about this. Dropped to DBG to avoid log trashing in case of big number of new flows.
Makes sense, pushed to master. Thanks Ian
CC: Yuanhan Liu <y...@fridaylinux.org> Fixes: 241bad15d99a ("dpif-netdev: associate flow with a mark id") Acked-by: Flavio Leitner <f...@sysclose.org> Signed-off-by: Ilya Maximets <i.maxim...@samsung.com> --- lib/dpif-netdev.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c index ec84378a0..4d6d0c372 100644 --- a/lib/dpif-netdev.c +++ b/lib/dpif-netdev.c @@ -2183,8 +2183,8 @@ megaflow_to_mark_find(const ovs_u128 *mega_ufid) } }- VLOG_WARN("Mark id for ufid "UUID_FMT" was not found\n",- UUID_ARGS((struct uuid *)mega_ufid)); + VLOG_DBG("Mark id for ufid "UUID_FMT" was not found\n", + UUID_ARGS((struct uuid *)mega_ufid)); return INVALID_FLOW_MARK; }
_______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
