[ovs-git] [openvswitch/ovs] 32697d: odp-execute: Fix length checking while executing c...

[Ilya Maximets]

  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 32697d4f6497c283a02e65e9c4515fc3183a55f6
      https://github.com/openvswitch/ovs/commit/32697d4f6497c283a02e65e9c4515fc3183a55f6
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    M lib/odp-execute.c

  Log Message:
  -----------
  odp-execute: Fix length checking while executing check_pkt_len action.

If dp-packet contains l2 padding or cutlen was applied to it, size will
be larger than the actual size of a payload and action will work
incorrectly.

Ex. Padding could be added during miniflow_extract() if detected.

Fixes: 5b34f8fc3b38 ("Add a new OVS action check_pkt_larger")
Reported-by: Miroslav Kubiczek <miroslav.kubiczek at adaptivemobile.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-discuss/2020-May/050157.html
Acked-by: Dumitru Ceara <dceara at redhat.com>
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: a64ee5493f0910012d201ea200d75365788acc3a
      https://github.com/openvswitch/ovs/commit/a64ee5493f0910012d201ea200d75365788acc3a
  Author: Yunjian Wang <wangyunjian at huawei.com>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    M ovsdb/monitor.c
    M ovsdb/rbac.c

  Log Message:
  -----------
  ovsdb: Remove duplicated include.

Signed-off-by: Yunjian Wang <wangyunjian at huawei.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: df5c293642cc07013e796e588eb7aead917e20a1
      https://github.com/openvswitch/ovs/commit/df5c293642cc07013e796e588eb7aead917e20a1
  Author: Tonghao Zhang <xiangxia.m.yue at gmail.com>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

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

  Log Message:
  -----------
  dpif-netdev: Delete the artificial flow limit.

The MAX_FLOWS constant was there from the introduction of dpif-netdev,
however, later new flow-limit mechanism was implemented that
controls number of datapath flows in a dynamic way on ofproto level.

So, we can just remove the limit and fully rely on ofproto to decide
what flow limit we need.  There are no limitations for flow table size
in dpif-netdev beside the artificial one.
'other_config:flow-limit' seems suitable to control this.

Suggested-by: Ilya Maximets <i.maximets at ovn.org>
Signed-off-by: Tonghao Zhang <xiangxia.m.yue at gmail.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 82a106ebfbf90b4b4d09782caf74d22f51af8d12
      https://github.com/openvswitch/ovs/commit/82a106ebfbf90b4b4d09782caf74d22f51af8d12
  Author: Adrian Moreno <amorenoz at redhat.com>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    M ofproto/bond.c
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  ofproto: Delete buckets when lb_output is false.

When lb-output-action is toggled back to "false" buckets are not being
deleted. Delete them as they will no longer be used.

Add unit test to verify buckets are correctly deleted.

Cc: Vishal Deep Ajmera <vishal.deep.ajmera at ericsson.com>
Signed-off-by: Adrian Moreno <amorenoz at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: d4a12d87a7a3a8d32a284c5e5b0121529188e580
      https://github.com/openvswitch/ovs/commit/d4a12d87a7a3a8d32a284c5e5b0121529188e580
  Author: Adrian Moreno <amorenoz at redhat.com>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    M lib/dpif-netdev-unixctl.man

  Log Message:
  -----------
  dpif-netdev-unixctl.man: Document bond-show command.

Document recently added ovs-appctl command.

Fixes: 9df65060cf4c ("userspace: Avoid dp_hash recirculation for balance-tcp bond mode.")
Acked-by: Flavio Leitner <fbl at sysclose.org>
Signed-off-by: Adrian Moreno <amorenoz at redhat.com>
Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 319fca52412b8514286a5ad012b16515a245d4e9
      https://github.com/openvswitch/ovs/commit/319fca52412b8514286a5ad012b16515a245d4e9
  Author: Ilya Maximets <i.maximets at ovn.org>
  Date:   2020-07-07 (Tue, 07 Jul 2020)

  Changed paths:
    M AUTHORS.rst

  Log Message:
  -----------
  AUTHORS: Add Adrian Moreno.

Signed-off-by: Ilya Maximets <i.maximets at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/edd04838c61c...319fca52412b