Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-04-05 Thread Simon Horman
On Thu, Apr 05, 2018 at 05:16:59PM +0300, Simon Horman wrote: > On Wed, Apr 04, 2018 at 04:51:01PM -0700, Ben Pfaff wrote: > > On Thu, Mar 22, 2018 at 03:32:58PM +0100, Simon Horman wrote: > > > On Thu, Mar 22, 2018 at 03:46:17PM +0200, Roi Dayan wrote: > > > > > > > > > > > > On 22/03/2018 15:44

Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-04-05 Thread Simon Horman
On Wed, Apr 04, 2018 at 04:51:01PM -0700, Ben Pfaff wrote: > On Thu, Mar 22, 2018 at 03:32:58PM +0100, Simon Horman wrote: > > On Thu, Mar 22, 2018 at 03:46:17PM +0200, Roi Dayan wrote: > > > > > > > > > On 22/03/2018 15:44, Roi Dayan wrote: > > > > > > > > > > > > On 22/03/2018 15:22, Roi Daya

Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-04-04 Thread Ben Pfaff
On Thu, Mar 22, 2018 at 03:32:58PM +0100, Simon Horman wrote: > On Thu, Mar 22, 2018 at 03:46:17PM +0200, Roi Dayan wrote: > > > > > > On 22/03/2018 15:44, Roi Dayan wrote: > > > > > > > > > On 22/03/2018 15:22, Roi Dayan wrote: > > > > Raise the error up instead of ignoring it. > > > > Before

Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-03-22 Thread Simon Horman
On Thu, Mar 22, 2018 at 03:46:17PM +0200, Roi Dayan wrote: > > > On 22/03/2018 15:44, Roi Dayan wrote: > > > > > > On 22/03/2018 15:22, Roi Dayan wrote: > > > Raise the error up instead of ignoring it. > > > Before this commit beside an error an incorrect rule was also printed. > > > > > > Sig

Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-03-22 Thread Roi Dayan
On 22/03/2018 15:44, Roi Dayan wrote: On 22/03/2018 15:22, Roi Dayan wrote: Raise the error up instead of ignoring it. Before this commit beside an error an incorrect rule was also printed. Signed-off-by: Roi Dayan Reviewed-by: Paul Blakey Signed-off-by: Simon Horman oops sorry about t

Re: [ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-03-22 Thread Roi Dayan
On 22/03/2018 15:22, Roi Dayan wrote: Raise the error up instead of ignoring it. Before this commit beside an error an incorrect rule was also printed. Signed-off-by: Roi Dayan Reviewed-by: Paul Blakey Signed-off-by: Simon Horman oops sorry about the signed off. i cherry-picked from maste

[ovs-dev] [PATCH branch-2.8] lib/tc: Handle error parsing action in nl_parse_single_action

2018-03-22 Thread Roi Dayan
Raise the error up instead of ignoring it. Before this commit beside an error an incorrect rule was also printed. Signed-off-by: Roi Dayan Reviewed-by: Paul Blakey Signed-off-by: Simon Horman --- lib/tc.c | 15 ++- 1 file changed, 10 insertions(+), 5 deletions(-) diff --git a/lib/