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