[ovs-git] [openvswitch/ovs] 132911: extract-odp-netlink-h: Portablitiy improvement

GitHub noreply at github.com
Thu Nov 26 09:37:57 UTC 2015


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 132911b3b54b89b4aa8abd1e185b4a71d01957b1
      https://github.com/openvswitch/ovs/commit/132911b3b54b89b4aa8abd1e185b4a71d01957b1
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M build-aux/extract-odp-netlink-h

  Log Message:
  -----------
  extract-odp-netlink-h: Portablitiy improvement

\t is GNU sed extension.  Use [[:space:]] instead.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 58e3dd5aa09ffdf0f7920e55927a06fc3ef70e7a
      https://github.com/openvswitch/ovs/commit/58e3dd5aa09ffdf0f7920e55927a06fc3ef70e7a
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/ofp-actions.h

  Log Message:
  -----------
  ofp-actions: Define IPPORT_FTP by ourselves

IPPORT_FTP is not always provided by system headers.  (eg. NetBSD, OS X)

This hides the enum on Linux but I don't think it causes a problem.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 57b0dbe3bf25c8b477b670ede5d19b43a2ed357b
      https://github.com/openvswitch/ovs/commit/57b0dbe3bf25c8b477b670ede5d19b43a2ed357b
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/tnl-arp-cache.c

  Log Message:
  -----------
  tnl-arp-cache: Fix NetBSD build

On NetBSD, netinet/icmp6.h has a few pre-requisite headers.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 1e450da53a9ae0357a421cedcbf2634ba051c401
      https://github.com/openvswitch/ovs/commit/1e450da53a9ae0357a421cedcbf2634ba051c401
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/netdev-bsd.c

  Log Message:
  -----------
  netdev-bsd: Update after eth_addr changes

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 43000bc793cb977cca8c7251c05639705d9a9895
      https://github.com/openvswitch/ovs/commit/43000bc793cb977cca8c7251c05639705d9a9895
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M m4/openvswitch.m4

  Log Message:
  -----------
  openvswitch.m4: Portability improvement

${parameter/pattern/string} substituion is not portable.
Replace them with $SED equivalent.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 6a54bae19648a305a52fb6a911723cbc49bc822b
      https://github.com/openvswitch/ovs/commit/6a54bae19648a305a52fb6a911723cbc49bc822b
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/daemon-unix.c

  Log Message:
  -----------
  daemon_switch_user: Improve portablility

NetBSD doesn't have [gs]etres[ug]id.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Andy Zhou <azhou at nicira.com>


  Commit: 622aff9668176cec0fde985928b863e871c42831
      https://github.com/openvswitch/ovs/commit/622aff9668176cec0fde985928b863e871c42831
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/completion.at

  Log Message:
  -----------
  completion.at: Improve portability

NetBSD's /bin/sh complains on the syntax of bash array.

While the use of eval might seem overkill, it's tricky to avoid
the error because the generated code will be a part of the surrounding
subshell and the syntax check is done a bit earlier than the execution
of these conditionals.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 1593af75c2685af3b53ec3cdb516bb0e89649ac7
      https://github.com/openvswitch/ovs/commit/1593af75c2685af3b53ec3cdb516bb0e89649ac7
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/test-atomic.c

  Log Message:
  -----------
  test-atomic: Bump test duration for multi-thread tests

This makes tests pass on on my single-cpu NetBSD-7 VM.

How frequently preemption happens depends on the cpu scheduler.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 12e2ba2e85cc811b946ab867d6971d2f0938fc04
      https://github.com/openvswitch/ovs/commit/12e2ba2e85cc811b946ab867d6971d2f0938fc04
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ofproto-macros.at
    M tests/ofproto.at
    M tests/ovs-ofctl.at

  Log Message:
  -----------
  ofproto-macros.at: Rename vconn_windows_sub to vconn_sub

This is a preparation to add non-windows stuff.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: f6783a7a3b9fcedc43c31e5fab0b7a810d42ab8c
      https://github.com/openvswitch/ovs/commit/f6783a7a3b9fcedc43c31e5fab0b7a810d42ab8c
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M debian/ovs-monitor-ipsec
    M ofproto/ipfix-gen-entities
    M utilities/ovs-dev.py
    M utilities/ovs-pipegen.py
    M utilities/qemu-wrap.py
    M vtep/ovs-vtep
    M xenserver/usr_share_openvswitch_scripts_ovs-xapi-sync

  Log Message:
  -----------
  use "/usr/bin/env python" for shebangs for python scripts

On NetBSD, a typical path of python interpreter is /usr/pkg/bin/python.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: eb95259dcdc2f488186e35bc2dab68861d7856e2
      https://github.com/openvswitch/ovs/commit/eb95259dcdc2f488186e35bc2dab68861d7856e2
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ovn-controller-vtep.at

  Log Message:
  -----------
  ovn-controller-vtep.at: Fix a race

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 65d43fdca9511a7dd5c98197a2721801abfe3260
      https://github.com/openvswitch/ovs/commit/65d43fdca9511a7dd5c98197a2721801abfe3260
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/dpif-netdev.c

  Log Message:
  -----------
  dpif_dummy_override: Allow overriding a non-existing provider

This allows --enable-dummy=system with a userland-only build.
It's useful for testsuite.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: e0b68d0eac6ba82cc3769ccbb75b689d45eeeb6a
      https://github.com/openvswitch/ovs/commit/e0b68d0eac6ba82cc3769ccbb75b689d45eeeb6a
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/vtep-ctl.at

  Log Message:
  -----------
  vtep-ctl.at: Fix a regex

} is an ordinary character in "basic" regex.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 4756794516e1e8b677aa73b0beeaa1525679b49f
      https://github.com/openvswitch/ovs/commit/4756794516e1e8b677aa73b0beeaa1525679b49f
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  ofproto-dpif.at: Fix a race

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 89e97cae68c1f827b6cc1edcd8a47318f39e99c0
      https://github.com/openvswitch/ovs/commit/89e97cae68c1f827b6cc1edcd8a47318f39e99c0
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M ofproto/ofproto-dpif.c

  Log Message:
  -----------
  ofproto-dpif: Remove a debug log

The message seems stale and it happens on port-mod.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 641c6188033cd2cc266a4f74b42562e849d5a3d3
      https://github.com/openvswitch/ovs/commit/641c6188033cd2cc266a4f74b42562e849d5a3d3
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ofproto.at

  Log Message:
  -----------
  ofproto.at: Portability improvement

== is a bash dialect.  Use = instead.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: a6d1a2997db48320d8bd0710f6c3adc83166ca01
      https://github.com/openvswitch/ovs/commit/a6d1a2997db48320d8bd0710f6c3adc83166ca01
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ofproto.at

  Log Message:
  -----------
  ofproto.at: Workaround a race

While a barrier serializes requests from the same connection,
it doesn't wait for requests from other connections to the switch.
Replace the barrier with infamous "sleep 1" to workaround the problem.

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: de1b46dea61af2917fb44cfbe58ae41aae4ff429
      https://github.com/openvswitch/ovs/commit/de1b46dea61af2917fb44cfbe58ae41aae4ff429
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M tests/ovs-ofctl.at

  Log Message:
  -----------
  ovs-ofctl.at: Fix a comment typo

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: ff882f96da022aab2c1be3b9ef9e2f83ae10812f
      https://github.com/openvswitch/ovs/commit/ff882f96da022aab2c1be3b9ef9e2f83ae10812f
  Author: YAMAMOTO Takashi <yamamoto at midokura.com>
  Date:   2015-11-26 (Thu, 26 Nov 2015)

  Changed paths:
    M lib/classifier-private.h

  Log Message:
  -----------
  classifier-private.h: Whitespace in a comment

Signed-off-by: YAMAMOTO Takashi <yamamoto at midokura.com>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/f2e0627402c6...ff882f96da02


More information about the git mailing list