From: Tom Gundersen Date: Sat, 26 Oct 2013 16:54:16 +0000 (+0200) Subject: udev: link-config: add rtnl support X-Git-Tag: v209~1773 X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?p=elogind.git;a=commitdiff_plain;h=43b3a5ef61859f06cdbaf26765cab8e1adac4296 udev: link-config: add rtnl support This adds support for setting the mac address, name and mtu. Example: [Link] MTU=1450 MACAddress=98:76:54:32:10:ab Name=wireless0 --- diff --git a/Makefile.am b/Makefile.am index 6ce4b7fff..e5ed3020b 100644 --- a/Makefile.am +++ b/Makefile.am @@ -2344,6 +2344,7 @@ libudev_core_la_LIBADD = \ libsystemd-label.la \ libsystemd-daemon-internal.la \ libsystemd-shared.la \ + libsystemd-rtnl.la \ $(BLKID_LIBS) \ $(KMOD_LIBS) diff --git a/src/udev/net/link-config-gperf.gperf b/src/udev/net/link-config-gperf.gperf index 9e2f4d4f5..182a66419 100644 --- a/src/udev/net/link-config-gperf.gperf +++ b/src/udev/net/link-config-gperf.gperf @@ -19,6 +19,9 @@ Match.Path, config_parse_string, 0, offsetof(link Match.Driver, config_parse_string, 0, offsetof(link_config, match_driver) Match.Type, config_parse_string, 0, offsetof(link_config, match_type) Link.Description, config_parse_string, 0, offsetof(link_config, description) +Link.MACAddress, config_parse_string, 0, offsetof(link_config, mac) +Link.Name, config_parse_string, 0, offsetof(link_config, name) +Link.MTU, config_parse_unsigned, 0, offsetof(link_config, mtu) Link.SpeedMBytes, config_parse_unsigned, 0, offsetof(link_config, speed) Link.Duplex, config_parse_string, 0, offsetof(link_config, duplex) Link.WakeOnLan, config_parse_string, 0, offsetof(link_config, wol) diff --git a/src/udev/net/link-config.c b/src/udev/net/link-config.c index 3beb28acb..902ed0955 100644 --- a/src/udev/net/link-config.c +++ b/src/udev/net/link-config.c @@ -19,10 +19,14 @@ along with systemd; If not, see . ***/ +#include + #include "link-config.h" #include "ethtool-util.h" +#include "libudev-private.h" +#include "sd-rtnl.h" #include "util.h" #include "log.h" #include "strv.h" @@ -35,6 +39,8 @@ struct link_config_ctx { int ethtool_fd; + sd_rtnl *rtnl; + char **link_dirs; usec_t *link_dirs_ts_usec; }; @@ -56,6 +62,12 @@ int link_config_ctx_new(link_config_ctx **ret) { return r; } + r = sd_rtnl_open(0, &ctx->rtnl); + if (r < 0) { + link_config_ctx_free(ctx); + return r; + } + LIST_HEAD_INIT(ctx->links); ctx->link_dirs = strv_new("/etc/net/links", @@ -104,7 +116,11 @@ void link_config_ctx_free(link_config_ctx *ctx) { if (!ctx) return; - close_nointr_nofail(ctx->ethtool_fd); + if (ctx->ethtool_fd >= 0) + close_nointr_nofail(ctx->ethtool_fd); + + sd_rtnl_unref(ctx->rtnl); + strv_free(ctx->link_dirs); free(ctx->link_dirs_ts_usec); link_configs_free(ctx); @@ -234,9 +250,54 @@ int link_config_get(link_config_ctx *ctx, struct udev_device *device, link_confi return -ENOENT; } +static int rtnl_set_properties(sd_rtnl *rtnl, int ifindex, const char *name, const char *mac, unsigned int mtu) { + _cleanup_sd_rtnl_message_unref_ sd_rtnl_message *message; + bool need_update; + int r; + + assert(rtnl); + assert(ifindex > 0); + + r = sd_rtnl_message_link_new(RTM_NEWLINK, ifindex, 0, 0, &message); + if (r < 0) + return r; + + if (name) { + r = sd_rtnl_message_append(message, IFLA_IFNAME, name); + if (r < 0) + return r; + + need_update = true; + } + + if (mac) { + r = sd_rtnl_message_append(message, IFLA_ADDRESS, ether_aton(mac)); + if (r < 0) + return r; + + need_update = true; + } + + if (mtu > 0) { + r = sd_rtnl_message_append(message, IFLA_MTU, &mtu); + if (r < 0) + return r; + + need_update = true; + } + + if (need_update) { + r = sd_rtnl_send_with_reply_and_block(rtnl, message, 250 * USEC_PER_MSEC, NULL); + if (r < 0) + return r; + } + + return 0; +} + int link_config_apply(link_config_ctx *ctx, link_config *config, struct udev_device *device) { const char *name; - int r; + int r, ifindex; name = udev_device_get_sysname(device); if (!name) @@ -275,5 +336,17 @@ int link_config_apply(link_config_ctx *ctx, link_config *config, struct udev_dev log_info("Set WakeOnLan of %s to %s", name, config->wol); } + ifindex = udev_device_get_ifindex(device); + if (ifindex <= 0) { + log_warning("Could not find ifindex"); + return -ENODEV; + } + + r = rtnl_set_properties(ctx->rtnl, ifindex, config->name, config->mac, config->mtu); + if (r < 0) { + log_warning("Could not set Name, MACAddress or MTU on %s", name); + return r; + } + return 0; } diff --git a/src/udev/net/link-config.h b/src/udev/net/link-config.h index 57d2c9c3c..849e48152 100644 --- a/src/udev/net/link-config.h +++ b/src/udev/net/link-config.h @@ -38,6 +38,9 @@ struct link_config { char *match_type; char *description; + char *mac; + char *name; + unsigned int mtu; unsigned int speed; char *duplex; char *wol; diff --git a/src/udev/udev-builtin-net_link.c b/src/udev/udev-builtin-net_link.c index d7cbe6a01..f4fb63ec5 100644 --- a/src/udev/udev-builtin-net_link.c +++ b/src/udev/udev-builtin-net_link.c @@ -37,7 +37,7 @@ static int builtin_net_link(struct udev_device *dev, int argc, char **argv, bool r = link_config_get(ctx, dev, &link); if (r < 0) { if (r == -ENOENT) { - log_info("No matching link configuration found"); + log_debug("No matching link configuration found"); return EXIT_SUCCESS; } else { log_error("Could not get link config"); @@ -47,7 +47,7 @@ static int builtin_net_link(struct udev_device *dev, int argc, char **argv, bool r = link_config_apply(ctx, link, dev); if (r < 0) { - log_error("Could not apply link config"); + log_error("Could not apply link config to %s", udev_device_get_sysname(dev)); return EXIT_FAILURE; }