On Thu, Mar 30, 2023 at 11:13:32AM +0800, Faicker Mo wrote: > Thanks for your testing. > Add the ignore msg "failed to offload flow" to OVS_TRAFFIC_VSWITCHD_STOP can > pass the fail test. > I'll post an update later.
Thanks. I'll look out for it. > From: Simon Horman <[email protected]> > Date: 2023-03-29 23:22:17 > To: Faicker Mo <[email protected]> > Cc: [email protected],Eelco Chaudron <[email protected]>,Ilya Maximets > <[email protected]> > Subject: Re: [ovs-dev] [PATCH v9] netdev-offload-tc: del ufid mapping if > device not exist>On Wed, Mar 15, 2023 at 11:26:55AM +0800, Faicker Mo wrote: > >> The device may be deleted and added with ifindex changed. > >> The tc rules on the device will be deleted if the device is deleted. > >> The func tc_del_filter will fail when flow del. The mapping of > >> ufid to tc will not be deleted. > >> The traffic will trigger the same flow(with same ufid) to put to tc > >> on the new device. Duplicated ufid mapping will be added. > >> If the hashmap is expanded, the old mapping entry will be the first entry, > >> and now the dp flow can't be deleted. > >> > >> > >> Signed-off-by: Faicker Mo <[email protected]> > > > >Hi, > > > >I have run the test added by this patch without the code changes of this > >patch, and verified that it fails reliably. > > > >I have also run the test with the code changes in a for loop on on a low-end > >Ubuntu 22.04 VM 34,500k times (for about 2 days). Sorry there is a typo here. It should be 34,500 (no k). I.e. just under 35k. ... _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
