> Sorry, I left my debug code in.
Oops, just realized it now. You've applied a older version and not the
latest. Here is a new patch to catch up.
$(INSTALL_PROGRAM) -D $(WAIT) $(DESTDIR)$(sbindir)/$(WAIT)
- ln -f -s $(sbindir)/udev $(DESTDIR)$(sbindir)/$(STARTER)
- ln -f -s $(sbindir)/$(SENDER) $(DESTDIR)$(hotplugdir)/10-udev.hotplug
$(INSTALL_PROGRAM) -D $(WAIT) $(DESTDIR)$(sbindir)/$(WAIT)
- ln -f -s $(sbindir)/udev $(DESTDIR)$(sbindir)/$(STARTER)
- ln -f -s $(sbindir)/$(SENDER) $(DESTDIR)$(hotplugdir)/10-udev.hotplug
- - ln -f -s $(sbindir)/$(WAIT) $(DESTDIR)$(hotplugdir)/00-wait_for_sysfs.hotplug
+ - ln -f -s $(sbindir)/$(WAIT) $(DESTDIR)$(hotplugdir)/05-wait_for_sysfs.hotplug
ifndef DESTDIR
- killall udevd
- rm -f $(udevdir)/.udev.tdb
ifndef DESTDIR
- killall udevd
- rm -f $(udevdir)/.udev.tdb
uninstall: uninstall-man uninstall-dev.d
- rm $(hotplugdir)/10-udev.hotplug
uninstall: uninstall-man uninstall-dev.d
- rm $(hotplugdir)/10-udev.hotplug
- - rm $(hotplugdir)/00-wait_for_sysfs.hotplug
+ - rm $(hotplugdir)/05-wait_for_sysfs.hotplug
- rm $(configdir)/rules.d/50-udev.rules
- rm $(configdir)/permissions.d/50-udev.permissions
- rm $(configdir)/udev.conf
- rm $(configdir)/rules.d/50-udev.rules
- rm $(configdir)/permissions.d/50-udev.permissions
- rm $(configdir)/udev.conf
/*
* wait_for_sysfs.c - small program to delay the execution
* of /etc/hotplug.d/ programs, until sysfs
/*
* wait_for_sysfs.c - small program to delay the execution
* of /etc/hotplug.d/ programs, until sysfs
- * is populated by the kernel. Depending on
+ * is fully populated by the kernel. Depending on
* the type of device, we wait for all expected
* directories and then just exit.
*
* the type of device, we wait for all expected
* directories and then just exit.
*
#define WAIT_MAX_SECONDS 5
#define WAIT_LOOP_PER_SECOND 20
#define WAIT_MAX_SECONDS 5
#define WAIT_LOOP_PER_SECOND 20
+/* wait for specific file to show up, normally the "dev"-file */
static int wait_for_class_device_attributes(struct sysfs_class_device *class_dev)
{
static struct class_file {
static int wait_for_class_device_attributes(struct sysfs_class_device *class_dev)
{
static struct class_file {
char *file;
} class_files[] = {
{ .subsystem = "net", .file = "ifindex" },
char *file;
} class_files[] = {
{ .subsystem = "net", .file = "ifindex" },
+ { .subsystem = "scsi_host", .file = "unique_id" },
+ { .subsystem = "scsi_device", .file = NULL },
+ { .subsystem = "pcmcia_socket", .file = "card_type" },
{ .subsystem = "usb_host", .file = NULL },
{ .subsystem = "usb_host", .file = NULL },
- { .subsystem = "pcmcia_socket", .file = NULL },
{ NULL, NULL }
};
struct class_file *classfile;
{ NULL, NULL }
};
struct class_file *classfile;
dbg("class '%s' specific file '%s' found", class_dev->classname, file);
return 0;
}
dbg("class '%s' specific file '%s' found", class_dev->classname, file);
return 0;
}
+
+ usleep(1000 * 1000 / WAIT_LOOP_PER_SECOND);
- dbg("error: getting bus '%s' specific file '%s'", class_dev->classname, file);
+ dbg("error: getting class '%s' specific file '%s'", class_dev->classname, file);
+/* skip waiting for physical device */
static int class_device_expect_no_device_link(struct sysfs_class_device *class_dev)
{
static int class_device_expect_no_device_link(struct sysfs_class_device *class_dev)
{
static char *devices_without_link[] = {
"nb",
"ram",
static char *devices_without_link[] = {
"nb",
"ram",
for (device = devices_without_link; *device != NULL; device++) {
int len = strlen(*device);
for (device = devices_without_link; *device != NULL; device++) {
int len = strlen(*device);
+/* skip waiting for the bus */
+static int class_device_expect_no_bus(struct sysfs_class_device *class_dev)
+{
+ static char *devices_without_bus[] = {
+ "scsi_host",
+ NULL
+ };
+ char **device;
+
+ for (device = devices_without_bus; *device != NULL; device++) {
+ int len = strlen(*device);
+
+ if (strncmp(class_dev->classname, *device, len) == 0)
+ return 1;
+ }
+
+ return 0;
+}
+
+/* wait for the bus and for a bus specific file to show up */
static int wait_for_bus_device(struct sysfs_device *device_dev)
{
static struct bus_file {
static int wait_for_bus_device(struct sysfs_device *device_dev)
{
static struct bus_file {
{ .bus = "usb-serial", .file = "detach_state" },
{ .bus = "ide", .file = "detach_state" },
{ .bus = "pci", .file = "vendor" },
{ .bus = "usb-serial", .file = "detach_state" },
{ .bus = "ide", .file = "detach_state" },
{ .bus = "pci", .file = "vendor" },
+ { .bus = "platform", .file = "detach_state" },
{ NULL }
};
struct bus_file *busfile;
{ NULL }
};
struct bus_file *busfile;
/* wait for a bus specific file to show up */
loop = WAIT_MAX_SECONDS * WAIT_LOOP_PER_SECOND;
while (--loop) {
/* wait for a bus specific file to show up */
loop = WAIT_MAX_SECONDS * WAIT_LOOP_PER_SECOND;
while (--loop) {
for (busfile = bus_files; busfile->bus != NULL; busfile++) {
if (strcmp(device_dev->bus, busfile->bus) == 0) {
for (busfile = bus_files; busfile->bus != NULL; busfile++) {
if (strcmp(device_dev->bus, busfile->bus) == 0) {
dbg("looking at bus '%s' for specific file '%s'", device_dev->bus, busfile->file);
if (sysfs_get_device_attr(device_dev, busfile->file) != NULL) {
dbg("bus '%s' specific file '%s' found", device_dev->bus, busfile->file);
return 0;
}
dbg("looking at bus '%s' for specific file '%s'", device_dev->bus, busfile->file);
if (sysfs_get_device_attr(device_dev, busfile->file) != NULL) {
dbg("bus '%s' specific file '%s' found", device_dev->bus, busfile->file);
return 0;
}
- if (busfile->bus == NULL) {
- info("error: unknown bus, update the build-in list '%s'", device_dev->bus);
- return -1;
- }
+ if (found == 0) {
+ info("error: unknown bus, please report to "
+ "<linux-hotplug-devel@lists.sourceforge.net> '%s'", device_dev->bus);
+ return -1;
+ }
+ usleep(1000 * 1000 / WAIT_LOOP_PER_SECOND);
}
dbg("error: getting bus '%s' specific file '%s'", device_dev->bus, busfile->file);
}
dbg("error: getting bus '%s' specific file '%s'", device_dev->bus, busfile->file);
+ init_logging("wait_for_sysfs");
+
if (argc != 2) {
dbg("error: subsystem");
return 1;
if (argc != 2) {
dbg("error: subsystem");
return 1;
+ /* we only wait on an add event */
if (strcmp(action, "add") != 0)
return 0;
if (strcmp(action, "add") != 0)
return 0;
class_dev = sysfs_open_class_device_path(filename);
if (class_dev)
break;
class_dev = sysfs_open_class_device_path(filename);
if (class_dev)
break;
+
+ usleep(1000 * 1000 / WAIT_LOOP_PER_SECOND);
}
if (class_dev == NULL) {
dbg("error: getting class_device");
}
if (class_dev == NULL) {
dbg("error: getting class_device");
goto exit;
}
dbg("class_device opened '%s'", filename);
goto exit;
}
dbg("class_device opened '%s'", filename);
- wait_for_class_device_attributes(class_dev);
+ if (wait_for_class_device_attributes(class_dev) != 0) {
+ rc = 4;
+ goto exit_class;
+ }
+ /* skip devices without /device-link */
if (class_device_expect_no_device_link(class_dev)) {
if (class_device_expect_no_device_link(class_dev)) {
- dbg("no device symlink expected");
- sysfs_close_class_device(class_dev);
- goto exit;
+ dbg("no device symlink expected for '%s', ", class_dev->name);
+ goto exit_class;
}
/* the symlink may be on the parent device */
}
/* the symlink may be on the parent device */
}
if (device_dev == NULL) {
dbg("error: getting /device-device");
}
if (device_dev == NULL) {
dbg("error: getting /device-device");
- sysfs_close_class_device(class_dev);
}
dbg("device symlink found pointing to '%s'", device_dev->path);
/* wait for the bus value */
}
dbg("device symlink found pointing to '%s'", device_dev->path);
/* wait for the bus value */
- if (wait_for_bus_device(device_dev) != 0)
- rc = 6;
- sysfs_close_class_device(class_dev);
+ if (class_device_expect_no_bus(class_dev)) {
+ dbg("no bus device expected for '%s', ", class_dev->classname);
+ } else {
+ if (wait_for_bus_device(device_dev) != 0)
+ rc = 6;
+ }
- /* finished */
- goto exit;
+exit_class:
+ sysfs_close_class_device(class_dev);
} else if ((strncmp(devpath, "/devices/", 9) == 0)) {
/* open the path we are called for */
} else if ((strncmp(devpath, "/devices/", 9) == 0)) {
/* open the path we are called for */
device_dev = sysfs_open_device_path(filename);
if (device_dev)
break;
device_dev = sysfs_open_device_path(filename);
if (device_dev)
break;
+
+ usleep(1000 * 1000 / WAIT_LOOP_PER_SECOND);
}
if (device_dev == NULL) {
dbg("error: getting /device-device");
}
if (device_dev == NULL) {
dbg("error: getting /device-device");
goto exit;
}
dbg("device_device opened '%s'", filename);
/* wait for the bus value */
if (wait_for_bus_device(device_dev) != 0)
goto exit;
}
dbg("device_device opened '%s'", filename);
/* wait for the bus value */
if (wait_for_bus_device(device_dev) != 0)
sysfs_close_device(device_dev);
sysfs_close_device(device_dev);
- /* finished */
- goto exit;
-
} else {
dbg("unhandled sysfs path, no need to wait");
}
exit:
if (rc == 0)
} else {
dbg("unhandled sysfs path, no need to wait");
}
exit:
if (rc == 0)
- info("result: waiting for sysfs successful '%s'", devpath);
+ dbg("result: waiting for sysfs successful '%s'", devpath);
- info("result: waiting for sysfs failed '%s'", devpath);
+ info("error: wait_for_sysfs needs an update to handle the device '%s' "
+ "properly, please report to <linux-hotplug-devel@lists.sourceforge.net>",
+ devpath);