[ovs-dev] [PATCH 1/2] mcast_snoop: make mcast_fport_bundle generic

Ben Pfaff blp at nicira.com
Wed Feb 4 00:07:24 UTC 2015


On Tue, Jan 20, 2015 at 01:20:56PM -0200, Flavio Leitner wrote:
> The struct mcast_fport_bundle will be used for ports
> forwarding Reports too, so make it generic.
> 
> Signed-off-by: Flavio Leitner <fbl at redhat.com>

I apologize that it has taken me a long time to review this patch.  As
usual, I have been busy; most recently, I was out the last five days to
an out-of-town corporate event where they kept us busy from 7am to 10pm
or later.  Hardly had time to breathe, let along review patches.

I need a little bit of help understanding this patch.  In particular,
the changes remove a 'vlan' parameter from a lot of function prototypes,
and I don't see anything replace it.  Was the VLAN somehow unused
previously?  I'd like to understand why it was needed before and somehow
not needed now.

Thanks,

Ben.



More information about the dev mailing list