On Tue, May 19, 2020 at 2:59 PM txfh2007 via discuss
<ovs-discuss@openvswitch.org> wrote:
>
> Hi all:
>     I am using OVS-2.13 together with DPDK-19.11 and hw-offload turnd on.  I 
> have found when I tried to capture pkts by using ovs-tcpdump, the main 
> process crash, I have found in log there is an assertion fail: "
> |util(dp_netdev_flow_43)|EMER|lib/cmap.c:922: assertion cmap_replace__(impl, 
> old_node, new_node, hash, h1) || cmap_replace__(impl, old_node, new_node, 
> hash, h2) failed in cmap_replace() "
>     After tracing by gdb, I have found it may caused by reload_affected_pmds 
> mechanism: when we add or remove ports in datapath(e.g. capture pkts by 
> ovs-tcpdump script), the reload_affected_pmds should call flow_mark_flush and 
> then mark_to_flow_disassociate several times, for the second time that we 
> call mark_to_flow_disassociate, as the flow->mark has already been set by 
> 0xffffffff, so the cmap_remove would emit an error.
>     I don't know how to avoid this error?
>
> Thanks for your help !
> Timo
I don't try to reproduce the crash, you can offer more information,
such as your commands which you run.
Other crash I found, but patch is not ready for now, because I am busy
for something else. You can try it. I hope it is useful for you.

diff --git a/lib/dpif-netdev.c b/lib/dpif-netdev.c
index d8df8b50ec71..ad3ad22249c1 100644
--- a/lib/dpif-netdev.c
+++ b/lib/dpif-netdev.c
@@ -2366,6 +2366,12 @@ mark_to_flow_disassociate(struct
dp_netdev_pmd_thread *pmd,
     struct cmap_node *mark_node = CONST_CAST(struct cmap_node *,
                                              &flow->mark_node);

+    if (OVS_UNLIKELY(mark == INVALID_FLOW_MARK)) {
+        dp_netdev_flow_unref(flow);
+        return ret;
+    }
+
     cmap_remove(&flow_mark.mark_to_flow, mark_node, hash_int(mark, 0));
     flow->mark = INVALID_FLOW_MARK;

> _______________________________________________
> discuss mailing list
> disc...@openvswitch.org
> https://mail.openvswitch.org/mailman/listinfo/ovs-discuss



-- 
Best regards, Tonghao
_______________________________________________
discuss mailing list
disc...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-discuss

Reply via email to