If we take over the hotplug call and manage the events we don't need
to call the event fake script in dev.d/. Just set all expected values
to the new network interface name and call hotplug.d/. This way the
device renaming is completely handled inside of udev and userspace
can't get confused.
# Copyright (C) 2004 Greg Kroah-Hartman <greg@kroah.com>
#
# Copyright (C) 2004 Greg Kroah-Hartman <greg@kroah.com>
#
-# See if we really did rename this device.
-if [ "$INTERFACE" = "$DEVNAME" ]; then
+# Do nothing if udev handles hotplug.d.
+if [ "$MANAGED_EVENT" == "1" ]; then
-# ok, we have renamed this device, so let's set $INTERFACE to the new name
-# and call the network hotplug script to handle it properly...
-INTERFACE=$DEVNAME
-export INTERFACE
+# ok, we have renamed this device, so let the network hotplug script
+# know about it to setup the device properly...
if [ -f /etc/hotplug.d/default/default.hotplug ]; then
exec /etc/hotplug.d/default/default.hotplug net
fi
if [ -f /etc/hotplug.d/default/default.hotplug ]; then
exec /etc/hotplug.d/default/default.hotplug net
fi
-/* (for now) true if udevsend is the helper */
+/* decide if we should manage the whole hotplug event
+ * for now look if the kernel calls udevsend instead of /sbin/hotplug
+ */
static int manage_hotplug_event(void) {
char helper[256];
int fd;
int len;
static int manage_hotplug_event(void) {
char helper[256];
int fd;
int len;
- /* false, if we are called directly */
- if (!getenv("MANAGED_EVENT"))
- goto exit;
+ /* don't handle hotplug.d if we are called directly */
+ if (!getenv("UDEVD_EVENT"))
+ return 0;
fd = open("/proc/sys/kernel/hotplug", O_RDONLY);
if (fd < 0)
fd = open("/proc/sys/kernel/hotplug", O_RDONLY);
if (fd < 0)
len = read(fd, helper, 256);
close(fd);
if (len < 0)
len = read(fd, helper, 256);
close(fd);
if (len < 0)
helper[len] = '\0';
if (strstr(helper, "udevsend"))
return 1;
helper[len] = '\0';
if (strstr(helper, "udevsend"))
return 1;
const char *action;
const char *devpath;
const char *subsystem;
const char *action;
const char *devpath;
const char *subsystem;
int retval = -EINVAL;
if (argc == 2 && strcmp(argv[1], "-V") == 0) {
int retval = -EINVAL;
if (argc == 2 && strcmp(argv[1], "-V") == 0) {
+ /* let the executed programs know if we handle the whole hotplug event */
+ managed_event = manage_hotplug_event();
+ if (managed_event)
+ setenv("MANAGED_EVENT", "1", 1);
+
action = getenv("ACTION");
devpath = getenv("DEVPATH");
subsystem = getenv("SUBSYSTEM");
action = getenv("ACTION");
devpath = getenv("DEVPATH");
subsystem = getenv("SUBSYSTEM");
- if (udev_hotplug_d && manage_hotplug_event())
+ if (udev_hotplug_d && managed_event)
udev_multiplex_directory(&udev, HOTPLUGD_DIR, HOTPLUG_SUFFIX);
exit:
udev_multiplex_directory(&udev, HOTPLUGD_DIR, HOTPLUG_SUFFIX);
exit:
if (retval != 0)
goto exit;
if (retval != 0)
goto exit;
- /* we've changed the name, now fake the devpath,
- * cause original kernel name sleeps with the fishes
- * and we don't get any event from the kernel now
+ /* we've changed the name, now fake the devpath, cause the
+ * original kernel name sleeps with the fishes and we don't
+ * get an event from the kernel with the new name
*/
pos = strrchr(udev->devpath, '/');
if (pos != NULL) {
pos[1] = '\0';
strfieldcat(udev->devpath, udev->name);
setenv("DEVPATH", udev->devpath, 1);
*/
pos = strrchr(udev->devpath, '/');
if (pos != NULL) {
pos[1] = '\0';
strfieldcat(udev->devpath, udev->name);
setenv("DEVPATH", udev->devpath, 1);
+ setenv("INTERFACE", udev->name, 1);
}
/* use netif name for the environment */
}
/* use netif name for the environment */
if (strncmp(key, "PHYSDEVPATH=", 12) == 0)
msg->physdevpath = &key[12];
}
if (strncmp(key, "PHYSDEVPATH=", 12) == 0)
msg->physdevpath = &key[12];
}
- msg->envp[i++] = "MANAGED_EVENT=1";
+ msg->envp[i++] = "UDEVD_EVENT=1";
msg->envp[i] = NULL;
/* if no seqnum is given, we move straight to exec queue */
msg->envp[i] = NULL;
/* if no seqnum is given, we move straight to exec queue */
}
/* prevent loops in the scripts we execute */
}
/* prevent loops in the scripts we execute */
- if (strncmp(key, "MANAGED_EVENT=", 14) == 0) {
+ if (strncmp(key, "UDEVD_EVENT=", 12) == 0) {
dbg("seems that the event source is not the kernel, just exit");
goto exit;
}
dbg("seems that the event source is not the kernel, just exit");
goto exit;
}