[ovs-git] [openvswitch/ovs] 6b241d: netdev-vport: Factor-out tunnel Push-pop code into...

Thu May 19 02:40:12 UTC 2016

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 6b241d6452916b367fc9cc8c1d83cb113f9c2e5f
      https://github.com/openvswitch/ovs/commit/6b241d6452916b367fc9cc8c1d83cb113f9c2e5f
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/automake.mk
    A lib/netdev-native-tnl.c
    A lib/netdev-native-tnl.h
    M lib/netdev-provider.h
    A lib/netdev-vport-private.h
    M lib/netdev-vport.c
    M lib/packets.h

  Log Message:
  -----------
  netdev-vport: Factor-out tunnel Push-pop code into separate module.

It is better to move tunnel push-pop action specific functions into
separate module.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 1c8f98d96a64163c857591e95c235c8dc2f8efa8
      https://github.com/openvswitch/ovs/commit/1c8f98d96a64163c857591e95c235c8dc2f8efa8
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c
    M lib/netdev-native-tnl.c
    M lib/netdev-native-tnl.h
    M lib/netdev-provider.h
    M lib/netdev.c
    M lib/netdev.h

  Log Message:
  -----------
  netdev: Return number of packet from netdev_pop_header()

Current tunnel-pop API does not allow the netdev implementation
retain a packet but STT can keep a packet from batch of packets
during TCP reassembly processing. To return exact count of
valid packet STT need to pass this number of packet parameter
as a reference.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: eb18b2a6f9e923c5ceeec9f6528fe9bdc7d1493e
      https://github.com/openvswitch/ovs/commit/eb18b2a6f9e923c5ceeec9f6528fe9bdc7d1493e
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dp-packet.h

  Log Message:
  -----------
  dp-packet: Add private data

This scratchpad can be used by any layer to keep private data.
STT will use it for TCP reassembly state.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: b513ea342e764191752a4063d8318b8488473c20
      https://github.com/openvswitch/ovs/commit/b513ea342e764191752a4063d8318b8488473c20
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dp-packet.h

  Log Message:
  -----------
  dp-packet: use packet reset function.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: f7ce48110d4cd918d79498fdbf43db19e4ac0abb
      https://github.com/openvswitch/ovs/commit/f7ce48110d4cd918d79498fdbf43db19e4ac0abb
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif-netdev: rename packet_batch

Next patch introduces new structure named packet_batch. So
I am renaming it to packet_batch_per_flow.
This does not change any functionality.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 1895cc8dbb64721b07f3f67167d5baebf224e240
      https://github.com/openvswitch/ovs/commit/1895cc8dbb64721b07f3f67167d5baebf224e240
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dp-packet.h
    M lib/dpif-netdev.c
    M lib/dpif.c
    M lib/netdev.c
    M lib/netdev.h
    M lib/odp-execute.c
    M lib/odp-execute.h
    M ofproto/ofproto-dpif-xlate.c

  Log Message:
  -----------
  dpif-netdev: create batch object

DPDK datapath operate on batch of packets. To pass the batch of
packets around we use packets array and count.  Next patch needs
to associate meta-data with each batch of packets. So Introducing
a batch structure to make handling the metadata easier.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 9235b4793e75e45075592bf203700f4ec76423f8
      https://github.com/openvswitch/ovs/commit/9235b4793e75e45075592bf203700f4ec76423f8
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c
    M lib/netdev.c
    M lib/netdev.h

  Log Message:
  -----------
  dpif-netdev: Fix memory leak in tunnel header pop action.

The tunnel header pop action can leak batch of packet
in case of error. Following patch fixex the error code path.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 4c742796418900728814630bd2db549d8d10263c
      https://github.com/openvswitch/ovs/commit/4c742796418900728814630bd2db549d8d10263c
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif-netdev: Fix memory leak in tunnel header push action.

in case of error from netdev_push_header() batch of packets was not
freed. Following patch fixes this issue.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: a260d966388ae8477d6787337569820386ea24b4
      https://github.com/openvswitch/ovs/commit/a260d966388ae8477d6787337569820386ea24b4
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif-netdev: Refactor fast path process function.

Once datapath support large packets, we need to segment packet before
sending it to upcall. Refactoring this code make it bit cleaner.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 66525ef3b8d604e47ca0bd1a7510b1ae9f3eb5ba
      https://github.com/openvswitch/ovs/commit/66525ef3b8d604e47ca0bd1a7510b1ae9f3eb5ba
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif-netdev: Refactor userspace action

Large segment support need to use this refactored function to
send individual segments.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 1bc3f0ed2615aa2475e4c92339253a6092746ad4
      https://github.com/openvswitch/ovs/commit/1bc3f0ed2615aa2475e4c92339253a6092746ad4
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/packets.c
    M lib/packets.h
    M ofproto/tunnel.c

  Log Message:
  -----------
  tunnel: Add IP ECN related functions.

Set and get functions for IP explicit congestion notification flag.
These function would be used by STT reassembly code.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

  Commit: 98e3f58e8982829c0dc582043f8bf193cd1875d6
      https://github.com/openvswitch/ovs/commit/98e3f58e8982829c0dc582043f8bf193cd1875d6
  Author: Pravin B Shelar <pshelar at ovn.org>
  Date:   2016-05-18 (Wed, 18 May 2016)

  Changed paths:
    M lib/tnl-ports.c
    M lib/tnl-ports.h
    M ofproto/tunnel.c

  Log Message:
  -----------
  tnl-ports: Handle STT ports.

STT uses TCP port so we need to filter traffic on basis of TCP
port numbers.

Signed-off-by: Pravin B Shelar <pshelar at ovn.org>
Acked-by: Jesse Gross <jesse at kernel.org>

Compare: https://github.com/openvswitch/ovs/compare/ae8b926072fc...98e3f58e8982