Re: [ovs-dev] [PATCH] miniflow: Fix miniflow push of L4 port numbers.

2015-02-03 Thread Ben Pfaff
On Mon, Feb 02, 2015 at 10:47:48PM +, Kevin Traynor wrote: > Replace a 64 bit copy of L4 src/dst ports that was also > including additional packet params (e.g. TCP Seq Num). This > was later resulting in all packets from the flow missing in > the EMC. > > Signed-off-by: Kevin Traynor > Report

Re: [ovs-dev] [PATCH] miniflow: Fix miniflow push of L4 port numbers.

2015-02-02 Thread Jarno Rajahalme
age- >> From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Kevin Traynor >> Sent: Monday, February 2, 2015 10:48 PM >> To: dev@openvswitch.org >> Subject: [ovs-dev] [PATCH] miniflow: Fix miniflow push of L4 port numbers. >> >> Replace a 64 bit copy of L4 s

Re: [ovs-dev] [PATCH] miniflow: Fix miniflow push of L4 port numbers.

2015-02-02 Thread Traynor, Kevin
dding, but it would make it less readable/consistent. Thanks, Kevin. > -Original Message- > From: dev [mailto:dev-boun...@openvswitch.org] On Behalf Of Kevin Traynor > Sent: Monday, February 2, 2015 10:48 PM > To: dev@openvswitch.org > Subject: [ovs-dev] [PATCH] miniflow: Fix

[ovs-dev] [PATCH] miniflow: Fix miniflow push of L4 port numbers.

2015-02-02 Thread Kevin Traynor
Replace a 64 bit copy of L4 src/dst ports that was also including additional packet params (e.g. TCP Seq Num). This was later resulting in all packets from the flow missing in the EMC. Signed-off-by: Kevin Traynor Reported-by: Ciara Loftus --- lib/flow.c |9 ++--- 1 files changed, 6 ins