On Tue, Nov 06, 2018 at 04:27:04PM +0300, Ilya Maximets wrote: > 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]>
Thanks, applied to master. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
