[systemd-commits] 2 commits - man/systemd.netdev.xml man/systemd.network.xml src/network

Tom Gundersen tomegun at kemper.freedesktop.org
Thu Dec 4 02:13:44 PST 2014


 man/systemd.netdev.xml                   |   30 ++++++++++
 man/systemd.network.xml                  |   13 ++++
 src/network/networkd-link.c              |   93 +++++++++++++++++++++++++++++++
 src/network/networkd-netdev-gperf.gperf  |   83 ++++++++++++++-------------
 src/network/networkd-netdev-vxlan.c      |   81 +++++++++++++++++++++++++++
 src/network/networkd-netdev-vxlan.h      |   10 +++
 src/network/networkd-network-gperf.gperf |    1 
 src/network/networkd-network.c           |    2 
 src/network/networkd.h                   |   13 ++++
 9 files changed, 286 insertions(+), 40 deletions(-)

New commits:
commit e1853b00ef7cb56cafd908327dd44b3ab48b402c
Author: Susant Sahani <susant at redhat.com>
Date:   Sat Nov 15 08:47:16 2014 +0530

    networkd: Add bridge port path cost
    
    This patch add support to specify path cost of the
    bridge port to be configured via conf file.
    
    Exampe: conf
    
    file: br.netdev
    
    [NetDev]
    Name=br-test
    Kind=bridge
    
    file: br.network
    [Match]
    Name=em1
    
    [Network]
    Bridge=br-test
    
    [BridgePort]
    Cost=332
    
     bridge link
    2: em1 state UP : <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 master
    br-test state disabled priority 32 cost 332

diff --git a/man/systemd.network.xml b/man/systemd.network.xml
index 4cc13b2..c9c946c 100644
--- a/man/systemd.network.xml
+++ b/man/systemd.network.xml
@@ -505,6 +505,19 @@
         </refsect1>
 
         <refsect1>
+                <title>[BridgePort] Section Options</title>
+                        <para>The <literal>[BridgePort]</literal> section accepts the following keys.</para>
+                        <variablelist class='network-directives'>
+                                <varlistentry>
+                                        <term><varname>Cost=</varname></term>
+                                        <listitem>
+                                          <para>Each port in a bridge may have different speed. Cost is used to decide which link to use. Faster interfaces should have lower costs</para>
+                                        </listitem>
+                                </varlistentry>
+                        </variablelist>
+        </refsect1>
+
+        <refsect1>
                 <title>Example</title>
                 <example>
                         <title>/etc/systemd/network/50-static.network</title>
diff --git a/src/network/networkd-link.c b/src/network/networkd-link.c
index 2eb0925..a4f8c59 100644
--- a/src/network/networkd-link.c
+++ b/src/network/networkd-link.c
@@ -685,6 +685,27 @@ int link_address_drop_handler(sd_rtnl *rtnl, sd_rtnl_message *m, void *userdata)
         return 1;
 }
 
+static int link_set_handler(sd_rtnl *rtnl, sd_rtnl_message *m, void *userdata) {
+        _cleanup_link_unref_ Link *link = userdata;
+        int r;
+
+        log_debug_link(link, "set link");
+
+        r = sd_rtnl_message_get_errno(m);
+        if (r < 0 && r != -EEXIST) {
+                log_struct_link(LOG_ERR, link,
+                                "MESSAGE=%-*s: could not join netdev: %s",
+                                IFNAMSIZ,
+                                link->ifname, strerror(-r),
+                                "ERRNO=%d", -r,
+                                NULL);
+                link_enter_failed(link);
+                return 1;
+        }
+
+        return 0;
+}
+
 static int set_hostname_handler(sd_bus *bus, sd_bus_message *m, void *userdata,
                                 sd_bus_error *ret_error) {
         _cleanup_link_unref_ Link *link = userdata;
@@ -802,6 +823,69 @@ int link_set_mtu(Link *link, uint32_t mtu) {
         return 0;
 }
 
+static int link_set_bridge(Link *link) {
+        _cleanup_rtnl_message_unref_ sd_rtnl_message *req = NULL;
+        int r;
+
+        assert(link);
+        assert(link->network);
+
+        if(link->network->cost == 0)
+                return 0;
+
+        r = sd_rtnl_message_new_link(link->manager->rtnl, &req,
+                                     RTM_SETLINK, link->ifindex);
+        if (r < 0) {
+                log_error_link(link, "Could not allocate RTM_SETLINK message");
+                return r;
+        }
+
+        r = sd_rtnl_message_link_set_family(req, PF_BRIDGE);
+        if (r < 0) {
+                log_error_link(link,
+                               "Could not set message family %s", strerror(-r));
+                return r;
+        }
+
+        r = sd_rtnl_message_open_container(req, IFLA_PROTINFO);
+        if (r < 0) {
+                log_error_link(link,
+                               "Could not append IFLA_PROTINFO attribute: %s",
+                               strerror(-r));
+                return r;
+        }
+
+        if(link->network->cost != 0) {
+                r = sd_rtnl_message_append_u32(req, IFLA_BRPORT_COST, link->network->cost);
+                if (r < 0) {
+                        log_error_link(link,
+                                       "Could not append IFLA_BRPORT_COST attribute: %s",
+                                       strerror(-r));
+                        return r;
+                }
+        }
+
+        r = sd_rtnl_message_close_container(req);
+        if (r < 0) {
+                log_error_link(link,
+                               "Could not append IFLA_LINKINFO attribute: %s",
+                               strerror(-r));
+                return r;
+        }
+
+        r = sd_rtnl_call_async(link->manager->rtnl, req, link_set_handler, link, 0, NULL);
+        if (r < 0) {
+                log_error_link(link,
+                               "Could not send rtnetlink message: %s",
+                               strerror(-r));
+                return r;
+        }
+
+        link_ref(link);
+
+        return r;
+}
+
 static void dhcp6_handler(sd_dhcp6_client *client, int event, void *userdata) {
         Link *link = userdata;
 
@@ -1037,6 +1121,15 @@ static int link_joined(Link *link) {
                 }
         }
 
+        if(link->network->bridge) {
+                r = link_set_bridge(link);
+                if (r < 0) {
+                        log_error_link(link,
+                                       "Could not set bridge message: %s",
+                                       strerror(-r));
+                }
+        }
+
         return link_enter_set_addresses(link);
 }
 
diff --git a/src/network/networkd-network-gperf.gperf b/src/network/networkd-network-gperf.gperf
index a736461..1aef090 100644
--- a/src/network/networkd-network-gperf.gperf
+++ b/src/network/networkd-network-gperf.gperf
@@ -58,6 +58,7 @@ DHCP.RequestBroadcast,       config_parse_bool,                  0,
 DHCP.CriticalConnection,     config_parse_bool,                  0,                             offsetof(Network, dhcp_critical)
 DHCP.VendorClassIdentifier,  config_parse_string,                0,                             offsetof(Network, dhcp_vendor_class_identifier)
 DHCP.RouteMetric,            config_parse_unsigned,              0,                             offsetof(Network, dhcp_route_metric)
+BridgePort.Cost,             config_parse_unsigned,              0,                             offsetof(Network, cost)
 /* backwards compatibility: do not add new entries to this section */
 DHCPv4.UseDNS,               config_parse_bool,                  0,                             offsetof(Network, dhcp_dns)
 DHCPv4.UseMTU,               config_parse_bool,                  0,                             offsetof(Network, dhcp_mtu)
diff --git a/src/network/networkd-network.c b/src/network/networkd-network.c
index e4bb1b0..6cfae75 100644
--- a/src/network/networkd-network.c
+++ b/src/network/networkd-network.c
@@ -90,7 +90,7 @@ static int network_load_one(Manager *manager, const char *filename) {
         network->llmnr = LLMNR_SUPPORT_YES;
 
         r = config_parse(NULL, filename, file,
-                         "Match\0Network\0Address\0Route\0DHCP\0DHCPv4\0",
+                         "Match\0Network\0Address\0Route\0DHCP\0DHCPv4\0BridgePort\0",
                          config_item_perf_lookup, network_network_gperf_lookup,
                          false, false, true, network);
         if (r < 0)
diff --git a/src/network/networkd.h b/src/network/networkd.h
index c0d32c4..1297ef9 100644
--- a/src/network/networkd.h
+++ b/src/network/networkd.h
@@ -106,6 +106,8 @@ struct Network {
 
         bool dhcp_server;
 
+        unsigned cost;
+
         LIST_HEAD(Address, static_addresses);
         LIST_HEAD(Route, static_routes);
 

commit 85a8eeee36b57c1ab382b0225fa9a87525bbeee9
Author: Susant Sahani <susant at redhat.com>
Date:   Sat Nov 15 08:24:59 2014 +0530

    networkd: support vxlan parameters
    
    V3: fix copy paste error
    V4: Make manual and config more readable
    
    Add vxlan paramertes to config.

diff --git a/man/systemd.netdev.xml b/man/systemd.netdev.xml
index 275ee52..45934f2 100644
--- a/man/systemd.netdev.xml
+++ b/man/systemd.netdev.xml
@@ -272,6 +272,36 @@
                                                 to discover remote MAC addresses.</para>
                                         </listitem>
                                 </varlistentry>
+                                <varlistentry>
+                                        <term><varname>FDBAgeingSec=</varname></term>
+                                        <listitem>
+                                                <para>The lifetime of Forwarding Database entry learnt by the kernel in seconds.</para>
+                                        </listitem>
+                                </varlistentry>
+                                <varlistentry>
+                                        <term><varname>ARPProxy=</varname></term>
+                                        <listitem>
+                                                <para>A boolean. When true, enables ARP proxy.</para>
+                                        </listitem>
+                                </varlistentry>
+                                <varlistentry>
+                                        <term><varname>L2MissNotification=</varname></term>
+                                        <listitem>
+                                                <para>A boolean. When true, enables netlink LLADDR miss notifications.</para>
+                                        </listitem>
+                                </varlistentry>
+                                <varlistentry>
+                                        <term><varname>L3MissNotification=</varname></term>
+                                        <listitem>
+                                                <para>A boolean. When true, enables netlink IP ADDR miss notifications.</para>
+                                        </listitem>
+                                </varlistentry>
+                                <varlistentry>
+                                        <term><varname>RouteShortCircuit=</varname></term>
+                                        <listitem>
+                                                <para>A boolean. When true route short circuit is turned on.</para>
+                                        </listitem>
+                                </varlistentry>
                         </variablelist>
         </refsect1>
         <refsect1>
diff --git a/src/network/networkd-netdev-gperf.gperf b/src/network/networkd-netdev-gperf.gperf
index c524ee5..b311ebe 100644
--- a/src/network/networkd-netdev-gperf.gperf
+++ b/src/network/networkd-netdev-gperf.gperf
@@ -18,42 +18,47 @@ struct ConfigPerfItem;
 %struct-type
 %includes
 %%
-Match.Host,              config_parse_net_condition,         CONDITION_HOST,                offsetof(NetDev, match_host)
-Match.Virtualization,    config_parse_net_condition,         CONDITION_VIRTUALIZATION,      offsetof(NetDev, match_virt)
-Match.KernelCommandLine, config_parse_net_condition,         CONDITION_KERNEL_COMMAND_LINE, offsetof(NetDev, match_kernel)
-Match.Architecture,      config_parse_net_condition,         CONDITION_ARCHITECTURE,        offsetof(NetDev, match_arch)
-NetDev.Description,      config_parse_string,                0,                             offsetof(NetDev, description)
-NetDev.Name,             config_parse_ifname,                0,                             offsetof(NetDev, ifname)
-NetDev.Kind,             config_parse_netdev_kind,           0,                             offsetof(NetDev, kind)
-NetDev.MTUBytes,         config_parse_iec_size,              0,                             offsetof(NetDev, mtu)
-NetDev.MACAddress,       config_parse_hwaddr,                0,                             offsetof(NetDev, mac)
-VLAN.Id,                 config_parse_uint64,                0,                             offsetof(VLan, id)
-MACVLAN.Mode,            config_parse_macvlan_mode,          0,                             offsetof(MacVlan, mode)
-Tunnel.Local,            config_parse_tunnel_address,        0,                             offsetof(Tunnel, local)
-Tunnel.Remote,           config_parse_tunnel_address,        0,                             offsetof(Tunnel, remote)
-Tunnel.TOS,              config_parse_unsigned,              0,                             offsetof(Tunnel, tos)
-Tunnel.TTL,              config_parse_unsigned,              0,                             offsetof(Tunnel, ttl)
-Tunnel.DiscoverPathMTU,  config_parse_bool,                  0,                             offsetof(Tunnel, pmtudisc)
-Peer.Name,               config_parse_ifname,                0,                             offsetof(Veth, ifname_peer)
-Peer.MACAddress,         config_parse_hwaddr,                0,                             offsetof(Veth, mac_peer)
-VXLAN.Id,                config_parse_uint64,                0,                             offsetof(VxLan, id)
-VXLAN.Group,             config_parse_tunnel_address,        0,                             offsetof(VxLan, group)
-VXLAN.TOS,               config_parse_unsigned,              0,                             offsetof(VxLan, tos)
-VXLAN.TTL,               config_parse_unsigned,              0,                             offsetof(VxLan, ttl)
-VXLAN.MacLearning,       config_parse_bool,                  0,                             offsetof(VxLan, learning)
-Tun.OneQueue,            config_parse_bool,                  0,                             offsetof(TunTap, one_queue)
-Tun.MultiQueue,          config_parse_bool,                  0,                             offsetof(TunTap, multi_queue)
-Tun.PacketInfo,          config_parse_bool,                  0,                             offsetof(TunTap, packet_info)
-Tun.User,                config_parse_string,                0,                             offsetof(TunTap, user_name)
-Tun.Group,               config_parse_string,                0,                             offsetof(TunTap, group_name)
-Tap.OneQueue,            config_parse_bool,                  0,                             offsetof(TunTap, one_queue)
-Tap.MultiQueue,          config_parse_bool,                  0,                             offsetof(TunTap, multi_queue)
-Tap.PacketInfo,          config_parse_bool,                  0,                             offsetof(TunTap, packet_info)
-Tap.User,                config_parse_string,                0,                             offsetof(TunTap, user_name)
-Tap.Group,               config_parse_string,                0,                             offsetof(TunTap, group_name)
-Bond.Mode,               config_parse_bond_mode,             0,                             offsetof(Bond, mode)
-Bond.TransmitHashPolicy, config_parse_bond_xmit_hash_policy, 0,                             offsetof(Bond, xmit_hash_policy)
-Bond.LACPTransmitRate,   config_parse_bond_lacp_rate,        0,                             offsetof(Bond, lacp_rate)
-Bond.MIIMonitorSec,      config_parse_sec,                   0,                             offsetof(Bond, miimon)
-Bond.UpDelaySec,         config_parse_sec,                   0,                             offsetof(Bond, updelay)
-Bond.DownDelaySec,       config_parse_sec,                   0,                             offsetof(Bond, downdelay)
+Match.Host,               config_parse_net_condition,         CONDITION_HOST,                offsetof(NetDev, match_host)
+Match.Virtualization,     config_parse_net_condition,         CONDITION_VIRTUALIZATION,      offsetof(NetDev, match_virt)
+Match.KernelCommandLine,  config_parse_net_condition,         CONDITION_KERNEL_COMMAND_LINE, offsetof(NetDev, match_kernel)
+Match.Architecture,       config_parse_net_condition,         CONDITION_ARCHITECTURE,        offsetof(NetDev, match_arch)
+NetDev.Description,       config_parse_string,                0,                             offsetof(NetDev, description)
+NetDev.Name,              config_parse_ifname,                0,                             offsetof(NetDev, ifname)
+NetDev.Kind,              config_parse_netdev_kind,           0,                             offsetof(NetDev, kind)
+NetDev.MTUBytes,          config_parse_iec_size,              0,                             offsetof(NetDev, mtu)
+NetDev.MACAddress,        config_parse_hwaddr,                0,                             offsetof(NetDev, mac)
+VLAN.Id,                  config_parse_uint64,                0,                             offsetof(VLan, id)
+MACVLAN.Mode,             config_parse_macvlan_mode,          0,                             offsetof(MacVlan, mode)
+Tunnel.Local,             config_parse_tunnel_address,        0,                             offsetof(Tunnel, local)
+Tunnel.Remote,            config_parse_tunnel_address,        0,                             offsetof(Tunnel, remote)
+Tunnel.TOS,               config_parse_unsigned,              0,                             offsetof(Tunnel, tos)
+Tunnel.TTL,               config_parse_unsigned,              0,                             offsetof(Tunnel, ttl)
+Tunnel.DiscoverPathMTU,   config_parse_bool,                  0,                             offsetof(Tunnel, pmtudisc)
+Peer.Name,                config_parse_ifname,                0,                             offsetof(Veth, ifname_peer)
+Peer.MACAddress,          config_parse_hwaddr,                0,                             offsetof(Veth, mac_peer)
+VXLAN.Id,                 config_parse_uint64,                0,                             offsetof(VxLan, id)
+VXLAN.Group,              config_parse_vxlan_group_address,   0,                             offsetof(VxLan, group)
+VXLAN.TOS,                config_parse_unsigned,              0,                             offsetof(VxLan, tos)
+VXLAN.TTL,                config_parse_unsigned,              0,                             offsetof(VxLan, ttl)
+VXLAN.MacLearning,        config_parse_bool,                  0,                             offsetof(VxLan, learning)
+VXLAN.ARPProxy,           config_parse_bool,                  0,                             offsetof(VxLan, arp_proxy)
+VXLAN.L2MissNotification, config_parse_bool,                  0,                             offsetof(VxLan, l2miss)
+VXLAN.L3MissNotification, config_parse_bool,                  0,                             offsetof(VxLan, l3miss)
+VXLAN.RouteShortCircuit,  config_parse_bool,                  0,                             offsetof(VxLan, route_short_circuit)
+VXLAN.FDBAgeingSec,       config_parse_sec,                   0,                             offsetof(VxLan, fdb_ageing)
+Tun.OneQueue,             config_parse_bool,                  0,                             offsetof(TunTap, one_queue)
+Tun.MultiQueue,           config_parse_bool,                  0,                             offsetof(TunTap, multi_queue)
+Tun.PacketInfo,           config_parse_bool,                  0,                             offsetof(TunTap, packet_info)
+Tun.User,                 config_parse_string,                0,                             offsetof(TunTap, user_name)
+Tun.Group,                config_parse_string,                0,                             offsetof(TunTap, group_name)
+Tap.OneQueue,             config_parse_bool,                  0,                             offsetof(TunTap, one_queue)
+Tap.MultiQueue,           config_parse_bool,                  0,                             offsetof(TunTap, multi_queue)
+Tap.PacketInfo,           config_parse_bool,                  0,                             offsetof(TunTap, packet_info)
+Tap.User,                 config_parse_string,                0,                             offsetof(TunTap, user_name)
+Tap.Group,                config_parse_string,                0,                             offsetof(TunTap, group_name)
+Bond.Mode,                config_parse_bond_mode,             0,                             offsetof(Bond, mode)
+Bond.TransmitHashPolicy,  config_parse_bond_xmit_hash_policy, 0,                             offsetof(Bond, xmit_hash_policy)
+Bond.LACPTransmitRate,    config_parse_bond_lacp_rate,        0,                             offsetof(Bond, lacp_rate)
+Bond.MIIMonitorSec,       config_parse_sec,                   0,                             offsetof(Bond, miimon)
+Bond.UpDelaySec,          config_parse_sec,                   0,                             offsetof(Bond, updelay)
+Bond.DownDelaySec,        config_parse_sec,                   0,                             offsetof(Bond, downdelay)
diff --git a/src/network/networkd-netdev-vxlan.c b/src/network/networkd-netdev-vxlan.c
index 7dec69d..b19240e 100644
--- a/src/network/networkd-netdev-vxlan.c
+++ b/src/network/networkd-netdev-vxlan.c
@@ -26,6 +26,7 @@
 #include "sd-rtnl.h"
 #include "networkd-netdev-vxlan.h"
 #include "networkd-link.h"
+#include "conf-parser.h"
 #include "missing.h"
 
 static int netdev_vxlan_fill_message_create(NetDev *netdev, Link *link, sd_rtnl_message *m) {
@@ -92,9 +93,89 @@ static int netdev_vxlan_fill_message_create(NetDev *netdev, Link *link, sd_rtnl_
                 return r;
         }
 
+        r = sd_rtnl_message_append_u8(m, IFLA_VXLAN_RSC, v->route_short_circuit);
+        if (r < 0) {
+                log_error_netdev(netdev,
+                                 "Could not append IFLA_VXLAN_RSC attribute: %s",
+                                 strerror(-r));
+                return r;
+        }
+
+        r = sd_rtnl_message_append_u8(m, IFLA_VXLAN_PROXY, v->arp_proxy);
+        if (r < 0) {
+                log_error_netdev(netdev,
+                                 "Could not append IFLA_VXLAN_PROXY attribute: %s",
+                                 strerror(-r));
+                return r;
+        }
+
+        r = sd_rtnl_message_append_u8(m, IFLA_VXLAN_L2MISS, v->l2miss);
+        if (r < 0) {
+                log_error_netdev(netdev,
+                                 "Could not append IFLA_VXLAN_L2MISS attribute: %s",
+                                 strerror(-r));
+                return r;
+        }
+
+        r = sd_rtnl_message_append_u8(m, IFLA_VXLAN_L3MISS, v->l3miss);
+        if (r < 0) {
+                log_error_netdev(netdev,
+                                 "Could not append IFLA_VXLAN_L3MISS attribute: %s",
+                                 strerror(-r));
+                return r;
+        }
+
+        if(v->fdb_ageing) {
+                r = sd_rtnl_message_append_u32(m, IFLA_VXLAN_AGEING, v->fdb_ageing / USEC_PER_SEC);
+                if (r < 0) {
+                        log_error_netdev(netdev,
+                                         "Could not append IFLA_VXLAN_AGEING attribute: %s",
+                                         strerror(-r));
+                        return r;
+                }
+        }
+
         return r;
 }
 
+int config_parse_vxlan_group_address(const char *unit,
+                                     const char *filename,
+                                     unsigned line,
+                                     const char *section,
+                                     unsigned section_line,
+                                     const char *lvalue,
+                                     int ltype,
+                                     const char *rvalue,
+                                     void *data,
+                                     void *userdata) {
+        VxLan *v = userdata;
+        union in_addr_union *addr = data, buffer;
+        int r, f;
+
+        assert(filename);
+        assert(lvalue);
+        assert(rvalue);
+        assert(data);
+
+        r = in_addr_from_string_auto(rvalue, &f, &buffer);
+        if (r < 0) {
+                log_syntax(unit, LOG_ERR, filename, line, EINVAL,
+                           "vxlan multicast group address is invalid, ignoring assignment: %s", rvalue);
+                return 0;
+        }
+
+        if(v->family != AF_UNSPEC && v->family != f) {
+                log_syntax(unit, LOG_ERR, filename, line, EINVAL,
+                           "vxlan multicast group incompatible, ignoring assignment: %s", rvalue);
+                return 0;
+        }
+
+        v->family = f;
+        *addr = buffer;
+
+        return 0;
+}
+
 static int netdev_vxlan_verify(NetDev *netdev, const char *filename) {
         VxLan *v = VXLAN(netdev);
 
diff --git a/src/network/networkd-netdev-vxlan.h b/src/network/networkd-netdev-vxlan.h
index 8c906f1..6339af9 100644
--- a/src/network/networkd-netdev-vxlan.h
+++ b/src/network/networkd-netdev-vxlan.h
@@ -33,10 +33,20 @@ struct VxLan {
         NetDev meta;
 
         uint64_t id;
+
+        int family;
         union in_addr_union group;
+
         unsigned tos;
         unsigned ttl;
+
+        usec_t fdb_ageing;
+
         bool learning;
+        bool arp_proxy;
+        bool route_short_circuit;
+        bool l2miss;
+        bool l3miss;
 };
 
 extern const NetDevVTable vxlan_vtable;
diff --git a/src/network/networkd.h b/src/network/networkd.h
index 1f31023..c0d32c4 100644
--- a/src/network/networkd.h
+++ b/src/network/networkd.h
@@ -259,6 +259,17 @@ int config_parse_tunnel_address(const char *unit,
                                 void *data,
                                 void *userdata);
 
+int config_parse_vxlan_group_address(const char *unit,
+                                     const char *filename,
+                                     unsigned line,
+                                     const char *section,
+                                     unsigned section_line,
+                                     const char *lvalue,
+                                     int ltype,
+                                     const char *rvalue,
+                                     void *data,
+                                     void *userdata);
+
 /* gperf */
 const struct ConfigPerfItem* network_network_gperf_lookup(const char *key, unsigned length);
 



More information about the systemd-commits mailing list