[ovs-git] [openvswitch/ovs] c98166: datapath: Remove support to set vport stats.

[GitHub]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: c981665a3d6cd0a2e60cf08f6a34691ba9e8f314
      https://github.com/openvswitch/ovs/commit/c981665a3d6cd0a2e60cf08f6a34691ba9e8f314
  Author: Pravin B Shelar <pshelar at nicira.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M datapath/datapath.c
    M datapath/vport.c
    M datapath/vport.h

  Log Message:
  -----------
  datapath: Remove support to set vport stats.

This was required for old compatibility code which update stats
on fake bond interface. Now vswitchd has dropped it. This
support was always deprecated, so finally removing it.

Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
Acked-by: Andy Zhou <azhou at nicira.com>


  Commit: f3e54aa79279898834f1ac3c5fc45cd1286a18d3
      https://github.com/openvswitch/ovs/commit/f3e54aa79279898834f1ac3c5fc45cd1286a18d3
  Author: Li RongQing <roy.qing.li at gmail.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M datapath/vport.c
    M datapath/vport.h

  Log Message:
  -----------
  datapath: change the data type of error status to atomic_long_t

Change the date type of error status from u64 to atomic_long_t, and use atomic
operation, then remove the lock which is used to protect the error status.

The operation of atomic maybe faster than spin lock.

Cc: Pravin Shelar <pshelar at nicira.com>
Signed-off-by: Li RongQing <roy.qing.li at gmail.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


  Commit: 3b2bbceb56ce4bbc679a31a0877e396db2c845d9
      https://github.com/openvswitch/ovs/commit/3b2bbceb56ce4bbc679a31a0877e396db2c845d9
  Author: Pravin B Shelar <pshelar at nicira.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M datapath/linux/compat/include/linux/netdevice.h

  Log Message:
  -----------
  datapath: fix sparse warning.

datapath/linux/datapath.c:1418:28: warning: symbol
'i' shadows an earlier one
datapath/linux/datapath.c:1396:18: originally declared here

Signed-off-by: Pravin B Shelar <pshelar at nicira.com>


  Commit: f6a0c895fcece5bdf538f3f0afc13275c062b4d6
      https://github.com/openvswitch/ovs/commit/f6a0c895fcece5bdf538f3f0afc13275c062b4d6
  Author: Thomas Graf <tgraf at noironetworks.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M .travis.yml
    M .travis/build.sh
    M acinclude.m4
    M datapath/linux/Modules.mk
    M datapath/linux/compat/include/linux/if.h
    M datapath/linux/compat/include/linux/netdevice.h
    M datapath/linux/compat/include/linux/skbuff.h
    M datapath/linux/compat/include/net/ip_tunnels.h
    A datapath/linux/compat/include/net/udp.h
    M datapath/linux/compat/include/net/vxlan.h
    M datapath/linux/compat/ip_tunnels_core.c
    M datapath/linux/compat/vxlan.c
    M datapath/vport-geneve.c
    M datapath/vport-gre.c
    M datapath/vport-internal_dev.c
    M datapath/vport-lisp.c
    M datapath/vport-netdev.c
    M datapath/vport-vxlan.c

  Log Message:
  -----------
  datapath: Provide compatibility for kernels up to 3.17

Port datapath to work with kernrels up to 3.17 and use 3.16.2 as
the new kernel for CI testing.

Tested with 3.14, 3.16.2, and net-next (3.17).

Signed-off-by: Thomas Graf <tgraf at noironetworks.com>
Co-authored-by: Madhu Challa <challa at noironetworks.com>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


  Commit: 114fce23a7f1c48c6bd63e75ef698698c4103c97
      https://github.com/openvswitch/ovs/commit/114fce23a7f1c48c6bd63e75ef698698c4103c97
  Author: Samuel Gauthier <samuel.gauthier at 6wind.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M acinclude.m4
    M datapath/datapath.c
    M datapath/linux/compat/include/net/genetlink.h

  Log Message:
  -----------
  datapath: restore OVS_FLOW_CMD_NEW notifications

Since commit fb5d1e9e127a ("openvswitch: Build flow cmd netlink reply only if needed."),
the new flows are not notified to the listeners of OVS_FLOW_MCGROUP.

This commit fixes the problem by using the genl function, ie
genl_has_listerners() instead of netlink_has_listeners().

Signed-off-by: Samuel Gauthier <samuel.gauthier at 6wind.com>
Signed-off-by: Nicolas Dichtel <nicolas.dichtel at 6wind.com>
Signed-off-by: David S. Miller <davem at davemloft.net>
Acked-by: Pravin B Shelar <pshelar at nicira.com>


  Commit: e74d48171e590b50cdcb2798a3e7c5c32313887b
      https://github.com/openvswitch/ovs/commit/e74d48171e590b50cdcb2798a3e7c5c32313887b
  Author: Pravin B Shelar <pshelar at nicira.com>
  Date:   2014-09-20 (Sat, 20 Sep 2014)

  Changed paths:
    M datapath/actions.c
    M datapath/datapath.c
    M datapath/datapath.h
    M datapath/flow.c
    M datapath/vport-lisp.c
    M datapath/vport.c

  Log Message:
  -----------
  datapath: Remove pkt_key from OVS_CB.

OVS keeps pointer to packet key in skb->cb, but the packet key is
store on stack. This could make code bit tricky. So it is better to
get rid of the pointer.

Signed-off-by: Pravin B Shelar <pshelar at nicira.com>
Acked-by: Andy Zhou <azhou at nicira.com>


Compare: https://github.com/openvswitch/ovs/compare/ac8c20812b88...e74d48171e59