Re: [ovs-dev] [PATCH] ovn-controller: Use ofputil_encode_set_config to set the switch config

2016-02-10 Thread Ben Pfaff
On Sat, Feb 06, 2016 at 03:45:29PM +0530, Numan Siddique wrote: > On 02/06/2016 03:01 AM, Ben Pfaff wrote: > > However, the problem you report probably indicates a bug somewhere else, > > because the OFPT_SET_CONFIG message is generated from the > > OFPT_GET_CONFIG_REPLY sent by the switch. If the

Re: [ovs-dev] [PATCH] ovn-controller: Use ofputil_encode_set_config to set the switch config

2016-02-06 Thread Numan Siddique
On 02/06/2016 03:01 AM, Ben Pfaff wrote: > However, the problem you report probably indicates a bug somewhere else, > because the OFPT_SET_CONFIG message is generated from the > OFPT_GET_CONFIG_REPLY sent by the switch. If the switch rejects the > OFPT_SET_CONFIG, then, it could mean that the OFPT

Re: [ovs-dev] [PATCH] ovn-controller: Use ofputil_encode_set_config to set the switch config

2016-02-05 Thread Ben Pfaff
On Wed, Feb 03, 2016 at 04:08:04PM +0530, Numan Siddique wrote: > After the commit ad99e2e, setting the switch config 'miss_send_len' > is failing in ovn-controller because of validation in OFPT_SET_CONFIG > messages. > > Signed-off-by: Numan Siddique This is an improvement in any case, because

[ovs-dev] [PATCH] ovn-controller: Use ofputil_encode_set_config to set the switch config

2016-02-03 Thread Numan Siddique
After the commit ad99e2e, setting the switch config 'miss_send_len' is failing in ovn-controller because of validation in OFPT_SET_CONFIG messages. Signed-off-by: Numan Siddique --- ovn/controller/pinctrl.c | 18 ++ 1 file changed, 6 insertions(+), 12 deletions(-) diff --git a/o