[ovs-git] [openvswitch/ovs] ced0d8: ofproto-dpif-xlate: Stop forwarding MLD reports to...

Mon Dec 21 23:03:06 UTC 2020

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: ced0d8fb9267139226e5e4157eac9475c3d4086c
      https://github.com/openvswitch/ovs/commit/ced0d8fb9267139226e5e4157eac9475c3d4086c
  Author: XiaoXiong Ding <dingxiaoxiong at huawei.com>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M ofproto/ofproto-dpif-xlate.c

  Log Message:
  -----------
  ofproto-dpif-xlate: Stop forwarding MLD reports to group ports.

According with rfc4541 section 2.1.1, a snooping switch
should forward membership reports only to ports with
routers attached.The current code violates the RFC
forwarding membership reports to group ports as well.
The same issue doesn't exist with IPv4.

Fixes: 06994f879c ("mcast-snooping: Add Multicast Listener Discovery support")
Signed-off-by: XiaoXiong Ding <dingxiaoxiong at huawei.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>

  Commit: a27d70a8984879bc0a66afc2d7c35149659be24d
      https://github.com/openvswitch/ovs/commit/a27d70a8984879bc0a66afc2d7c35149659be24d
  Author: Eelco Chaudron <echaudro at redhat.com>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M NEWS
    M lib/conntrack-private.h
    M lib/conntrack.c
    M tests/system-traffic.at

  Log Message:
  -----------
  conntrack: add generic IP protocol support

Currently, userspace conntrack only tracks TCP, UDP, and ICMP, and all
other IP protocols are discarded, and the +inv state is returned. This
is not in line with the kernel conntrack. Where if no L4 information can
be extracted it's treated as generic L3. The change below mimics the
behavior of the kernel.

Signed-off-by: Eelco Chaudron <echaudro at redhat.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>

  Commit: e8451e1443e7c677190da9ddce5dbd4dfffc3590
      https://github.com/openvswitch/ovs/commit/e8451e1443e7c677190da9ddce5dbd4dfffc3590
  Author: Lorenzo Bianconi <lorenzo.bianconi at redhat.com>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M ovsdb/raft-private.h
    M ovsdb/raft.c

  Log Message:
  -----------
  raft: Add some debugging information to cluster/status command.

Introduce the following info useful for cluster debugging to
cluster/status command:
- time elapsed from last start/complete election
- election trigger (e.g. timeout)
- number of disconnections
- time elapsed from last raft messaged received

Acked-by: Dumitru Ceara <dceara at redhat.com>
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>

  Commit: b2b7e388f4d75baaa894e0433d51e09f944f2c5a
      https://github.com/openvswitch/ovs/commit/b2b7e388f4d75baaa894e0433d51e09f944f2c5a
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M tests/test-stream.c

  Log Message:
  -----------
  test-stream: Silence memory leak report.

AddressSanitizer reports this as a leak.
Let's just free the memory before exiting to avoid the noise.

'stream_close()' doesn't update the pointer, so this will not
change the return value.

Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Acked-by: Paolo Valerio <pvalerio at redhat.com>

  Commit: 9eeb44aadd63be7a1ef9f7fe5cfef8005935313b
      https://github.com/openvswitch/ovs/commit/9eeb44aadd63be7a1ef9f7fe5cfef8005935313b
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M ovsdb/ovsdb-tool.c

  Log Message:
  -----------
  ovsdb-tool: Fix datum leak in the show-log command.

Fixes: 4e92542cefb7 ("ovsdb-tool: Make "show-log" convert raw JSON to easier-to-read syntax.")
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
Acked-by: Dumitru Ceara <dceara at redhat.com>

  Commit: a35e9ab30215cb1841707704dee0658bb714ec44
      https://github.com/openvswitch/ovs/commit/a35e9ab30215cb1841707704dee0658bb714ec44
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M NEWS

  Log Message:
  -----------
  NEWS: Move '--offload-stats' entry to correct release.

Patch landed to 2.13, not 2.12.

Fixes: 164413156cf9 ("Add offload packets statistics")
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
Acked-by: Flavio Leitner <fbl at sysclose.org>

  Commit: 731378733619086403dee9d88b3bf34cee2544f9
      https://github.com/openvswitch/ovs/commit/731378733619086403dee9d88b3bf34cee2544f9
  Author: Brad Cowie <brad at wand.net.nz>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M Documentation/tutorials/faucet.rst

  Log Message:
  -----------
  Update tutorial for newer versions of Faucet and Open vSwitch.

Newer versions of Faucet use a dynamic OpenFlow pipeline based on what
features are enabled in the configuration file. Update log output, flow
table dumps and explanations to be consistent with newer Faucet versions.

Remove mentions of bugs that we have since fixed in Faucet since the
tutorial was originally written.

Adds documentation on changes to Open vSwitch commands to recommend
using a version that is compatible with the features of the tutorial.

Reported-by: Matthias Ableidinger <ableimat at gmx.at>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2018-August/047180.html
Signed-off-by: Brad Cowie <brad at wand.net.nz>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>

  Commit: c5b4b0ce95a31f1a2fadc8eecd8027434357b9eb
      https://github.com/openvswitch/ovs/commit/c5b4b0ce95a31f1a2fadc8eecd8027434357b9eb
  Author: Jianbo Liu <jianbol at nvidia.com>
  Date:   2020-12-21 (Mon, 21 Dec 2020)

  Changed paths:
    M lib/dpif-netlink.c
    M lib/dpif.c
    M lib/dpif.h
    M lib/netdev-offload-provider.h
    M lib/netdev-offload-tc.c
    M lib/netdev-offload.c
    M lib/netdev-offload.h
    M ofproto/ofproto-dpif-upcall.c
    M tests/system-offloads-traffic.at

  Log Message:
  -----------
  dpif-netlink: Fix issues of the offloaded flows counter.

The n_offloaded_flows counter is saved in dpif, and this is the first
one when ofproto is created. When flow operation is done by ovs-appctl
commands, such as, dpctl/add-flow, a new dpif is opened, and the
n_offloaded_flows in it can't be used. So, instead of using counter,
the number of offloaded flows is queried from each netdev, then sum
them up. To achieve this, a new API is added in netdev_flow_api to get
how many flows assigned to a netdev.

In order to get better performance, this number is calculated directly
from tc_to_ufid hmap for netdev-offload-tc, because flow dumping by tc
takes much time if there are many flows offloaded.

Fixes: af0618470507 ("dpif-netlink: Count the number of offloaded rules")
Signed-off-by: Jianbo Liu <jianbol at nvidia.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>

Compare: https://github.com/openvswitch/ovs/compare/02f76fb42ae9...c5b4b0ce95a3