On Jun 17, 2013, at 6:26 PM, Ethan Jackson <[email protected]> wrote:

> Acked-by: Ethan Jackson <[email protected]>
> 
> I'd go ahead and just move stp_should_process_flow() and
> stp_process_packet() into ofproto-dpif-xlate as part of this patch.
> I'm planning to do it anyways as part of my larger restructuring.
> It's fine as is though if you'd prefer to leave it.

Since a bunch of the other STP related functions are in there, I just left it.  
We should probably move all of those functions into stp.{ch} for consistency.  
It sounded like you were going to be moving a bunch of that stuff with your 
ofproto-dpif restructuring.

Thanks for the review.  I pushed this to master.

--Justin


_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to