[PATCH] quick fix for libsysfs bus

There seems to be a
mismatch in udev and in libsysfs as to what to expect if the mnt point
has a slash on the end or not. If I use the included patch, it breaks
something in udev. If I patch sysfs_get_mnt_path I break udev as well
because what you're expecting. I need to sit down and go through
the library and creaate a rule as to trailing slashes. Adding the env
brought this to light.

[PATCH] libsysfs changes for sysfsutils 0.3.0

Here's the patch to up the library to the sysfsutils-0_3_0 level. The
following changes:

1) adds class name to sysfs_class_device structure
2) adds bus to sysfs_device
3) gets rid of code that made assumptions as to bus addresses being
unique across buses, which isn't the case.

I still owe you:

1) change getpagesize->sysconf. This is in the CVS tree and part of other
changes we're currently testing. Patch will follow.
2) you need a function to get a sysfs_class_device's parent. We hadn't
considered class devices to have parents, the one example of a multilevel
is the block class. We will add this function and send the patch to you.

[PATCH] add test and documentation for new %D devfs format modifier

[PATCH] Add format modifier for devfs like naming

On Monday 24 November 2003 01:29, Greg KH wrote:
> I think with the ability to capture the output of the CALLOUT rule,
> combined with the ability to put format modifiers in the CALLOUT program
> string, we now have everything in place to emulate the existing devfs
> naming scheme.  Anyone want to verify this or not?

I would prefer to have the ability of creating partition nodes in devfs
style built-in to udev. Devfs used to call the whole disk e.g.
"/dev/dasd/0123/disk" and the partitions "/dev/dasd/0123/part[1-3]".
This can obviously be done with a CALLOUT rule, but its common enough
to make it a format modifier. AFAIK, this scheme has been used for
ide, scsi and dasd disks, which is about 99% of all disks ever connected
to Linux.

[PATCH] more config file parsing robustness

udev kept on segfaulting when it was in use, and not having the time
(and building it with DEBUG=true showing nothing), I have not tracked it
until tonight.  Seems like I made a type-o, and forgotten the ':'
between one line's group and permission parameters.  Attached patch
should stop the segfault, and warn at that at least.

[PATCH] changed the default location of the database to /udev/.udev.tdb to be LSB compliant

Finally the Debian people can get off my back...

[PATCH] udev.8 tweak numeric id text

04-udev.8-tweak-numeric-id-text.diff
  o change "return value" to "returned string"
  o add textual owner/group example for udev.permissions
  o mention klibc compile as special case for udev.permissions
  o remove duplicated numeric owner clause

[PATCH] udev-test.pl add subdir test

03-udev-test.pl-add-subdir-test.diff
  o duplicate existing test and change it to explicitely test the subdir handling

[PATCH] namedev.c strcat tweak

02-namedev.c-strcat-tweak.diff
  o cat the substitution to the already known end of the string instead of searching it another time

[PATCH] overall whitespace + debug text conditioning

01-overall-whitespace+debug-text-conditioning.diff
  o cleanup whitespace
  o clarify a few comments
  o enclose all printed debug string values in ''

[PATCH] klibc makefile fixes

[PATCH] get rid of functions in klibc_fixups that are now in klibc

[PATCH] sync up with the 0.84 version of klibc

[PATCH] fix udev init.d script to handle all class devices in sysfs.

[PATCH] fix the test.block and test.tty scripts due to their moveing. Also add a test.all script.

[PATCH] udev-test.pl - tweaks

I found two missing characters and optimized $PWD.

[PATCH] 007_bk version change to Makefile.

[PATCH] v007 release

[PATCH] v007 Changelog

[PATCH] update default config file with a CALLOUT rule, and more documentation.

[PATCH] updated the man page with the latest format specifier changes.

[PATCH] added ability to put format specifiers in the CALLOUT program string.

[PATCH] tweak udev-test.pl to report '0' errors if that's what happened.

[PATCH] - format char for CALLOUT output

here is a patch for inserting the callout output into NAME=.
ID= supports the usual wildcard to compare with the output.

I've moved all wildcard matching to a function cause this was the third occurrence.
Also attached is the last whitespace cleanup and debug text corrections.
The callout patch depends on the whitespace patch.

CALLOUT, BUS="usb", PROGRAM="/bin/echo -n return", ID="ret*", NAME="webcam-%c-"

results in:

Nov 21 17:33:51 pim udev[20399]: get_major_minor: found major = 81, minor = 0
Nov 21 17:33:51 pim udev[20399]: exec_callout: callout to '/bin/echo -n return'
Nov 21 17:33:51 pim udev[20399]: exec_callout: callout returned 'return'
Nov 21 17:33:51 pim udev[20399]: get_attr: substitute callout output 'return'
Nov 21 17:33:51 pim udev[20399]: udev_add_device: name = webcam-return-
Nov 21 17:33:51 pim udev[20399]: create_node: mknod(/udev/webcam-return-, 020660, 81, 0)

[PATCH] more namedev whitespace cleanups

attached is the last whitespace cleanup and debug text corrections.

[PATCH] only build klibc_fixups.c if we are actually using klibc.

[PATCH] add support for string group and string user names in udev.permissions.

This only works if you are using glibc.

Patch based on patch from Marco d'Itri <md@Linux.IT>

[PATCH] add getgrnam and getpwnam to klibc_fixups files.

Hopefully the klibc_fixups code will not be needed eventually.

[PATCH] fix segfault in parsing bad udev.permissions file

[PATCH] remove Makefile.klibc

[PATCH] static klibc udev does not link against crt0.o

 On Wed, Nov 19, Greg KH wrote:

> > I did 'make KLIBC=true' in the current bk tree.
>
> try 'make -f Makefile.klibc' in the current tree.  For some reason I
> couldn't figure out how to have Makefile work for both KLIBC=true and
> KLIBC=false.  But I didn't try too hard :)

I dont understand that.

please do rm -f Makefile.klibc; apply this patch and tell me what fails.
works for me.

[PATCH] add udev-test perl script from Kay Sievers <kay.sievers@vrfy.org> which blows away my puny shell scripts.

[PATCH] added debian's version of udev.permissions.

Hm, should probably move all of the config file stuff to a new directory soon...

[PATCH] support arguments in callout exec

here is argument support for CALLOUT exec:

CALLOUT, PROGRAM="/bin/echo -n xxx", BUS="usb", ID="xxx", NAME="webcam%n"

results in:

Nov 20 02:35:20 pim udev[30422]: get_major_minor: found major = 81, minor = 0
Nov 20 02:35:20 pim udev[30422]: exec_callout: callout to /bin/echo -n xxx
Nov 20 02:35:20 pim udev[30422]: exec_callout: callout returned 'xxx'
Nov 20 02:35:20 pim udev[30422]: get_attr: kernel number appended: 0

The feature is really nice, but the maximum argument count is hard coded to 8.

[PATCH] namedev.c - change order of fields in CALLOUT

I want to bring the CALLOUT field ordering in line with the other
methods, cause the current parsing relies on the ordering it's good
to have it like the others. The BUS= is now the first expected field.

Also made the last two remaining field names to uppercase and the man page
callout example is updated.

[PATCH] namedev.c whitespace + debug text cleanup

here is mainly a whitespace cleanup for namedev.c. I changed the
dbg_parse() output a bit for better readability:

current:

Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER temp='/2-1.1' id='00:07.1'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER temp='/2-1' id='00:07.1'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER temp='/2-1.1' id='00:0b.0'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER temp='/2-1' id='00:0b.0'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:00:59 pim udev[25582]: do_number: NUMBER temp='/2-1.1' id='2-1.1'
Nov 19 19:00:59 pim udev[25582]: do_number: device id '2-1.1' becomes 'webcam%n' - owner='', group ='', mode=0

becomes:

Nov 19 19:23:40 pim udev[26091]: do_number: search '00:07.1' in '/2-1.1', path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:23:40 pim udev[26091]: do_number: search '00:07.1' in '/2-1', path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1'
Nov 19 19:23:40 pim udev[26091]: do_number: search '00:0b.0' in '/2-1.1', path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:23:40 pim udev[26091]: do_number: search '00:0b.0' in '/2-1', path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1'
Nov 19 19:23:40 pim udev[26091]: do_number: search '2-1.1' in '/2-1.1', path='/sys/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1.1'
Nov 19 19:23:40 pim udev[26091]: do_number: found id '2-1.1', 'video0' becomes 'webcam%n' - owner='', group ='', mode=0

[PATCH] man page with udev.permissions wildcard

here is the permissions wildcard text for the man page.

[PATCH] change to 006_bk version

[PATCH] v006

[PATCH] v006 changelog

[PATCH] add another line to udev.permissions in the proper format.

[PATCH] tweak replace_test

[PATCH] fix permissions to work properly now.

Forgot about umask in mknod...

[PATCH] add real udev.permissions file to test directory.

[PATCH] fix namedev.c to build with older version of gcc.

[PATCH] apply permissions.conf support for wildcard and default name

Permissions given in udev.permissions are not applied if no METHOD from
udev.config is found. I've added do_kernelname() to scan for known
permissions if we only use the default method.

Simple support for wildcards is also added:

#name:user:group:mode
hdb*:2702:2702:0660

results in:

drwxr-xr-x    2 root     root          240 Nov 19 03:45 .
drwxr-xr-x   23 root     root          528 Nov 17 03:36 ..
brw-r--r--    1 root     root       3,   0 Nov 19 03:45 hda
brw-r--r--    1 root     root       3,   1 Nov 19 03:45 hda1
brw-r--r--    1 root     root       3,   2 Nov 19 03:45 hda2
brw-r--r--    1 root     root       3,   4 Nov 19 03:45 hda4
brw-r-----    1 kay      kay        3,  64 Nov 19 03:45 hdb
brw-r-----    1 kay      kay        3,  65 Nov 19 03:45 hdb1
brw-r--r--    1 root     root      22,   0 Nov 19 03:45 hdc
crw-r--r--    1 root     root      81,   0 Nov 19 03:34 webcam0

[PATCH] man page with included placeholder list

  mention the printf-like support for string substitution in the NAME field

[PATCH] add dumb test for all of the different modifiers.

[PATCH] update the TODO list with more items that people can easily do.

[PATCH] faster test scripts

I've attached a patch against 005 for both the block and tty scripts.  I
didn't bother running udev backgrounded, since as you say, the new code
runs a lot faster.

[PATCH] move the test.block and test.tty scripts to the test/ directory

[PATCH] add remove actions to the test scripts

[PATCH] turn DEBUG_PARSER off by default.

[PATCH] add some documentation for the %b modifier to the default config file.

[PATCH] fix make install rule for when the udev symlink is already there.

[PATCH] more robust config file parsing in namedev.c

After getting a number of different crashes for udev reading broken
udev.config files, I decided to try to make the parser a little
more robust.

The behaviour is changed to stop reading the configuration file
and logging the broken entry instead of silently ignoring it (is
that good? It's easy to just print and continue).
All strcpy()'s to a fixed length string are now implicitly limited
to the bounds of the target string.

I kept the -ENODEV return code for now, not sure if there should be
different ones.

[PATCH] add bus id modifier

On Tuesday 18 November 2003 02:14, Greg KH wrote:
> On Mon, Nov 17, 2003 at 06:33:32PM +0100, Arnd Bergmann wrote:
> > That would at least be part of the solution I'm looking for. How about
> > extra format characters for bus_id and for the result of a callout
> > program?
>
> Sure, I can see the use for that.  Want to send a patch?  :)

> Take a look at the current bk tree (which has moved to
> bk://linuxusb.bkbits.net/udev/ )  I've made finding that device a lot
> easier now, and it works for all rule types.

Great, just what I was missing. I didn't see the any link to the bk
repository. Here's the patch for the bus_id. I'll need to think about
the handling of callout results a bit more.

[PATCH] change release target in makefile.

[PATCH] change debug level on printf values for now.

[PATCH] updated demo config file.

[PATCH] add some documentation of the modifiers to the default config file.

[PATCH] add demo config file.

[PATCH] add printf option to label test to verify it works.

[PATCH] fix up printf-like functionality due to previous changes.

[PATCH] implement printf-like placeholder support for NAME

> Problem is, if you use the LABEL rule to match a device, like a SCSI
> vendor, then all of the partitions, as well as the main block device,
> will end up with the same name.  That's why I added the "add the number"
> hack to the LABEL rule.
>
> So yes, your patch is correct in that we shouldn't always be adding the
> number to any match for LABEL (like for char devices), but if we do
> that, then we break partitions.  Your '%' patch fixes this, but I'd just
> like to extend it a bit.  Let me see what I can come up with...

Oh, I see. Do you mean something like this:

LABEL, BUS="usb", model="Creative Labs WebCam 3", NAME="webcam%n-%M:%m-test"

results in: "webcam0-81:0-test"

Nov 15 16:51:53 pim udev[16193]: get_class_dev: looking at /sys/class/video4linux/video0
Nov 15 16:51:53 pim udev[16193]: get_class_dev: class_dev->name = video0
Nov 15 16:51:53 pim udev[16193]: get_major_minor: dev = 81:0
Nov 15 16:51:53 pim udev[16193]: get_major_minor: found major = 81, minor = 0
Nov 15 16:51:53 pim udev[16193]: udev_add_device: name = webcam0-81:0-test
Nov 15 16:51:53 pim udev[16193]: create_node: mknod(/udev/webcam0-81:0-test, 020666, 81, 0)

  implement printf-like placeholder support for NAME
  %n-kernel number, %M-major number, %m-minor number

[PATCH] get the major/minor number before we name the device.

Will come in handy later...

[PATCH] add scsi_id "extra" program from Patrick Mansfield <patmans@us.ibm.com>

[PATCH] Add multipath "extra" program from Christophe Varoqui, <christophe.varoqui@free.fr>

[PATCH] trailing whitespace cleanups.

[PATCH] splig LABEL and NUMBER into separate functions.

Now items are processed in priority order, fixing that bug.

[PATCH] add TOPO regression test.

[PATCH] move TOPOLOGY rule to it's own function.

[PATCH] fix bug where NUMBER and TOPOLOGY would not work for partitions.

[PATCH] clean up the way we find the sysdevice for a block device for namedev.

[PATCH] updated label test script (tests for partitions now.)

[PATCH] split REPLACE and CALLOUT into separate functions

Finally starting to process this in priority order, not config file order.

[PATCH] add debug line for REPLACE call.

[PATCH] add replace test

[PATCH] add more sysfs test tree files.

[PATCH] change UDEV_SYSFS_PATH environment variable due to libsysfs change.

[PATCH] fix bug in klibc's isspace function.

[PATCH] patch for libsysfs sysfs directory handling

Here's a quick patch to:

1) Add an environment variable "SYSFS_PATH" that libsysfs will check
for when getting mount point before searching system's sysfs mount
point.

2) A quick fix to sysfs_get_link where the bug was.

I have tested this out with libsysfs and with udev. I couldn't test
with klibc because I haven't got the tree to build with klibc without
my changes either. I made the link but get an error finding
linux/linits.h. I will figure that out.

Please have a look at the patch. If it's agreeable, please test it. I
really want to add some generic path manipulation functions for the
sysfs_get_link error, rather than my patch's hack. But, I haven't had
time yet to do that. You really sounded like you needed this for
testing, so I'm sending it out to you. I should probably add a
function to set the env variable(?).

[PATCH] fix udev-add.c to build properly with older versions of gcc.

[PATCH] more manpage tweaks

  man page style fixes
  present the tiny udev in bold font :)

[PATCH] add support for subdirs

  support subdirectory creation/removal for NAME="/devfs/is/crazy/video0"
  create parent subdirs for device node if needed
  remove subdirs when last node is removed

[PATCH] add uid/gid to nodes

  set uid/gid of node specified in udev.permissions
  only numeric id's are supported cause we can't resolve with
  klibc or libc before real /etc is mounted

[PATCH] add prototype for ftruncate to klibc

[PATCH] Remove a few items from the TODO list that are already done.

[PATCH] DESTDIR for udev

[PATCH] version number to 005_bk

[PATCH] udev init script

I integrated udev with Fedora Core.  The main piece is simply building
/udev on boot, since we don't have an initramfs yet. We should also
clear out /udev on shutdown, for /udev directories mounted on persistent
media.

The attached script goes in /etc/init.d

Then do "chkconfig --add udev"

And the rest is handled automatically.  I made it for Fedora but it will
probably work, with little change, on any Linux system.

Right now it only does sysfs-based discovery of block and tty devices,
since those are the only types of devices I have on my system.  There is
a TODO in the script where we would add the other device types.

[PATCH] pull some klibc stuff into the make Makefile to try to stay in sync.

[PATCH] klibc build fixes

Still can't seem to build both klibc and glibc versions from same makefile :(

[PATCH] Fixup path for kernel includes when building with klibc.

Fixup path for kernel includes when building with klibc.

klibc expects the symlink klibc/linux to point to a relatively up
to date tree, use -Iklibc/linux/include to make linux/ and asm/
includes resolve properly, as these won't exist in the regular
klibc/klibc/include location.

[PATCH] 005 release

[PATCH] add klibc linux symlink info to the README

[PATCH] get 'make release' to work properly again.

[PATCH] udev: sleep_for_dev() bits

OK, I fixed that bug you hinted at earlier in my previous
sleep_for_dev() patch.  I am sure you fixed it, but here we go
nonetheless, just in case.

I actually changed it up a bit.  It is probably faster to count down
from SECONDS_TO_WAIT_FOR_DEV than count up.

I also made the lone 'path' argument const, since it can be.

Some other misc. bits, too.

[PATCH] added README info for how to build using klibc.

[PATCH] turn off debugging if we are building with klibc