treewide: correctly apply IFNAMSIZ limit
[project/netifd.git] / system-linux.c
index c0ba2794415f21ab4de42410a6ecaa368fb07826..9fe869fd8d151d6df19bf725a9e616bde4276dcf 100644 (file)
@@ -89,9 +89,10 @@ static int cb_rtnl_event(struct nl_msg *msg, void *arg);
 static void handle_hotplug_event(struct uloop_fd *u, unsigned int events);
 static int system_add_proto_tunnel(const char *name, const uint8_t proto,
                                        const unsigned int link, struct blob_attr **tb);
-static int __system_del_ip_tunnel(const char *name, struct blob_attr **tb);
 
 static char dev_buf[256];
+static const char *proc_path = "/proc";
+static const char *sysfs_path = "/sys";
 
 static void
 handler_nl_event(struct uloop_fd *u, unsigned int events)
@@ -281,7 +282,7 @@ int system_init(void)
        return 0;
 }
 
-static void system_set_sysctl(const char *path, const char *val)
+static void write_file(const char *path, const char *val)
 {
        int fd;
 
@@ -293,352 +294,330 @@ static void system_set_sysctl(const char *path, const char *val)
        close(fd);
 }
 
-static void system_set_dev_sysctl(const char *path, const char *device, const char *val)
+static int read_file(const char *path, char *buf, const size_t buf_sz)
 {
-       snprintf(dev_buf, sizeof(dev_buf), path, device);
-       system_set_sysctl(dev_buf, val);
-}
+       int fd = -1, ret = -1;
 
-static void system_set_disable_ipv6(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/disable_ipv6", dev->ifname, val);
-}
+       fd = open(path, O_RDONLY);
+       if (fd < 0)
+               goto out;
 
-static void system_set_ip6segmentrouting(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/seg6_enabled", dev->ifname, val);
-}
+       ssize_t len = read(fd, buf, buf_sz - 1);
+       if (len < 0)
+               goto out;
 
-static void system_set_rpfilter(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/rp_filter", dev->ifname, val);
-}
+       ret = buf[len] = 0;
 
-static void system_set_acceptlocal(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/accept_local", dev->ifname, val);
-}
+out:
+       if (fd >= 0)
+               close(fd);
 
-static void system_set_igmpversion(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/force_igmp_version", dev->ifname, val);
+       return ret;
 }
 
-static void system_set_mldversion(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/force_mld_version", dev->ifname, val);
-}
 
-static void system_set_neigh4reachabletime(struct device *dev, const char *val)
+static const char *
+dev_sysctl_path(const char *prefix, const char *ifname, const char *file)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/base_reachable_time_ms", dev->ifname, val);
-}
+       snprintf(dev_buf, sizeof(dev_buf), "%s/sys/net/%s/%s/%s", proc_path, prefix, ifname, file);
 
-static void system_set_neigh6reachabletime(struct device *dev, const char *val)
-{
-       system_set_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/base_reachable_time_ms", dev->ifname, val);
+       return dev_buf;
 }
 
-static void system_set_neigh4gcstaletime(struct device *dev, const char *val)
+static const char *
+dev_sysfs_path(const char *ifname, const char *file)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/gc_stale_time", dev->ifname, val);
+       snprintf(dev_buf, sizeof(dev_buf), "%s/class/net/%s/%s", sysfs_path, ifname, file);
+
+       return dev_buf;
 }
 
-static void system_set_neigh6gcstaletime(struct device *dev, const char *val)
+static void
+system_set_dev_sysctl(const char *prefix, const char *file, const char *ifname,
+                     const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/gc_stale_time", dev->ifname, val);
+       write_file(dev_sysctl_path(prefix, ifname, file), val);
 }
 
-static void system_set_neigh4locktime(struct device *dev, const char *val)
+static int
+system_get_dev_sysctl(const char *prefix, const char *file, const char *ifname,
+                     char *buf, size_t buf_sz)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/locktime", dev->ifname, val);
+       return read_file(dev_sysctl_path(prefix, ifname, file), buf, buf_sz);
 }
 
-static void system_set_dadtransmits(struct device *dev, const char *val)
+static void
+system_set_dev_sysfs(const char *file, const char *ifname, const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/dad_transmits", dev->ifname, val);
+       if (!val)
+               return;
+
+       write_file(dev_sysfs_path(ifname, file), val);
 }
 
-static void system_set_sendredirects(struct device *dev, const char *val)
+static void
+system_set_dev_sysfs_int(const char *file, const char *ifname, int val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/send_redirects", dev->ifname, val);
+       char buf[16];
+
+       snprintf(buf, sizeof(buf), "%d", val);
+       system_set_dev_sysfs(file, ifname, buf);
 }
 
-static void system_set_drop_v4_unicast_in_l2_multicast(struct device *dev, const char *val)
+static int
+system_get_dev_sysfs(const char *file, const char *ifname, char *buf, size_t buf_sz)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/drop_unicast_in_l2_multicast", dev->ifname, val);
+       return read_file(dev_sysfs_path(ifname, file), buf, buf_sz);
 }
 
-static void system_set_drop_v6_unicast_in_l2_multicast(struct device *dev, const char *val)
+static void system_set_disable_ipv6(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/drop_unicast_in_l2_multicast", dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "disable_ipv6", dev->ifname, val);
 }
 
-static void system_set_drop_gratuitous_arp(struct device *dev, const char *val)
+static void system_set_ip6segmentrouting(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/drop_gratuitous_arp", dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "seg6_enabled", dev->ifname, val);
 }
 
-static void system_set_drop_unsolicited_na(struct device *dev, const char *val)
+static void system_set_rpfilter(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv6/conf/%s/drop_unsolicited_na", dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "rp_filter", dev->ifname, val);
 }
 
-static void system_set_arp_accept(struct device *dev, const char *val)
+static void system_set_acceptlocal(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/proc/sys/net/ipv4/conf/%s/arp_accept", dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "accept_local", dev->ifname, val);
 }
 
-static void system_bridge_set_multicast_to_unicast(struct device *dev, const char *val)
+static void system_set_igmpversion(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/multicast_to_unicast", dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "force_igmp_version", dev->ifname, val);
 }
 
-static void system_bridge_set_multicast_fast_leave(struct device *dev, const char *val)
+static void system_set_mldversion(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/multicast_fast_leave", dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "force_mld_version", dev->ifname, val);
 }
 
-static void system_bridge_set_hairpin_mode(struct device *dev, const char *val)
+static void system_set_neigh4reachabletime(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/hairpin_mode", dev->ifname, val);
+       system_set_dev_sysctl("ipv4/neigh", "base_reachable_time_ms", dev->ifname, val);
 }
 
-static void system_bridge_set_isolated(struct device *dev, const char *val)
+static void system_set_neigh6reachabletime(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/isolated", dev->ifname, val);
+       system_set_dev_sysctl("ipv6/neigh", "base_reachable_time_ms", dev->ifname, val);
 }
 
-static void system_bridge_set_multicast_router(struct device *dev, const char *val, bool bridge)
+static void system_set_neigh4gcstaletime(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl(bridge ? "/sys/class/net/%s/bridge/multicast_router" :
-                                      "/sys/class/net/%s/brport/multicast_router",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv4/neigh", "gc_stale_time", dev->ifname, val);
 }
 
-static void system_bridge_set_robustness(struct device *dev, const char *val)
+static void system_set_neigh6gcstaletime(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_startup_query_count",
-                             dev->ifname, val);
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_last_member_count",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv6/neigh", "gc_stale_time", dev->ifname, val);
 }
 
-static void system_bridge_set_query_interval(struct device *dev, const char *val)
+static void system_set_neigh4locktime(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_query_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv4/neigh", "locktime", dev->ifname, val);
 }
 
-static void system_bridge_set_query_response_interval(struct device *dev, const char *val)
+static void system_set_dadtransmits(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_query_response_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "dad_transmits", dev->ifname, val);
 }
 
-static void system_bridge_set_last_member_interval(struct device *dev, const char *val)
+static void system_set_sendredirects(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_last_member_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "send_redirects", dev->ifname, val);
 }
 
-static void system_bridge_set_membership_interval(struct device *dev, const char *val)
+static void system_set_drop_v4_unicast_in_l2_multicast(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_membership_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "drop_unicast_in_l2_multicast", dev->ifname, val);
 }
 
-static void system_bridge_set_other_querier_timeout(struct device *dev, const char *val)
+static void system_set_drop_v6_unicast_in_l2_multicast(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_querier_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "drop_unicast_in_l2_multicast", dev->ifname, val);
 }
 
-static void system_bridge_set_startup_query_interval(struct device *dev, const char *val)
+static void system_set_drop_gratuitous_arp(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_startup_query_interval",
-                             dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "drop_gratuitous_arp", dev->ifname, val);
 }
 
-static void system_bridge_set_stp_state(struct device *dev, const char *val)
+static void system_set_drop_unsolicited_na(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/stp_state", dev->ifname, val);
+       system_set_dev_sysctl("ipv6/conf", "drop_unsolicited_na", dev->ifname, val);
 }
 
-static void system_bridge_set_forward_delay(struct device *dev, const char *val)
+static void system_set_arp_accept(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/forward_delay", dev->ifname, val);
+       system_set_dev_sysctl("ipv4/conf", "arp_accept", dev->ifname, val);
 }
 
-static void system_bridge_set_priority(struct device *dev, const char *val)
+static void system_bridge_set_multicast_to_unicast(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/priority", dev->ifname, val);
+       system_set_dev_sysfs("brport/multicast_to_unicast", dev->ifname, val);
 }
 
-static void system_bridge_set_ageing_time(struct device *dev, const char *val)
+static void system_bridge_set_multicast_fast_leave(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/ageing_time", dev->ifname, val);
+       system_set_dev_sysfs("brport/multicast_fast_leave", dev->ifname, val);
 }
 
-static void system_bridge_set_hello_time(struct device *dev, const char *val)
+static void system_bridge_set_hairpin_mode(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/hello_time", dev->ifname, val);
+       system_set_dev_sysfs("brport/hairpin_mode", dev->ifname, val);
 }
 
-static void system_bridge_set_max_age(struct device *dev, const char *val)
+static void system_bridge_set_proxyarp_wifi(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/max_age", dev->ifname, val);
+       system_set_dev_sysfs("brport/proxyarp_wifi", dev->ifname, val);
 }
 
-static void system_bridge_set_learning(struct device *dev, const char *val)
+static void system_bridge_set_bpdu_filter(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/learning", dev->ifname, val);
+       system_set_dev_sysfs("brport/bpdu_filter", dev->ifname, val);
 }
 
-static void system_bridge_set_unicast_flood(struct device *dev, const char *val)
+static void system_bridge_set_isolated(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/class/net/%s/brport/unicast_flood", dev->ifname, val);
+       system_set_dev_sysfs("brport/isolated", dev->ifname, val);
 }
 
-static void system_bridge_set_vlan_filtering(struct device *dev, const char *val)
+static void system_bridge_set_multicast_router(struct device *dev, const char *val)
 {
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/vlan_filtering", dev->ifname, val);
+       system_set_dev_sysfs("brport/multicast_router", dev->ifname, val);
 }
 
-static int system_get_sysctl(const char *path, char *buf, const size_t buf_sz)
+void system_bridge_set_stp_state(struct device *dev, bool val)
 {
-       int fd = -1, ret = -1;
+       const char *valstr = val ? "1" : "0";
 
-       fd = open(path, O_RDONLY);
-       if (fd < 0)
-               goto out;
-
-       ssize_t len = read(fd, buf, buf_sz - 1);
-       if (len < 0)
-               goto out;
-
-       ret = buf[len] = 0;
-
-out:
-       if (fd >= 0)
-               close(fd);
+       system_set_dev_sysfs("bridge/stp_state", dev->ifname, valstr);
+}
 
-       return ret;
+static void system_bridge_set_learning(struct device *dev, const char *val)
+{
+       system_set_dev_sysfs("brport/learning", dev->ifname, val);
 }
 
-static int
-system_get_dev_sysctl(const char *path, const char *device, char *buf, const size_t buf_sz)
+static void system_bridge_set_unicast_flood(struct device *dev, const char *val)
 {
-       snprintf(dev_buf, sizeof(dev_buf), path, device);
-       return system_get_sysctl(dev_buf, buf, buf_sz);
+       system_set_dev_sysfs("brport/unicast_flood", dev->ifname, val);
 }
 
 static int system_get_disable_ipv6(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/disable_ipv6",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv6/conf", "disable_ipv6",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_ip6segmentrouting(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/seg6_enabled",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv6/conf", "seg6_enabled",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_rpfilter(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/rp_filter",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv4/conf", "rp_filter",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_acceptlocal(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/accept_local",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv4/conf", "accept_local",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_igmpversion(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/force_igmp_version",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv4/conf", "force_igmp_version",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_mldversion(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/force_mld_version",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv6/conf", "force_mld_version",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_neigh4reachabletime(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/base_reachable_time_ms",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv4/neigh", "base_reachable_time_ms",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_neigh6reachabletime(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/base_reachable_time_ms",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv6/neigh", "base_reachable_time_ms",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_neigh4gcstaletime(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/gc_stale_time",
-                       dev->ifname, buf, buf_sz);
+       return system_get_dev_sysctl("ipv4/neigh", "gc_stale_time",
+                                    dev->ifname, buf, buf_sz);
 }
 
 static int system_get_neigh6gcstaletime(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/neigh/%s/gc_stale_time",
+       return system_get_dev_sysctl("ipv6/neigh", "gc_stale_time",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_neigh4locktime(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/neigh/%s/locktime",
+       return system_get_dev_sysctl("ipv4/neigh", "locktime",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_dadtransmits(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/dad_transmits",
+       return system_get_dev_sysctl("ipv6/conf", "dad_transmits",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_sendredirects(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/send_redirects",
+       return system_get_dev_sysctl("ipv4/conf", "send_redirects",
                        dev->ifname, buf, buf_sz);
 }
 
 
 static int system_get_drop_v4_unicast_in_l2_multicast(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/drop_unicast_in_l2_multicast",
+       return system_get_dev_sysctl("ipv4/conf", "drop_unicast_in_l2_multicast",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_drop_v6_unicast_in_l2_multicast(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/drop_unicast_in_l2_multicast",
+       return system_get_dev_sysctl("ipv6/conf", "drop_unicast_in_l2_multicast",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_drop_gratuitous_arp(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/drop_gratuitous_arp",
+       return system_get_dev_sysctl("ipv4/conf", "drop_gratuitous_arp",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_drop_unsolicited_na(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv6/conf/%s/drop_unsolicited_na",
+       return system_get_dev_sysctl("ipv6/conf", "drop_unsolicited_na",
                        dev->ifname, buf, buf_sz);
 }
 
 static int system_get_arp_accept(struct device *dev, char *buf, const size_t buf_sz)
 {
-       return system_get_dev_sysctl("/proc/sys/net/ipv4/conf/%s/arp_accept",
+       return system_get_dev_sysctl("ipv4/conf", "arp_accept",
                        dev->ifname, buf, buf_sz);
 }
 
@@ -661,7 +640,7 @@ static int cb_rtnl_event(struct nl_msg *msg, void *arg)
        if (!dev)
                goto out;
 
-       if (!system_get_dev_sysctl("/sys/class/net/%s/carrier", dev->ifname, buf, sizeof(buf)))
+       if (!system_get_dev_sysfs("carrier", dev->ifname, buf, sizeof(buf)))
                link_state = strtoul(buf, NULL, 0);
 
        if (dev->type == &simple_device_type)
@@ -776,9 +755,44 @@ static int system_rtnl_call(struct nl_msg *msg)
        return nl_wait_for_ack(sock_rtnl);
 }
 
+static struct nl_msg *__system_ifinfo_msg(int af, int index, const char *ifname, uint16_t type, uint16_t flags)
+{
+       struct nl_msg *msg;
+       struct ifinfomsg iim = {
+               .ifi_family = af,
+               .ifi_index = index,
+       };
+
+       msg = nlmsg_alloc_simple(type, flags | NLM_F_REQUEST);
+       if (!msg)
+               return NULL;
+
+       nlmsg_append(msg, &iim, sizeof(iim), 0);
+       if (ifname)
+               nla_put_string(msg, IFLA_IFNAME, ifname);
+
+       return msg;
+}
+
+static struct nl_msg *system_ifinfo_msg(const char *ifname, uint16_t type, uint16_t flags)
+{
+       return __system_ifinfo_msg(AF_UNSPEC, 0, ifname, type, flags);
+}
+
+static int system_link_del(const char *ifname)
+{
+       struct nl_msg *msg;
+
+       msg = system_ifinfo_msg(ifname, RTM_DELLINK, 0);
+       if (!msg)
+               return -1;
+
+       return system_rtnl_call(msg);
+}
+
 int system_bridge_delbr(struct device *bridge)
 {
-       return ioctl(sock_ioctl, SIOCBRDELBR, bridge->ifname);
+       return system_link_del(bridge->ifname);
 }
 
 static int system_bridge_if(const char *bridge, struct device *dev, int cmd, void *data)
@@ -794,15 +808,11 @@ static int system_bridge_if(const char *bridge, struct device *dev, int cmd, voi
        return ioctl(sock_ioctl, cmd, &ifr);
 }
 
-static bool system_is_bridge(const char *name, char *buf, int buflen)
+static bool system_is_bridge(const char *name)
 {
        struct stat st;
 
-       snprintf(buf, buflen, "/sys/devices/virtual/net/%s/bridge", name);
-       if (stat(buf, &st) < 0)
-               return false;
-
-       return true;
+       return stat(dev_sysfs_path(name, "bridge"), &st) >= 0;
 }
 
 static char *system_get_bridge(const char *name, char *buf, int buflen)
@@ -811,7 +821,7 @@ static char *system_get_bridge(const char *name, char *buf, int buflen)
        ssize_t len = -1;
        glob_t gl;
 
-       snprintf(buf, buflen, "/sys/devices/virtual/net/*/brif/%s/bridge", name);
+       snprintf(buf, buflen, "%s/devices/virtual/net/*/brif/%s/bridge", sysfs_path, name);
        if (glob(buf, GLOB_NOSORT, NULL, &gl) < 0)
                return NULL;
 
@@ -835,17 +845,19 @@ static void
 system_bridge_set_wireless(struct device *bridge, struct device *dev)
 {
        bool mcast_to_ucast = dev->wireless_ap;
-       bool hairpin = true;
+       bool hairpin;
 
        if (bridge->settings.flags & DEV_OPT_MULTICAST_TO_UNICAST &&
            !bridge->settings.multicast_to_unicast)
                mcast_to_ucast = false;
 
-       if (!mcast_to_ucast || dev->wireless_isolate)
+       hairpin = mcast_to_ucast || dev->wireless_proxyarp;
+       if (dev->wireless_isolate)
                hairpin = false;
 
        system_bridge_set_multicast_to_unicast(dev, mcast_to_ucast ? "1" : "0");
        system_bridge_set_hairpin_mode(dev, hairpin ? "1" : "0");
+       system_bridge_set_proxyarp_wifi(dev, dev->wireless_proxyarp ? "1" : "0");
 }
 
 int system_bridge_addif(struct device *bridge, struct device *dev)
@@ -872,7 +884,7 @@ retry:
 
        if (dev->settings.flags & DEV_OPT_MULTICAST_ROUTER) {
                snprintf(buf, sizeof(buf), "%u", dev->settings.multicast_router);
-               system_bridge_set_multicast_router(dev, buf, false);
+               system_bridge_set_multicast_router(dev, buf);
        }
 
        if (dev->settings.flags & DEV_OPT_MULTICAST_FAST_LEAVE &&
@@ -891,6 +903,9 @@ retry:
            dev->settings.isolate)
                system_bridge_set_isolated(dev, "1");
 
+       if (dev->bpdu_filter)
+               system_bridge_set_bpdu_filter(dev, dev->bpdu_filter ? "1" : "0");
+
        return ret;
 }
 
@@ -901,23 +916,21 @@ int system_bridge_delif(struct device *bridge, struct device *dev)
 
 int system_bridge_vlan(const char *iface, uint16_t vid, bool add, unsigned int vflags)
 {
-       struct ifinfomsg ifi = { .ifi_family = PF_BRIDGE, };
        struct bridge_vlan_info vinfo = { .vid = vid, };
        unsigned short flags = 0;
        struct nlattr *afspec;
        struct nl_msg *nlm;
+       int index;
        int ret = 0;
 
-       ifi.ifi_index = if_nametoindex(iface);
-       if (!ifi.ifi_index)
+       index = if_nametoindex(iface);
+       if (!index)
                return -1;
 
-       nlm = nlmsg_alloc_simple(add ? RTM_SETLINK : RTM_DELLINK, NLM_F_REQUEST);
+       nlm = __system_ifinfo_msg(PF_BRIDGE, index, NULL, add ? RTM_SETLINK : RTM_DELLINK, 0);
        if (!nlm)
                return -1;
 
-       nlmsg_append(nlm, &ifi, sizeof(ifi), 0);
-
        if (vflags & BRVLAN_F_SELF)
                flags |= BRIDGE_FLAGS_SELF;
 
@@ -946,6 +959,91 @@ failure:
        return ret;
 }
 
+int system_bonding_set_device(struct device *dev, struct bonding_config *cfg)
+{
+       const char *ifname = dev->ifname;
+       struct blob_attr *cur;
+       char op = cfg ? '+' : '-';
+       char buf[64];
+       int rem;
+
+       snprintf(dev_buf, sizeof(dev_buf), "%s/class/net/bonding_masters", sysfs_path);
+       snprintf(buf, sizeof(buf), "%c%s", op, ifname);
+       write_file(dev_buf, buf);
+
+       if (!cfg)
+               return 0;
+
+       system_set_dev_sysfs("bonding/mode", ifname, bonding_policy_str[cfg->policy]);
+
+       system_set_dev_sysfs_int("bonding/all_ports_active", ifname, cfg->all_ports_active);
+
+       if (cfg->policy == BONDING_MODE_BALANCE_XOR ||
+           cfg->policy == BONDING_MODE_BALANCE_TLB ||
+           cfg->policy == BONDING_MODE_8023AD)
+               system_set_dev_sysfs("bonding/xmit_hash_policy", ifname, cfg->xmit_hash_policy);
+
+       if (cfg->policy == BONDING_MODE_8023AD) {
+               system_set_dev_sysfs("bonding/ad_actor_system", ifname, cfg->ad_actor_system);
+               system_set_dev_sysfs_int("bonding/ad_actor_sys_prio", ifname, cfg->ad_actor_sys_prio);
+               system_set_dev_sysfs("bonding/ad_select", ifname, cfg->ad_select);
+               system_set_dev_sysfs("bonding/lacp_rate", ifname, cfg->lacp_rate);
+               system_set_dev_sysfs_int("bonding/min_links", ifname, cfg->min_links);
+       }
+
+       if (cfg->policy == BONDING_MODE_BALANCE_RR)
+               system_set_dev_sysfs_int("bonding/packets_per_slave", ifname, cfg->packets_per_port);
+
+       if (cfg->policy == BONDING_MODE_BALANCE_TLB ||
+           cfg->policy == BONDING_MODE_BALANCE_ALB)
+               system_set_dev_sysfs_int("bonding/lp_interval", ifname, cfg->lp_interval);
+
+       if (cfg->policy == BONDING_MODE_BALANCE_TLB)
+               system_set_dev_sysfs_int("bonding/tlb_dynamic_lb", ifname, cfg->dynamic_lb);
+       system_set_dev_sysfs_int("bonding/resend_igmp", ifname, cfg->resend_igmp);
+       system_set_dev_sysfs_int("bonding/num_grat_arp", ifname, cfg->num_peer_notif);
+       system_set_dev_sysfs("bonding/primary_reselect", ifname, cfg->primary_reselect);
+       system_set_dev_sysfs("bonding/fail_over_mac", ifname, cfg->failover_mac);
+
+       system_set_dev_sysfs_int((cfg->monitor_arp ?
+                                 "bonding/arp_interval" :
+                                 "bonding/miimon"), ifname, cfg->monitor_interval);
+
+       blobmsg_for_each_attr(cur, cfg->arp_target, rem) {
+               snprintf(buf, sizeof(buf), "+%s", blobmsg_get_string(cur));
+               system_set_dev_sysfs("bonding/arp_ip_target", ifname, buf);
+       }
+
+       system_set_dev_sysfs_int("bonding/arp_all_targets", ifname, cfg->arp_all_targets);
+       if (cfg->policy < BONDING_MODE_8023AD)
+               system_set_dev_sysfs("bonding/arp_validate", ifname, cfg->arp_validate);
+       system_set_dev_sysfs_int("bonding/use_carrier", ifname, cfg->use_carrier);
+       if (!cfg->monitor_arp && cfg->monitor_interval) {
+               system_set_dev_sysfs_int("bonding/updelay", ifname, cfg->updelay);
+               system_set_dev_sysfs_int("bonding/downdelay", ifname, cfg->downdelay);
+       }
+
+       return 0;
+}
+
+int system_bonding_set_port(struct device *dev, struct device *port, bool add, bool primary)
+{
+       const char *port_name = port->ifname;
+       const char op_ch = add ? '+' : '-';
+       char buf[IFNAMSIZ + 1];
+
+       snprintf(buf, sizeof(buf), "%c%s", op_ch, port_name);
+       system_if_down(port);
+       system_set_dev_sysfs("bonding/slaves", dev->ifname, buf);
+       system_if_up(port);
+
+       if (primary)
+               system_set_dev_sysfs("bonding/primary", dev->ifname,
+                                    add ? port_name : "");
+
+       return 0;
+}
+
 int system_if_resolve(struct device *dev)
 {
        struct ifreq ifr;
@@ -1155,7 +1253,7 @@ void system_if_clear_state(struct device *dev)
 
        system_if_flags(dev->ifname, 0, IFF_UP);
 
-       if (system_is_bridge(dev->ifname, buf, sizeof(buf))) {
+       if (system_is_bridge(dev->ifname)) {
                D(SYSTEM, "Delete existing bridge named '%s'\n", dev->ifname);
                system_bridge_delbr(dev);
                return;
@@ -1182,121 +1280,93 @@ sec_to_jiffies(int val)
        return (unsigned long) val * 100;
 }
 
-static void system_bridge_conf_multicast_deps(struct device *bridge,
-                                             struct bridge_config *cfg,
-                                             char *buf,
-                                             int buf_len)
+int system_bridge_addbr(struct device *bridge, struct bridge_config *cfg)
 {
-       int val;
-
-       if (cfg->flags & BRIDGE_OPT_ROBUSTNESS ||
-           cfg->flags & BRIDGE_OPT_QUERY_INTERVAL ||
-           cfg->flags & BRIDGE_OPT_QUERY_RESPONSE_INTERVAL) {
-               val = cfg->robustness * cfg->query_interval +
-                       cfg->query_response_interval;
-
-               snprintf(buf, buf_len, "%i", val);
-               system_bridge_set_membership_interval(bridge, buf);
-
-               val = cfg->robustness * cfg->query_interval +
-                       cfg->query_response_interval / 2;
-
-               snprintf(buf, buf_len, "%i", val);
-               system_bridge_set_other_querier_timeout(bridge, buf);
-       }
+       struct nlattr *linkinfo, *data;
+       struct nl_msg *msg;
+       uint64_t val;
+       int rv;
 
-       if (cfg->flags & BRIDGE_OPT_QUERY_INTERVAL) {
-               val = cfg->query_interval / 4;
+       msg = system_ifinfo_msg(bridge->ifname, RTM_NEWLINK, NLM_F_CREATE | NLM_F_EXCL);
+       if (!msg)
+               return -1;
 
-               snprintf(buf, buf_len, "%i", val);
-               system_bridge_set_startup_query_interval(bridge, buf);
-       }
-}
+       if (!(linkinfo = nla_nest_start(msg, IFLA_LINKINFO)))
+               goto nla_put_failure;
 
-static void system_bridge_conf_multicast(struct device *bridge,
-                                        struct bridge_config *cfg,
-                                        char *buf,
-                                        int buf_len)
-{
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_snooping",
-               bridge->ifname, cfg->igmp_snoop ? "1" : "0");
+       nla_put_string(msg, IFLA_INFO_KIND, "bridge");
 
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/multicast_querier",
-               bridge->ifname, cfg->multicast_querier ? "1" : "0");
+       if (!(data = nla_nest_start(msg, IFLA_INFO_DATA)))
+               goto nla_put_failure;
 
-       snprintf(buf, buf_len, "%i", cfg->hash_max);
-       system_set_dev_sysctl("/sys/devices/virtual/net/%s/bridge/hash_max",
-               bridge->ifname, buf);
+       nla_put_u32(msg, IFLA_BR_STP_STATE, cfg->stp);
+       nla_put_u32(msg, IFLA_BR_FORWARD_DELAY, sec_to_jiffies(cfg->forward_delay));
+       nla_put_u8(msg, IFLA_BR_MCAST_SNOOPING, !!cfg->igmp_snoop);
+       nla_put_u8(msg, IFLA_BR_MCAST_QUERIER, !!cfg->multicast_querier);
+       nla_put_u32(msg, IFLA_BR_MCAST_HASH_MAX, cfg->hash_max);
 
-       if (bridge->settings.flags & DEV_OPT_MULTICAST_ROUTER) {
-               snprintf(buf, buf_len, "%u", bridge->settings.multicast_router);
-               system_bridge_set_multicast_router(bridge, buf, true);
-       }
+       if (bridge->settings.flags & DEV_OPT_MULTICAST_ROUTER)
+               nla_put_u8(msg, IFLA_BR_MCAST_ROUTER, !!bridge->settings.multicast_router);
 
        if (cfg->flags & BRIDGE_OPT_ROBUSTNESS) {
-               snprintf(buf, buf_len, "%i", cfg->robustness);
-               system_bridge_set_robustness(bridge, buf);
+               nla_put_u32(msg, IFLA_BR_MCAST_STARTUP_QUERY_CNT, cfg->robustness);
+               nla_put_u32(msg, IFLA_BR_MCAST_LAST_MEMBER_CNT, cfg->robustness);
        }
 
-       if (cfg->flags & BRIDGE_OPT_QUERY_INTERVAL) {
-               snprintf(buf, buf_len, "%i", cfg->query_interval);
-               system_bridge_set_query_interval(bridge, buf);
-       }
+       if (cfg->flags & BRIDGE_OPT_QUERY_INTERVAL)
+               nla_put_u64(msg, IFLA_BR_MCAST_QUERY_INTVL, cfg->query_interval);
 
-       if (cfg->flags & BRIDGE_OPT_QUERY_RESPONSE_INTERVAL) {
-               snprintf(buf, buf_len, "%i", cfg->query_response_interval);
-               system_bridge_set_query_response_interval(bridge, buf);
-       }
+       if (cfg->flags & BRIDGE_OPT_QUERY_RESPONSE_INTERVAL)
+               nla_put_u64(msg, IFLA_BR_MCAST_QUERY_RESPONSE_INTVL, cfg->query_response_interval);
 
-       if (cfg->flags & BRIDGE_OPT_LAST_MEMBER_INTERVAL) {
-               snprintf(buf, buf_len, "%i", cfg->last_member_interval);
-               system_bridge_set_last_member_interval(bridge, buf);
-       }
+       if (cfg->flags & BRIDGE_OPT_LAST_MEMBER_INTERVAL)
+               nla_put_u64(msg, IFLA_BR_MCAST_LAST_MEMBER_INTVL, cfg->last_member_interval);
 
-       system_bridge_conf_multicast_deps(bridge, cfg, buf, buf_len);
-}
+       if (cfg->flags & BRIDGE_OPT_ROBUSTNESS ||
+           cfg->flags & BRIDGE_OPT_QUERY_INTERVAL ||
+           cfg->flags & BRIDGE_OPT_QUERY_RESPONSE_INTERVAL) {
+               val = cfg->robustness * cfg->query_interval +
+                       cfg->query_response_interval;
 
-int system_bridge_addbr(struct device *bridge, struct bridge_config *cfg)
-{
-       char buf[64];
+               nla_put_u64(msg, IFLA_BR_MCAST_MEMBERSHIP_INTVL, val);
 
-       if (ioctl(sock_ioctl, SIOCBRADDBR, bridge->ifname) < 0)
-               return -1;
+               val -= cfg->query_response_interval / 2;
 
-       system_bridge_set_stp_state(bridge, cfg->stp ? "1" : "0");
+               nla_put_u64(msg, IFLA_BR_MCAST_QUERIER_INTVL, val);
+       }
 
-       snprintf(buf, sizeof(buf), "%lu", sec_to_jiffies(cfg->forward_delay));
-       system_bridge_set_forward_delay(bridge, buf);
+       if (cfg->flags & BRIDGE_OPT_QUERY_INTERVAL) {
+               val = cfg->query_interval / 4;
 
-       system_bridge_conf_multicast(bridge, cfg, buf, sizeof(buf));
-       system_bridge_set_vlan_filtering(bridge, cfg->vlan_filtering ? "1" : "0");
+               nla_put_u64(msg, IFLA_BR_MCAST_STARTUP_QUERY_INTVL, val);
+       }
 
-       snprintf(buf, sizeof(buf), "%d", cfg->priority);
-       system_bridge_set_priority(bridge, buf);
+       nla_put_u8(msg, IFLA_BR_VLAN_FILTERING, !!cfg->vlan_filtering);
+       nla_put_u16(msg, IFLA_BR_PRIORITY, cfg->priority);
+       nla_put_u32(msg, IFLA_BR_HELLO_TIME, sec_to_jiffies(cfg->hello_time));
+       nla_put_u32(msg, IFLA_BR_MAX_AGE, sec_to_jiffies(cfg->max_age));
 
-       if (cfg->flags & BRIDGE_OPT_AGEING_TIME) {
-               snprintf(buf, sizeof(buf), "%lu", sec_to_jiffies(cfg->ageing_time));
-               system_bridge_set_ageing_time(bridge, buf);
-       }
+       if (cfg->flags & BRIDGE_OPT_AGEING_TIME)
+               nla_put_u32(msg, IFLA_BR_AGEING_TIME, sec_to_jiffies(cfg->ageing_time));
 
-       if (cfg->flags & BRIDGE_OPT_HELLO_TIME) {
-               snprintf(buf, sizeof(buf), "%lu", sec_to_jiffies(cfg->hello_time));
-               system_bridge_set_hello_time(bridge, buf);
-       }
+       nla_nest_end(msg, data);
+       nla_nest_end(msg, linkinfo);
 
-       if (cfg->flags & BRIDGE_OPT_MAX_AGE) {
-               snprintf(buf, sizeof(buf), "%lu", sec_to_jiffies(cfg->max_age));
-               system_bridge_set_max_age(bridge, buf);
-       }
+       rv = system_rtnl_call(msg);
+       if (rv)
+               D(SYSTEM, "Error adding bridge '%s': %d\n", bridge->ifname, rv);
 
-       return 0;
+       return rv;
+
+nla_put_failure:
+       nlmsg_free(msg);
+       return -ENOMEM;
 }
 
 int system_macvlan_add(struct device *macvlan, struct device *dev, struct macvlan_config *cfg)
 {
        struct nl_msg *msg;
        struct nlattr *linkinfo, *data;
-       struct ifinfomsg iim = { .ifi_family = AF_UNSPEC, };
        int i, rv;
        static const struct {
                const char *name;
@@ -1308,16 +1378,12 @@ int system_macvlan_add(struct device *macvlan, struct device *dev, struct macvla
                { "passthru", MACVLAN_MODE_PASSTHRU },
        };
 
-       msg = nlmsg_alloc_simple(RTM_NEWLINK, NLM_F_REQUEST | NLM_F_CREATE | NLM_F_EXCL);
-
+       msg = system_ifinfo_msg(macvlan->ifname, RTM_NEWLINK, NLM_F_CREATE | NLM_F_EXCL);
        if (!msg)
                return -1;
 
-       nlmsg_append(msg, &iim, sizeof(iim), 0);
-
        if (cfg->flags & MACVLAN_OPT_MACADDR)
                nla_put(msg, IFLA_ADDRESS, sizeof(cfg->macaddr), cfg->macaddr);
-       nla_put_string(msg, IFLA_IFNAME, macvlan->ifname);
        nla_put_u32(msg, IFLA_LINK, dev->ifindex);
 
        if (!(linkinfo = nla_nest_start(msg, IFLA_LINKINFO)))
@@ -1355,45 +1421,20 @@ nla_put_failure:
 int system_link_netns_move(struct device *dev, int netns_fd, const char *target_ifname)
 {
        struct nl_msg *msg;
-       struct ifinfomsg iim = {
-               .ifi_family = AF_UNSPEC,
-       };
+       int index;
 
        if (!dev)
                return -1;
 
-       iim.ifi_index = system_if_resolve(dev);
-       msg = nlmsg_alloc_simple(RTM_NEWLINK, NLM_F_REQUEST);
-
+       index = system_if_resolve(dev);
+       msg = __system_ifinfo_msg(AF_UNSPEC, index, target_ifname, RTM_NEWLINK, 0);
        if (!msg)
                return -1;
 
-       nlmsg_append(msg, &iim, sizeof(iim), 0);
-       if (target_ifname)
-               nla_put_string(msg, IFLA_IFNAME, target_ifname);
-
        nla_put_u32(msg, IFLA_NET_NS_FD, netns_fd);
        return system_rtnl_call(msg);
 }
 
-static int system_link_del(const char *ifname)
-{
-       struct nl_msg *msg;
-       struct ifinfomsg iim = {
-               .ifi_family = AF_UNSPEC,
-               .ifi_index = 0,
-       };
-
-       msg = nlmsg_alloc_simple(RTM_DELLINK, NLM_F_REQUEST);
-
-       if (!msg)
-               return -1;
-
-       nlmsg_append(msg, &iim, sizeof(iim), 0);
-       nla_put_string(msg, IFLA_IFNAME, ifname);
-       return system_rtnl_call(msg);
-}
-
 int system_macvlan_del(struct device *macvlan)
 {
        return system_link_del(macvlan->ifname);
@@ -2150,21 +2191,11 @@ struct device *
 system_if_get_parent(struct device *dev)
 {
        char buf[64], *devname;
-       int ifindex, iflink, len;
-       FILE *f;
-
-       snprintf(buf, sizeof(buf), "/sys/class/net/%s/iflink", dev->ifname);
-       f = fopen(buf, "r");
-       if (!f)
-               return NULL;
+       int ifindex, iflink;
 
-       len = fread(buf, 1, sizeof(buf) - 1, f);
-       fclose(f);
-
-       if (len <= 0)
+       if (system_get_dev_sysfs("iflink", dev->ifname, buf, sizeof(buf)) < 0)
                return NULL;
 
-       buf[len] = 0;
        iflink = strtoul(buf, NULL, 0);
        ifindex = system_if_resolve(dev);
        if (!iflink || iflink == ifindex)
@@ -2263,11 +2294,174 @@ static void system_add_link_modes(struct blob_buf *b, __u32 mask)
 bool
 system_if_force_external(const char *ifname)
 {
-       char buf[64];
        struct stat s;
 
-       snprintf(buf, sizeof(buf), "/sys/class/net/%s/phy80211", ifname);
-       return stat(buf, &s) == 0;
+       return stat(dev_sysfs_path(ifname, "phy80211"), &s) == 0;
+}
+
+static const char *
+system_netdevtype_name(unsigned short dev_type)
+{
+       unsigned int i;
+
+       for (i = 0; i < ARRAY_SIZE(netdev_types); i++) {
+               if (netdev_types[i].id == dev_type)
+                       return netdev_types[i].name;
+       }
+
+       /* the last key is used by default */
+       i = ARRAY_SIZE(netdev_types) - 1;
+
+       return netdev_types[i].name;
+}
+
+static void
+system_add_devtype(struct blob_buf *b, const char *ifname)
+{
+       char buf[100];
+       bool found = false;
+
+       if (!system_get_dev_sysfs("uevent", ifname, buf, sizeof(buf))) {
+               const char *info = "DEVTYPE=";
+               char *context = NULL;
+               const char *line = strtok_r(buf, "\r\n", &context);
+
+               while (line != NULL) {
+                       char *index = strstr(line, info);
+
+                       if (index != NULL) {
+                               blobmsg_add_string(b, "devtype", index + strlen(info));
+                               found = true;
+                               break;
+                       }
+
+                       line = strtok_r(NULL, "\r\n", &context);
+               }
+       }
+
+       if (!found) {
+               unsigned short number = 0;
+               const char *name = NULL;
+
+               if (!system_get_dev_sysfs("type", ifname, buf, sizeof(buf))) {
+                       number = strtoul(buf, NULL, 0);
+                       name = system_netdevtype_name(number);
+                       blobmsg_add_string(b, "devtype", name);
+               }
+       }
+}
+
+#define DIV_ROUND_UP(n,d) (((n) + (d) - 1) / (d))
+
+static int32_t
+ethtool_feature_count(const char *ifname)
+{
+       struct {
+               struct ethtool_sset_info hdr;
+               uint32_t buf;
+       } req = {
+               .hdr = {
+                       .cmd = ETHTOOL_GSSET_INFO,
+                       .sset_mask = 1 << ETH_SS_FEATURES
+               }
+       };
+
+       struct ifreq ifr = {
+               .ifr_data = (void *)&req
+       };
+
+       strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name) - 1);
+
+       if (ioctl(sock_ioctl, SIOCETHTOOL, &ifr) != 0)
+               return -1;
+
+       if (!req.hdr.sset_mask)
+               return 0;
+
+       return req.buf;
+}
+
+static int32_t
+ethtool_feature_index(const char *ifname, const char *keyname)
+{
+       struct ethtool_gstrings *feature_names;
+       struct ifreq ifr = { 0 };
+       int32_t n_features, i;
+
+       n_features = ethtool_feature_count(ifname);
+
+       if (n_features <= 0)
+               return -1;
+
+       feature_names = calloc(1, sizeof(*feature_names) + n_features * ETH_GSTRING_LEN);
+
+       if (!feature_names)
+               return -1;
+
+       feature_names->cmd = ETHTOOL_GSTRINGS;
+       feature_names->string_set = ETH_SS_FEATURES;
+       feature_names->len = n_features;
+
+       strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name) - 1);
+       ifr.ifr_data = (void *)feature_names;
+
+       if (ioctl(sock_ioctl, SIOCETHTOOL, &ifr) != 0) {
+               free(feature_names);
+
+               return -1;
+       }
+
+       for (i = 0; i < feature_names->len; i++)
+               if (!strcmp((char *)&feature_names->data[i * ETH_GSTRING_LEN], keyname))
+                       break;
+
+       if (i >= feature_names->len)
+               i = -1;
+
+       free(feature_names);
+
+       return i;
+}
+
+static bool
+ethtool_feature_value(const char *ifname, const char *keyname)
+{
+       struct ethtool_get_features_block *feature_block;
+       struct ethtool_gfeatures *feature_values;
+       struct ifreq ifr = { 0 };
+       int32_t feature_idx;
+       bool active;
+
+       feature_idx = ethtool_feature_index(ifname, keyname);
+
+       if (feature_idx < 0)
+               return false;
+
+       feature_values = calloc(1,
+               sizeof(*feature_values) +
+               sizeof(feature_values->features[0]) * DIV_ROUND_UP(feature_idx, 32));
+
+       if (!feature_values)
+               return false;
+
+       feature_values->cmd = ETHTOOL_GFEATURES;
+       feature_values->size = DIV_ROUND_UP(feature_idx, 32);
+
+       strncpy(ifr.ifr_name, ifname, sizeof(ifr.ifr_name) - 1);
+       ifr.ifr_data = (void *)feature_values;
+
+       if (ioctl(sock_ioctl, SIOCETHTOOL, &ifr) != 0) {
+               free(feature_values);
+
+               return false;
+       }
+
+       feature_block = &feature_values->features[feature_idx / 32];
+       active = feature_block->active & (1U << feature_idx % 32);
+
+       free(feature_values);
+
+       return active;
 }
 
 int
@@ -2305,6 +2499,11 @@ system_if_dump_info(struct device *dev, struct blob_buf *b)
                blobmsg_add_u8(b, "autoneg", !!ecmd.autoneg);
        }
 
+       blobmsg_add_u8(b, "hw-tc-offload",
+               ethtool_feature_value(dev->ifname, "hw-tc-offload"));
+
+       system_add_devtype(b, dev->ifname);
+
        return 0;
 }
 
@@ -2321,13 +2520,11 @@ system_if_dump_stats(struct device *dev, struct blob_buf *b)
                "rx_errors",      "tx_bytes",          "tx_window_errors",
                "rx_fifo_errors", "tx_carrier_errors",
        };
-       char buf[64];
        int stats_dir;
        int i;
        uint64_t val = 0;
 
-       snprintf(buf, sizeof(buf), "/sys/class/net/%s/statistics", dev->ifname);
-       stats_dir = open(buf, O_DIRECTORY);
+       stats_dir = open(dev_sysfs_path(dev->ifname, "statistics"), O_DIRECTORY);
        if (stats_dir < 0)
                return -1;
 
@@ -2569,14 +2766,12 @@ int system_del_route(struct device *dev, struct device_route *route)
 
 int system_flush_routes(void)
 {
-       const char *names[] = {
-               "/proc/sys/net/ipv4/route/flush",
-               "/proc/sys/net/ipv6/route/flush"
-       };
+       const char *names[] = { "ipv4", "ipv6" };
        int fd, i;
 
        for (i = 0; i < ARRAY_SIZE(names); i++) {
-               fd = open(names[i], O_WRONLY);
+               snprintf(dev_buf, sizeof(dev_buf), "%s/sys/net/%s/route/flush", proc_path, names[i]);
+               fd = open(dev_buf, O_WRONLY);
                if (fd < 0)
                        continue;
 
@@ -2779,6 +2974,15 @@ static int system_iprule(struct iprule *rule, int cmd)
        if (rule->flags & IPRULE_SUP_PREFIXLEN)
                nla_put_u32(msg, FRA_SUPPRESS_PREFIXLEN, rule->sup_prefixlen);
 
+       if (rule->flags & IPRULE_UIDRANGE) {
+               struct fib_rule_uid_range uidrange = {
+                       .start = rule->uidrange_start,
+                       .end = rule->uidrange_end
+               };
+
+               nla_put(msg, FRA_UID_RANGE, sizeof(uidrange), &uidrange);
+       }
+
        if (rule->flags & IPRULE_GOTO)
                nla_put_u32(msg, FRA_GOTO, rule->gotoid);
 
@@ -3669,7 +3873,7 @@ static int system_add_sit_tunnel(const char *name, const unsigned int link, stru
        return ret;
 
 failure:
-       __system_del_ip_tunnel(name, tb);
+       system_link_del(name);
        return ret;
 }
 
@@ -3731,33 +3935,9 @@ static int system_add_proto_tunnel(const char *name, const uint8_t proto, const
        return -1;
 }
 
-static int __system_del_ip_tunnel(const char *name, struct blob_attr **tb)
+int system_del_ip_tunnel(const struct device *dev)
 {
-       struct blob_attr *cur;
-       const char *str;
-
-       if (!(cur = tb[TUNNEL_ATTR_TYPE]))
-               return -EINVAL;
-       str = blobmsg_data(cur);
-
-       if (!strcmp(str, "greip") || !strcmp(str, "gretapip") ||
-           !strcmp(str, "greip6") || !strcmp(str, "gretapip6") ||
-           !strcmp(str, "vtiip") || !strcmp(str, "vtiip6") ||
-           !strcmp(str, "vxlan") || !strcmp(str, "vxlan6") ||
-           !strcmp(str, "xfrm"))
-               return system_link_del(name);
-       else
-               return tunnel_ioctl(name, SIOCDELTUNNEL, NULL);
-}
-
-int system_del_ip_tunnel(const char *name, struct blob_attr *attr)
-{
-       struct blob_attr *tb[__TUNNEL_ATTR_MAX];
-
-       blobmsg_parse(tunnel_attr_list.params, __TUNNEL_ATTR_MAX, tb,
-               blob_data(attr), blob_len(attr));
-
-       return __system_del_ip_tunnel(name, tb);
+       return system_link_del(dev->ifname);
 }
 
 int system_update_ipv6_mtu(struct device *dev, int mtu)
@@ -3766,10 +3946,7 @@ int system_update_ipv6_mtu(struct device *dev, int mtu)
        char buf[64];
        int fd;
 
-       snprintf(buf, sizeof(buf), "/proc/sys/net/ipv6/conf/%s/mtu",
-                       dev->ifname);
-
-       fd = open(buf, O_RDWR);
+       fd = open(dev_sysctl_path("ipv6/conf", dev->ifname, "mtu"), O_RDWR);
        if (fd < 0)
                return ret;
 
@@ -3790,7 +3967,7 @@ out:
        return ret;
 }
 
-int system_add_ip_tunnel(const char *name, struct blob_attr *attr)
+int system_add_ip_tunnel(const struct device *dev, struct blob_attr *attr)
 {
        struct blob_attr *tb[__TUNNEL_ATTR_MAX];
        struct blob_attr *cur;
@@ -3799,7 +3976,7 @@ int system_add_ip_tunnel(const char *name, struct blob_attr *attr)
        blobmsg_parse(tunnel_attr_list.params, __TUNNEL_ATTR_MAX, tb,
                blob_data(attr), blob_len(attr));
 
-       __system_del_ip_tunnel(name, tb);
+       system_link_del(dev->ifname);
 
        if (!(cur = tb[TUNNEL_ATTR_TYPE]))
                return -EINVAL;
@@ -3823,37 +4000,37 @@ int system_add_ip_tunnel(const char *name, struct blob_attr *attr)
        }
 
        if (!strcmp(str, "sit"))
-               return system_add_sit_tunnel(name, link, tb);
+               return system_add_sit_tunnel(dev->ifname, link, tb);
 #ifdef IFLA_IPTUN_MAX
        else if (!strcmp(str, "ipip6")) {
-               return system_add_ip6_tunnel(name, link, tb);
+               return system_add_ip6_tunnel(dev->ifname, link, tb);
        } else if (!strcmp(str, "greip")) {
-               return system_add_gre_tunnel(name, "gre", link, tb, false);
+               return system_add_gre_tunnel(dev->ifname, "gre", link, tb, false);
        } else if (!strcmp(str, "gretapip"))  {
-               return system_add_gre_tunnel(name, "gretap", link, tb, false);
+               return system_add_gre_tunnel(dev->ifname, "gretap", link, tb, false);
        } else if (!strcmp(str, "greip6")) {
-               return system_add_gre_tunnel(name, "ip6gre", link, tb, true);
+               return system_add_gre_tunnel(dev->ifname, "ip6gre", link, tb, true);
        } else if (!strcmp(str, "gretapip6")) {
-               return system_add_gre_tunnel(name, "ip6gretap", link, tb, true);
+               return system_add_gre_tunnel(dev->ifname, "ip6gretap", link, tb, true);
 #ifdef IFLA_VTI_MAX
        } else if (!strcmp(str, "vtiip")) {
-               return system_add_vti_tunnel(name, "vti", link, tb, false);
+               return system_add_vti_tunnel(dev->ifname, "vti", link, tb, false);
        } else if (!strcmp(str, "vtiip6")) {
-               return system_add_vti_tunnel(name, "vti6", link, tb, true);
+               return system_add_vti_tunnel(dev->ifname, "vti6", link, tb, true);
 #endif
 #ifdef IFLA_XFRM_MAX
        } else if (!strcmp(str, "xfrm")) {
-               return system_add_xfrm_tunnel(name, "xfrm", link, tb);
+               return system_add_xfrm_tunnel(dev->ifname, "xfrm", link, tb);
 #endif
 #ifdef IFLA_VXLAN_MAX
        } else if(!strcmp(str, "vxlan")) {
-               return system_add_vxlan(name, link, tb, false);
+               return system_add_vxlan(dev->ifname, link, tb, false);
        } else if(!strcmp(str, "vxlan6")) {
-               return system_add_vxlan(name, link, tb, true);
+               return system_add_vxlan(dev->ifname, link, tb, true);
 #endif
 #endif
        } else if (!strcmp(str, "ipip")) {
-               return system_add_proto_tunnel(name, IPPROTO_IPIP, link, tb);
+               return system_add_proto_tunnel(dev->ifname, IPPROTO_IPIP, link, tb);
        }
        else
                return -EINVAL;