[ovs-git] [openvswitch/ovs] 1c33f0: netdev: Pass 'netdev_class' to ->run() and ->wait(...

GitHub noreply at github.com
Mon Aug 15 18:16:53 UTC 2016


  Branch: refs/heads/master
  Home:   https://github.com/openvswitch/ovs
  Commit: 1c33f0c35e6b81d600798f255a896a85507d51ed
      https://github.com/openvswitch/ovs/commit/1c33f0c35e6b81d600798f255a896a85507d51ed
  Author: Daniele Di Proietto <diproiettod at vmware.com>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M lib/netdev-bsd.c
    M lib/netdev-dummy.c
    M lib/netdev-linux.c
    M lib/netdev-provider.h
    M lib/netdev-vport.c
    M lib/netdev.c

  Log Message:
  -----------
  netdev: Pass 'netdev_class' to ->run() and ->wait().

This will allow run() and wait() methods to be shared between different
classes and still perform class-specific work.

Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: e98d0cb3ac85ba2266a82ddd63bba42cd378ad3f
      https://github.com/openvswitch/ovs/commit/e98d0cb3ac85ba2266a82ddd63bba42cd378ad3f
  Author: Daniele Di Proietto <diproiettod at vmware.com>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M lib/dpif-netdev.c
    M lib/netdev-dummy.c
    M tests/bridge.at
    M tests/dpctl.at
    M tests/mpls-xlate.at
    M tests/netdev-type.at
    M tests/ofproto-dpif.at
    M tests/ovn-controller.at
    M tests/ovs-vswitchd.at
    M tests/pmd.at
    M tests/tunnel-push-pop-ipv6.at
    M tests/tunnel-push-pop.at
    M tests/tunnel.at

  Log Message:
  -----------
  netdev-dummy: Add dummy-internal class.

"internal" netdevs are treated specially in OVS (e.g. for MTU), but
the dummy datapath remaps both "system" and "internal" devices to the
same "dummy" netdev class, so there's no way to discern those in tests.

This commit adds a new "dummy-internal" netdev type, which will be used
by the dummy datapath for internal ports, so that other parts of the
code can understand which ports are internal just by looking at the
netdev object.

The alternative solution, using the original interface type ("internal")
instead of the translated netdev type ("dummy"), is harder to implement,
because in so many places only the netdev object is available.

Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: ae59d13433965693f2fd350a28379b690e425981
      https://github.com/openvswitch/ovs/commit/ae59d13433965693f2fd350a28379b690e425981
  Author: Daniele Di Proietto <diproiettod at vmware.com>
  Date:   2016-08-15 (Mon, 15 Aug 2016)

  Changed paths:
    M lib/netdev-dummy.c
    M tests/ofproto-dpif.at

  Log Message:
  -----------
  tests: Add a new MTU test.

Also, netdev-dummy needs to call netdev_change_seq_changed() in
set_mtu().

Signed-off-by: Daniele Di Proietto <diproiettod at vmware.com>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/3262e36909da...ae59d1343396


More information about the git mailing list