On Oct 14, 2012, at 8:41 PM, Pravin B Shelar <pshe...@nicira.com> wrote: > This patch was posted by Kyle. I fixed few issues found in earlier > version. > > V7: > - Fix according to comments posted on V6. > V6: > - Fix more comments addressed from Jesse. > V5: > - Address another round of comments from Jesse. > V4: > - Address 2 comments from Jesse: > - When processing actions, if OVS_CB(skb)->tun_key is NULL, point it at one > on the stack temporarily. This goes away when we remove the ability to set > tun_id outside the scope of tun_key. > - Move tun_key to the end of struct sw_flow_key. > V3: > - Fix issues found during review by Jesse. > - Add a NEWS entry around tunnel code no longer assuming symmetric input and > output tunnel keys. > > V2: > - Fix blank line addition/removal found by Simon. > - Fix hex printing output found by Simon.
Pravin: Can you comment if the main changes you made to my patch are around handling tunnel configuration per-flow instead of per-port? I looked at that part specifically, and it appears you made changes similar to what I had been making and was going to resubmit later this week. Other than this, did you make any other changes? I didn't look closer at the rest of the patch, but it appears to be mostly my stock patch. If the above is true: Acked-by: Kyle Mestery <kmest...@cisco.com> Also, I'll rework the second patch in my series later this week once I'm back from traveling and repost that one. Thanks, Kyle _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
