On 8/30/22 12:02, Dumitru Ceara wrote: > On 8/29/22 13:30, Xavier Simonart wrote: >> active_v4_flows count was intialized when the northd node was computed. >> However, neither sb_multicast_group nor en_sb_igmp_group causes northd >> updates. Hence this count could keep increasing while processing igmp groups. >> >> This issue was sometimes 'hidden' by northd recomputes due to lflow unable >> to be incrementally processed (sb busy). >> >> active_v4_flows is now reinitialized right before building flows >> (i.e. as part of the lflow node, which is computed on igmp group changes). >> >> Reported-at: https://bugzilla.redhat.com/show_bug.cgi?id=2094710 >> >> Signed-off-by: Xavier Simonart <[email protected]> > Looks good to me, thanks! > > Acked-by Dumitru Ceara <[email protected]> >
Fixing the ack tag so that patchwork marks it properly: Acked-by: Dumitru Ceara <[email protected]> >> --- > Note for the maintainers: there's an extra whitespace before --- which > causes the notes below to be added to the commit log when applying the > patch. I guess that should be fixed up at apply time. > > Regards, > Dumitru > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
