On Tue, May 20, 2014 at 03:41:55PM -0700, Ben Pfaff wrote: > On Fri, Apr 11, 2014 at 06:34:19PM -0300, Flavio Leitner wrote: > > This patch adds the functions needed for multicast snooping > > in order to send the packets to the correct ports. > > > > They are marked as OVS_UNUSED for now. > > > > Acked-by: Thomas Graf <tg...@redhat.com> > > Signed-off-by: Flavio Leitner <f...@redhat.com> > > I think these are OK. > > Do the NULL cases (e.g. "mcast group port is unknown, dropping") > indicate bugs, or can they happen in normal operation?
Good question. My understanding is that they should not happen, but to be honest, I didn't fully understand the life cycle of xbundle to know if there is a transient state where this could happen, so I followed what is in mac-learning.c. > I think it would be best to squash this with whatever commit > introduces the first user, dropping the OVS_UNUSED. Ok. fbl _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
