Re: [ovs-dev] [PATCH] ovs-ofctl: Only allow usable protocols for group commands

2014-11-06 Thread Simon Horman
On Thu, Nov 06, 2014 at 03:01:23PM -0800, Ben Pfaff wrote: > On Wed, Nov 05, 2014 at 01:57:39PM +0900, Simon Horman wrote: > > parse_ofp_group_mod_str() may limit the usable protocols according > > to the group and in particular its actions. However, without this > > change ovs-ofctl ignores this c

Re: [ovs-dev] [PATCH] ovs-ofctl: Only allow usable protocols for group commands

2014-11-06 Thread Ben Pfaff
On Wed, Nov 05, 2014 at 01:57:39PM +0900, Simon Horman wrote: > parse_ofp_group_mod_str() may limit the usable protocols according > to the group and in particular its actions. However, without this > change ovs-ofctl ignores this calculation. > > Signed-off-by: Simon Horman It looks useful, but

Re: [ovs-dev] [PATCH] ovs-ofctl: Only allow usable protocols for group commands

2014-11-05 Thread Jarno Rajahalme
Thanks, Simon! Acked-by: Jarno Rajahalme Pushed to master, Jarno On Nov 4, 2014, at 8:57 PM, Simon Horman wrote: > parse_ofp_group_mod_str() may limit the usable protocols according > to the group and in particular its actions. However, without this > change ovs-ofctl ignores this c

[ovs-dev] [PATCH] ovs-ofctl: Only allow usable protocols for group commands

2014-11-04 Thread Simon Horman
parse_ofp_group_mod_str() may limit the usable protocols according to the group and in particular its actions. However, without this change ovs-ofctl ignores this calculation. Signed-off-by: Simon Horman --- utilities/ovs-ofctl.c | 13 - 1 file changed, 8 insertions(+), 5 deletions(-