On Fri, Mar 01, 2013 at 09:03:44PM +0800, Cong Wang wrote: > WARNING: This patch *only* compiles! NOT tested yet! > > This patch adds the initial IGMP snooping support. > I sent out this patch early to get some early reviews, > especially the design of the code. > > Any comments are welcome! > > Cc: Ben Pfaff <b...@nicira.com> > Cc: Jesse Gross <je...@nicira.com> > Not-Yet-Signed-off-by: Cong Wang <amw...@redhat.com>
Hi Cong. IGMP snooping is a nice feature to have. The code looks OK at first skim. One thing we've been trying to do as we introduce new features is to try to make them useful apart from the "normal" action where we can. Do you have an idea how that can be done for IGMP snooping? I'd prefer to avoid adding a new struct flow member for this case. I hope we can reuse an existing member. Thanks, Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
