On Wed, Feb 04, 2015 at 11:07:05AM -0200, Flavio Leitner wrote:
> On Tuesday, February 03, 2015 04:07:24 PM Ben Pfaff wrote:
> > 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 <f...@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.
> 
> understandable, I just don't want to miss the 2.4 train.
>  
> > 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.
> 
> Yes, it wasn't used at all so I took it out in the cleanup.

Oh, that makes sense.  I'll take another quick look then.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to