Re: [ovs-dev] [PATCH branch-2.7 00/14] Backports for branch-2.7

2017-04-19 Thread Joe Stringer
On 18 April 2017 at 17:20, Jarno Rajahalme  wrote:
> For the series:
>
> Acked-by: Jarno Rajahalme 
>
> One of the userspace changes (indicate if had labels) changes the OVS library 
> interface, so leave it for you to decide what to do with that.

Thanks, applied to branch-2.7 without that labels change.
___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


Re: [ovs-dev] [PATCH branch-2.7 00/14] Backports for branch-2.7

2017-04-18 Thread Jarno Rajahalme
For the series:

Acked-by: Jarno Rajahalme 

One of the userspace changes (indicate if had labels) changes the OVS library 
interface, so leave it for you to decide what to do with that.

  Jarno

> On Apr 18, 2017, at 5:09 PM, Joe Stringer  wrote:
> 
> This is a trimmed down set of the backports presented by Jarno here:
> https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329817.html
> 
> The fixes (including label inheritance and subsequent fixups) are included
> in this series, while the unrelated refactors and unnecessary compat changes
> are dropped.
> 
> Jarno acked this proposal already, but I'm sending it out to the list for
> completeness.
> 
> The following patch is new in this proposal, as suggested by Jarno:
> "datapath: Avoid struct copy on conntrack labels."
> 
> Jarno Rajahalme (9):
>  datapath: Use inverted tuple in ovs_ct_find_existing() if NATted.
>  datapath: Do not trigger events for unconfirmed connections.
>  lib: Indicate if netlink message had labels.
>  datapath: Unionize ovs_key_ct_label with a u32 array.
>  datapath: Simplify labels length logic.
>  datapath: Refactor labels initialization.
>  datapath: Inherit master's labels.
>  datapath: Avoid struct copy on conntrack labels.
>  ofp-util: Ignore unknown fields in ofputil_decode_packet_in2().
> 
> Jiri Benc (2):
>  datapath: remove unused functions
>  datapath: remove unnecessary EXPORT_SYMBOLs
> 
> Pablo Neira Ayuso (1):
>  datapath: handle NF_REPEAT from nf_conntrack_in()
> 
> Thadeu Lima de Souza Cascardo (1):
>  datapath: fix flow stats accounting when node 0 is not possible
> 
> Yi-Hung Wei (1):
>  nx-match: Fix oxm decode.
> 
> datapath/conntrack.c   | 172 ++---
> datapath/datapath.c|   2 -
> datapath/flow.c|   6 +-
> datapath/flow_table.c  |   3 +-
> datapath/linux/compat/include/linux/openvswitch.h  |   8 +-
> .../include/net/netfilter/nf_conntrack_core.h  |  21 +++
> datapath/vport-netdev.c|   1 -
> datapath/vport.c   |  17 --
> datapath/vport.h   |   1 -
> lib/ct-dpif.h  |   1 +
> lib/netlink-conntrack.c|   1 +
> lib/nx-match.c |  23 ++-
> lib/nx-match.h |   4 +-
> lib/ofp-util.c |   2 +-
> tests/system-traffic.at|  52 +++
> 15 files changed, 223 insertions(+), 91 deletions(-)
> 
> -- 
> 2.11.1
> 

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev


[ovs-dev] [PATCH branch-2.7 00/14] Backports for branch-2.7

2017-04-18 Thread Joe Stringer
This is a trimmed down set of the backports presented by Jarno here:
https://mail.openvswitch.org/pipermail/ovs-dev/2017-March/329817.html

The fixes (including label inheritance and subsequent fixups) are included
in this series, while the unrelated refactors and unnecessary compat changes
are dropped.

Jarno acked this proposal already, but I'm sending it out to the list for
completeness.

The following patch is new in this proposal, as suggested by Jarno:
"datapath: Avoid struct copy on conntrack labels."

Jarno Rajahalme (9):
  datapath: Use inverted tuple in ovs_ct_find_existing() if NATted.
  datapath: Do not trigger events for unconfirmed connections.
  lib: Indicate if netlink message had labels.
  datapath: Unionize ovs_key_ct_label with a u32 array.
  datapath: Simplify labels length logic.
  datapath: Refactor labels initialization.
  datapath: Inherit master's labels.
  datapath: Avoid struct copy on conntrack labels.
  ofp-util: Ignore unknown fields in ofputil_decode_packet_in2().

Jiri Benc (2):
  datapath: remove unused functions
  datapath: remove unnecessary EXPORT_SYMBOLs

Pablo Neira Ayuso (1):
  datapath: handle NF_REPEAT from nf_conntrack_in()

Thadeu Lima de Souza Cascardo (1):
  datapath: fix flow stats accounting when node 0 is not possible

Yi-Hung Wei (1):
  nx-match: Fix oxm decode.

 datapath/conntrack.c   | 172 ++---
 datapath/datapath.c|   2 -
 datapath/flow.c|   6 +-
 datapath/flow_table.c  |   3 +-
 datapath/linux/compat/include/linux/openvswitch.h  |   8 +-
 .../include/net/netfilter/nf_conntrack_core.h  |  21 +++
 datapath/vport-netdev.c|   1 -
 datapath/vport.c   |  17 --
 datapath/vport.h   |   1 -
 lib/ct-dpif.h  |   1 +
 lib/netlink-conntrack.c|   1 +
 lib/nx-match.c |  23 ++-
 lib/nx-match.h |   4 +-
 lib/ofp-util.c |   2 +-
 tests/system-traffic.at|  52 +++
 15 files changed, 223 insertions(+), 91 deletions(-)

-- 
2.11.1

___
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev