On 22/11/2013 06:35, Ben Pfaff wrote:
On Thu, Nov 21, 2013 at 05:04:34PM +0800, Alexander Wu wrote:
V3:
   No change.

V2:
   No change.

V1:
   Make the enums allowed

Signed-off-by: Alexander Wu <alexander...@huawei.com>
Reviewed-by: Simon Horman <ho...@verge.net.au>

This misjudges the purpose of this function.  It does not deliver an
"allow" or "drop" verdict.  Instead, it gives the caller its best
guess whether a particular message means that the controller has
decided to accept this OpenFlow connection.

I see that this function could be improved in a few ways.  I'll send
out a patch to do that.  But I believe that this patch should be
dropped.


Thanks, I'll remove this patch next version.

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to