Re: [ovs-dev] [PATCH] datapath: Additional logging for -EINVAL on flow setups.

2014-07-02 Thread Jesse Gross
On Wed, Jul 2, 2014 at 4:46 PM, Thomas Graf wrote: > On 07/01/14 at 05:43pm, Jesse Gross wrote: >> There are many possible ways that a flow can be invalid so we've >> added logging for most of them. This adds logs for the remaining >> possible cases so there isn't any ambiguity while debugging. >>

Re: [ovs-dev] [PATCH] datapath: Additional logging for -EINVAL on flow setups.

2014-07-02 Thread Thomas Graf
On 07/01/14 at 05:43pm, Jesse Gross wrote: > There are many possible ways that a flow can be invalid so we've > added logging for most of them. This adds logs for the remaining > possible cases so there isn't any ambiguity while debugging. > > CC: Federico Iezzi > Signed-off-by: Jesse Gross LGT

Re: [ovs-dev] [PATCH] datapath: Additional logging for -EINVAL on flow setups.

2014-07-02 Thread Thomas Graf
On 07/01/14 at 05:43pm, Jesse Gross wrote: > There are many possible ways that a flow can be invalid so we've > added logging for most of them. This adds logs for the remaining > possible cases so there isn't any ambiguity while debugging. > > CC: Federico Iezzi > Signed-off-by: Jesse Gross It'

Re: [ovs-dev] [PATCH] datapath: Additional logging for -EINVAL on flow setups.

2014-07-02 Thread Ben Pfaff
On Tue, Jul 01, 2014 at 05:43:39PM -0700, Jesse Gross wrote: > There are many possible ways that a flow can be invalid so we've > added logging for most of them. This adds logs for the remaining > possible cases so there isn't any ambiguity while debugging. > > CC: Federico Iezzi > Signed-off-by:

[ovs-dev] [PATCH] datapath: Additional logging for -EINVAL on flow setups.

2014-07-01 Thread Jesse Gross
There are many possible ways that a flow can be invalid so we've added logging for most of them. This adds logs for the remaining possible cases so there isn't any ambiguity while debugging. CC: Federico Iezzi Signed-off-by: Jesse Gross --- datapath/datapath.c | 12 +--- datapath/fl