Re: udev and dbus

[Olaf Hering <olh@suse.de>]

 On Thu, Mar 11, Kay Sievers wrote:

> On Thu, 2004-03-11 at 19:32, Greg KH wrote:
> > On Thu, Mar 11, 2004 at 07:12:55PM +0100, Kay Sievers wrote:
> > > 
> > > I expect lot of lazy people to put their scripts in there, cause it's so
> > > comfortable to live inside the serialized hotplug events.
> > 
> > But people want to know about the device node creation and removal, they
> > don't care about the "raw" hotplug events.  They want udev to handle the
> > raw hotplug events for them.
> 
> I don't agree.
> 
> > > Something like this:
> > > "Nice, I can mount my USB-stick with udev.d/ right after udev has
> > > created my partition node, cause the scsi.agent was too fast..."
> > 
> > No, how can scsi.agent know about what the device node name was?  It
> > can't.
> 
> udevinfo! That's what it's made for :)

what about nodes="`UDEV_PRINT_NODES=1 udev $1`"
which fills $nodes will the name and the symlinks for DEVPATH? This can
work for add and remove events.

diff -pur udev-klibc/udev-add.c udev-klibc-debug/udev-add.c
--- udev-klibc/udev-add.c	2004-03-14 20:01:19.000000000 +0100
+++ udev-klibc-debug/udev-add.c	2004-03-14 20:56:15.573334811 +0100
@@ -197,6 +197,7 @@ static int create_node(struct udevice *d
 	int tail;
 	char *pos;
 	int len;
+	char *printnodes;
 
 	strfieldcpy(filename, udev_root);
 	strfieldcat(filename, dev->name);
@@ -217,6 +218,8 @@ static int create_node(struct udevice *d
 		return -EINVAL;
 	}
 
+	printnodes = getenv(UDEV_PRINT_NODES);
+
 	/* create parent directories if needed */
 	if (strrchr(dev->name, '/'))
 		create_path(filename);
@@ -257,6 +260,8 @@ static int create_node(struct udevice *d
 		unlink_entry(filename);
 		info("creating device node '%s'", filename);
 		make_node(filename, dev->major, dev->minor, dev->mode, uid, gid);
+		if (printnodes)
+			fprintf(stdout, "%s", filename);
 	} else {
 		info("creating device node '%s', major = '%d', minor = '%d', "
 		     "mode = '%#o', uid = '%d', gid = '%d'", filename,
@@ -316,8 +321,13 @@ static int create_node(struct udevice *d
 			if (retval != 0)
 				dbg("symlink(%s, %s) failed with error '%s'",
 				    linktarget, filename, strerror(errno));
+			else 
+			if (printnodes)
+				fprintf(stdout, " %s", filename);
 		}
 	}
+	if (printnodes)
+		fprintf(stdout, "\n");
 
 	return retval;
 }
diff -pur udev-klibc/udev-remove.c udev-klibc-debug/udev-remove.c
--- udev-klibc/udev-remove.c	2004-03-14 20:01:19.000000000 +0100
+++ udev-klibc-debug/udev-remove.c	2004-03-14 20:56:20.710112187 +0100
@@ -73,11 +73,14 @@ static int delete_node(struct udevice *d
 	int retval;
 	int i;
 	char *pos;
+	char *printnodes;
 	int len;
 
 	strfieldcpy(filename, udev_root);
 	strfieldcat(filename, dev->name);
 
+	printnodes = getenv(UDEV_PRINT_NODES);
+
 	info("removing device node '%s'", filename);
 	retval = unlink(filename);
 	if (errno = ENOENT)
@@ -88,6 +91,9 @@ static int delete_node(struct udevice *d
 		return retval;
 	}
 
+	if (printnodes)
+		fprintf(stdout, "%s", filename);
+
 	/* remove partition nodes */
 	if (dev->partitions > 0) {
 		info("removing partitions '%s[1-%i]'", filename, dev->partitions);
@@ -116,10 +122,14 @@ static int delete_node(struct udevice *d
 				filename, strerror(errno));
 			return retval;
 		}
+		if (printnodes)
+			fprintf(stdout, " %s", filename);
 		if (strchr(dev->symlink, '/')) {
 			delete_path(filename);
 		}
 	}
+	if (printnodes)
+		fprintf(stdout, "\n");
 
 	return retval;
 }
diff -pur udev-klibc/udev.h udev-klibc-debug/udev.h
--- udev-klibc/udev.h	2004-03-14 20:01:19.000000000 +0100
+++ udev-klibc-debug/udev.h	2004-03-14 20:55:53.998269342 +0100
@@ -103,6 +103,8 @@ do { \
 	    pos = pos + len + strspn(pos, separator), len = strcspn(pos, separator)) \
 		if (len > 0)
 
+#define UDEV_PRINT_NODES "UDEV_PRINT_NODES"
+
 static inline char *get_action(void)
 {
 	char *action;
-- 
USB is for mice, FireWire is for men!

sUse lINUX ag, n√úRNBERG


-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id\x1470&alloc_id638&opÃk
_______________________________________________
Linux-hotplug-devel mailing list  http://linux-hotplug.sourceforge.net
Linux-hotplug-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-hotplug-devel