On 02.11.2018 21:25, Ben Pfaff wrote: > revalidator_purge() iterates and modifies umap->cmap. This should > not happen in quiescent state, because cmap implementation based > on rcu protected variables. Let's narrow the quiescent period > to avoid possible wrong memory accesses. > > CC: Joe Stringer <[email protected]> > Fixes: 9fce0584a643 ("revalidator: Use 'cmap' for storing ukeys.") > Reported-by: Ilya Maximets <[email protected]> > Signed-off-by: Ben Pfaff <[email protected]> > --- > ofproto/ofproto-dpif-upcall.c | 38 ++++++++++++++++---------------------- > 1 file changed, 16 insertions(+), 22 deletions(-) >
Thanks, Acked-by: Ilya Maximets <[email protected]> _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
