On Tue, 2013-03-05 at 09:34 -0800, Ben Pfaff wrote: > 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.
Great, thanks for your positive feedback! > > 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? Not yet, currently I have no idea on how to integrate it with openflow tables. > > I'd prefer to avoid adding a new struct flow member for this case. I > hope we can reuse an existing member. > Yeah, makes sense, I will dig which members can be reused. Thanks! _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
