[ovs-dev] [PATCH 1/2] ofproto-dpif: Move process_special() to ofproto-dpif-xlate.c.
Justin Pettit
jpettit at nicira.com
Tue Jun 18 07:11:08 UTC 2013
On Jun 17, 2013, at 6:26 PM, Ethan Jackson <ethan at nicira.com> wrote:
> Acked-by: Ethan Jackson <ethan at nicira.com>
>
> 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
More information about the dev
mailing list