On Thu, Jun 28, 2012 at 06:16:06AM +0000, Amit Tewari wrote: > As per our analysis and requirement we provided the patch for selected > openflow messages which carry useful data in data field for controller from > switch. > > Please confirm whether you have applied the patch we provided earlier?
No. It doesn't make sense to add a special case for every message in the middle of the code. If we need to handle this at all, we should handle it in a single place without duplicate special cases. Also, your patch was against an old version of Open vSwitch. We only accept patches against the newest version. > Please let me know if am missing any openflow message. See, that's the problem. You *can* miss some messages if you do it your way. _______________________________________________ discuss mailing list discuss@openvswitch.org http://openvswitch.org/mailman/listinfo/discuss
