[ovs-git] [openvswitch/ovs] 6e1a6b: netdev-tc-offloads: Fix travis compilation error

GitHub noreply at github.com
Thu Nov 23 16:52:35 UTC 2017


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 6e1a6bfd997796b93b5c01bd4321ae52b3faa4cd
      https://github.com/openvswitch/ovs/commit/6e1a6bfd997796b93b5c01bd4321ae52b3faa4cd
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Fix travis compilation error

Travis complains about variable length array usage, use
a fixed size instead.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: 408671c4655eceabf730d55520654711383235d0
      https://github.com/openvswitch/ovs/commit/408671c4655eceabf730d55520654711383235d0
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/tc.c

  Log Message:
  -----------
  netdev-tc-offloads: Fix accidental skipping of extended pedit keys

We only support extended pedit keys for now, so it's the type we
expect. Skip the legacy ones instead.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: d6118e628988a2fc8a8648d420d4e27f5ada9091
      https://github.com/openvswitch/ovs/commit/d6118e628988a2fc8a8648d420d4e27f5ada9091
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/tc.c
    M lib/tc.h

  Log Message:
  -----------
  netdev-tc-offloads: Verify csum flags on dump from tc

On dump, parse and verify the tc csum action update flags
in the same way as we put them.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: e13bbbab3cf8d9ca91465f6462e6483efe92340b
      https://github.com/openvswitch/ovs/commit/e13bbbab3cf8d9ca91465f6462e6483efe92340b
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/tc.c

  Log Message:
  -----------
  netdev-tc-offloads: Remove redundant brackets

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: 518bbe9971e49e2a67efdc3ffce906705eb776a2
      https://github.com/openvswitch/ovs/commit/518bbe9971e49e2a67efdc3ffce906705eb776a2
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/netdev-tc-offloads.c

  Log Message:
  -----------
  netdev-tc-offloads: Remove redundant loop handling ovs action set

OVS action set always has a single nested OVS_KEY_ATTR_*
so there is no need to iterate it.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: fbaf1bf9ef696c4aba24ace9aa8857cd6465a251
      https://github.com/openvswitch/ovs/commit/fbaf1bf9ef696c4aba24ace9aa8857cd6465a251
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/tc.c

  Log Message:
  -----------
  tc: Fix wrong struct variable order

Fix the struct variable order to corrospond with
it's usage.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


  Commit: a7ce5b85291155a3eec096786416c256f3db2709
      https://github.com/openvswitch/ovs/commit/a7ce5b85291155a3eec096786416c256f3db2709
  Author: Paul Blakey <paulb at mellanox.com>
  Date:   2017-11-23 (Thu, 23 Nov 2017)

  Changed paths:
    M lib/tc.c

  Log Message:
  -----------
  tc: Send csum action only if we need to update csum

Currently we send the tc csum action even if it's not needed.
Fix that by sending it only if csum update flags isn't zero.

Signed-off-by: Paul Blakey <paulb at mellanox.com>
Reviewed-by: Roi Dayan <roid at mellanox.com>
Signed-off-by: Simon Horman <simon.horman at netronome.com>


Compare: https://github.com/openvswitch/ovs/compare/3728b3b0316b...a7ce5b852911


More information about the git mailing list