On 01.11.2018 21:30, Ben Pfaff wrote: > On Thu, Nov 01, 2018 at 04:58:41PM +0300, Ilya Maximets wrote: >> Datapath implementation of 'flow_flush()' could use cmaps. >> dpif-netdev actively uses them to store flow tables and the polling >> threads. Flushing flows while in a quiescent state may lead to >> wrong memory accesses because implementaion of concurrent hash maps >> relies on the rcu protected pointers which could float away. >> >> CC: Alex Wang <[email protected]> >> Fixes: 1f8675481e8c ("ofproto-dpif-upcall: Fix ovs-vswitchd crash.") >> Signed-off-by: Ilya Maximets <[email protected]> > > I think this patch becomes unnecessary if my logic for the first one is > correct.
Sure. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
