On Fri, Nov 16, 2012 at 10:43:39AM +0900, Simon Horman wrote: > On Mon, Nov 12, 2012 at 10:23:00AM -0800, Ben Pfaff wrote: > > On Wed, Nov 07, 2012 at 05:03:05PM +0900, Simon Horman wrote: > > > Versions may be configured using a comma delimited list as > > > the value for 'openflow-versions' the 'other-config' column > > > of the Bridge table. > > > > > > Currently list elements other than 'OpenFlow10' and 'OpenFlow12' will > > > be ignored. > > > > > > If the list is empty, then OpenFlow10 is used. > > > This default is consistent with the behaviour of ovs-vswtichd > > > prior to this patch. > > > > > > Signed-off-by: Simon Horman <ho...@verge.net.au> > > > > This seems to me like core enough functionality that we should add a new > > column for it. I suggest "protocols" as the name and that it should be > > a set of strings, one protocol name per string, with a default used if > > the set is empty. > > That sounds reasonable to me.
The implementation of ofputil_versions_from_string() in this patch calls ofputil_version_from_string() as a helper. Reasonable enough on the face of it, but ofputil_version_from_string() exits with a fatal error if its input in invalid. That's OK in ovs-ofctl, etc., but we don't want ovs-vswitchd to do that. There's a stray VLOG_WARN in here, too. Thanks, Ben. _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
