The 05/26/2021 14:59, Simon Horman wrote: > On Fri, May 21, 2021 at 01:01:29AM +0000, Jianbo Liu wrote: > > The 05/11/2021 11:38, Roi Dayan wrote: > > > From: Jianbo Liu <[email protected]> > > > > > > There is a race condidtion between purger and handler. Handler may > > > create new ukey and install it while executing 'ovs-appctl > > > revalidator/purge' command. However, before handler calls > > > transition_ukey() in handle_upcalls(), purger may get this ukey from > > > umap, then evict and delete it. This will trigger ovs_abort in > > > transition_ukey() for handler because it is trying to set state to > > > EVICTED or OPERATIONAL, but ukey is already in DELETED state. > > > To fix this issue, purger must not delete ukey in VISIBLE state. > > > > Friendly ping for review ... > > Thanks. > > This looks ok to me, but it seems to make the testsuite unhappy.
Thanks. It looks dpif_netdev behaves differently from dpif_netlink. I will fix it. > > https://github.com/horms2/ovs/actions/runs/878488417 > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Ftravis-ci.org%2Fgithub%2Fhorms2%2Fovs%2Fbuilds%2F772454219&data=04%7C01%7Cjianbol%40nvidia.com%7Cd5f41ad0947645f50ff708d920462a52%7C43083d15727340c1b7db39efd9ccc17a%7C0%7C0%7C637576308895924312%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C2000&sdata=drU7kbpU4RyMbveYqv2p3xQ39tE8h52t27hteFn%2FBIM%3D&reserved=0 -- _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
