[ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-06-23 Thread Andre Mantas
This patch allows ofp_packet_out messages to be added to bundles. In a multi controller scenario, packet_out messages inside bundles can serve as a commit_reply for other controllers - since the original commit_reply is only forwarded to the controller that sent the commit_request. Tested with

Re: [ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-06-29 Thread Jarno Rajahalme
> On Jun 23, 2016, at 4:51 AM, Andre Mantas wrote: > > This patch allows ofp_packet_out messages to be added to bundles. In a > multi controller scenario, packet_out messages inside bundles can serve > as a commit_reply for other controllers - since the original commit_reply > is only forward

Re: [ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-07-13 Thread André Mantas
Thanks for the review. I'm not sure if I will be able to address your second point tho. Would this be a problem? As for the third point, would ofproto.c be a good place for the helper function? Something like: static enum ofperr ofproto_extract_packet_out(struct ofproto *p, const struct ofp_heade

Re: [ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-07-14 Thread Jarno Rajahalme
I've been working on adding group support for bundles for the last week or so, so I have most of the related code fresh in my head now. So, please submit a new RFC patch with the changes you can make and I'll take it from there. Jarno > On Jul 13, 2016, at 5:35 PM, André Mantas wrote: > > T

Re: [ovs-dev] [PATCH] ofproto: ofp_packet_out messages in bundles

2016-07-16 Thread André Mantas
I've sent the new patch. I wasn't sure how to do it so I just sent the first patch with subject [PATCH 1/2] and the new patch with [PATCH 2/2]. Hope it doesn't cause much problem. Alternatively I can send a pull request in github if you want. Thanks. Jarno Rajahalme escreveu no dia quinta, 14/0