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 <[email protected]> > Reviewed-by: Simon Horman <[email protected]>
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. _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
