Re: [ovs-dev] [PATCH] flow: Only show IPv4 flow information for IPv4 flows

2012-09-20 Thread Ben Pfaff
On Fri, Sep 21, 2012 at 09:26:43AM +0900, Simon Horman wrote: > On Thu, Sep 20, 2012 at 08:29:19AM -0700, Ben Pfaff wrote: > > On Thu, Sep 20, 2012 at 04:14:26PM +0900, Simon Horman wrote: > > > Signed-off-by: Simon Horman > > > > > > --- > > > > > > I'm unsure of the history of why this is so,

Re: [ovs-dev] [PATCH] flow: Only show IPv4 flow information for IPv4 flows

2012-09-20 Thread Simon Horman
On Thu, Sep 20, 2012 at 08:29:19AM -0700, Ben Pfaff wrote: > On Thu, Sep 20, 2012 at 04:14:26PM +0900, Simon Horman wrote: > > Signed-off-by: Simon Horman > > > > --- > > > > I'm unsure of the history of why this is so, > > but it seems misleading at best. > > This seems reasonable but it shoul

Re: [ovs-dev] [PATCH] flow: Only show IPv4 flow information for IPv4 flows

2012-09-20 Thread Ben Pfaff
On Thu, Sep 20, 2012 at 04:14:26PM +0900, Simon Horman wrote: > Signed-off-by: Simon Horman > > --- > > I'm unsure of the history of why this is so, > but it seems misleading at best. This seems reasonable but it should handle ARP as well. Thanks, Ben.

[ovs-dev] [PATCH] flow: Only show IPv4 flow information for IPv4 flows

2012-09-20 Thread Simon Horman
Signed-off-by: Simon Horman --- I'm unsure of the history of why this is so, but it seems misleading at best. --- lib/flow.c |2 +- tests/ofp-print.at |2 +- tests/ofproto.at | 10 +- 3 files changed, 7 insertions(+), 7 deletions(-) diff --git a/lib/flow.c b/lib/flo