File 1098-udev-link_setup-respect-kernel-name-assign... of Package systemd.578

Overview Repositories Revisions Requests Users Attributes Meta

File 1098-udev-link_setup-respect-kernel-name-assign-policy.patch of Package systemd.578 (Revision b88c65b2d8c409aaada43c073eca2758)

Currently displaying revision b88c65b2d8c409aaada43c073eca2758 , Show latest

Based on 04b67d49254d956d31bcfe80340fb9df7ed332d3 Mon Sep 17 00:00:00 2001
From: Tom Gundersen <teg@jklm.no>
Date: Tue, 15 Jul 2014 17:57:43 +0200
Subject: [PATCH] udev: link_setup - respect kernel name assign policy

Newer kernels export meta-information about the origin of an ifname. Respect this
from the ifname rename logic. We do not rename any interfaces that was originally
named by userspace, nor once which have already been renamed from userspace.

Moreover, we optionally do not (the default) rename interfaces which the kernel
claims to have named in a predictable way.
---
 man/udev.xml               |   13 ++++++++++-
 network/99-default.link    |    2 -
 src/shared/missing.h       |   21 +++++++++++++++++
 src/udev/net/link-config.c |   53 +++++++++++++++++++++++++++++++++++++--------
 src/udev/net/link-config.h |    1 
 5 files changed, 79 insertions(+), 11 deletions(-)

Index: man/udev.xml
===================================================================
--- man/udev.xml.orig
+++ man/udev.xml
@@ -914,10 +914,12 @@
               successful one is used. The name is not set directly, but
               is exported to udev as the property <literal>ID_NET_NAME</literal>,
               which is, by default, used by a udev rule to set
-              <literal>NAME</literal>. The available policies are:
+              <literal>NAME</literal>. If the name has already
+              been set by userspace, no renaming is performed.
+              The available policies are:
             </para>
             <variablelist>
-              <varlistentry>
+             <varlistentry>
                 <term><literal>database</literal></term>
                 <listitem>
                   <para>The name is set based on entries in the Hardware
Index: src/shared/missing.h
===================================================================
--- src/shared/missing.h.orig
+++ src/shared/missing.h
@@ -362,3 +362,24 @@ static inline int setns(int fd, int nsty
         return syscall(__NR_setns, fd, nstype);
 }
 #endif
+
+#ifndef NET_ADDR_RANDOM
+#  define NET_ADDR_RANDOM 1
+#endif
+
+#ifndef NET_NAME_ENUM
+#  define NET_NAME_ENUM 1
+#endif
+
+#ifndef NET_NAME_PREDICTABLE
+#  define NET_NAME_PREDICTABLE 2
+#endif
+
+#ifndef NET_NAME_USER
+#  define NET_NAME_USER 3
+#endif
+
+#ifndef NET_NAME_RENAMED
+#  define NET_NAME_RENAMED 4
+#endif
+
Index: src/udev/net/link-config.c
===================================================================
--- src/udev/net/link-config.c.orig
+++ src/udev/net/link-config.c
@@ -20,10 +20,11 @@
 ***/
 
 #include <netinet/ether.h>
-#include <net/if.h>
+#include <linux/netdevice.h>
 
 #include "sd-id128.h"
 
+#include "missing.h"
 #include "link-config.h"
 #include "ethtool-util.h"
 
@@ -297,23 +298,30 @@ static bool mac_is_random(struct udev_de
                 return false;
 
         /* check for NET_ADDR_RANDOM */
-        return type == 1;
+        return type == NET_ADDR_RANDOM;
 }
 
-static bool mac_is_permanent(struct udev_device *device) {
+static bool should_rename(struct udev_device *device) {
         const char *s;
         unsigned type;
         int r;
 
-        s = udev_device_get_sysattr_value(device, "addr_assign_type");
+        s = udev_device_get_sysattr_value(device, "name_assign_type");
         if (!s)
-                return true; /* if we don't know, assume it is permanent */
+                return true; /* if we don't know, assume we should rename */
         r = safe_atou(s, &type);
         if (r < 0)
                 return true;
 
-        /* check for NET_ADDR_PERM */
-        return type == 0;
+        switch (type) {
+        case NET_NAME_USER:
+        case NET_NAME_RENAMED:
+                return false; /* these were already named by userspace, do not touch again */
+        case NET_NAME_PREDICTABLE:
+        case NET_NAME_ENUM:
+        default:
+                return true; /* the name is known to be bad, or of an unknown type */
+        }
 }
 
 #define HASH_KEY SD_ID128_MAKE(d3,1e,48,fa,90,fe,4b,4c,9d,af,d5,d7,a1,b1,2e,8a)
@@ -429,16 +437,16 @@ int link_config_apply(link_config_ctx *c
                 }
         }
 
-        if (new_name)
-                *name = new_name; /* a name was set by a policy */
-        else if (config->name)
-                *name = config->name; /* a name was set manually in the config */
-        else
-                *name = NULL;
+        if (should_rename(device)) {
+                if (!new_name)
+                        /* if not set by policy, fall back manually set name */
+                        new_name = config->name;
+        } else
+                new_name = NULL;
 
         switch (config->mac_policy) {
                 case MACPOLICY_PERSISTENT:
-                        if (!mac_is_permanent(device)) {
+                        if (mac_is_random(device)) {
                                 r = get_mac(device, false, &generated_mac);
                                 if (r == -ENOENT)
                                         break;
@@ -467,6 +475,8 @@ int link_config_apply(link_config_ctx *c
                 return r;
         }
 
+        *name = new_name;
+
         return 0;
 }

Places

File 1098-udev-link_setup-respect-kernel-name-assign-policy.patch of Package systemd.578 (Revision b88c65b2d8c409aaada43c073eca2758)

Places