[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

[PATCH] turn off debugging in namedev

[PATCH] added vsyslog support to klibc.

[PATCH] add ftruncate to klibc.

probably got it wrong too...

[PATCH] klibc specific tweaks

[PATCH] libsysfs does not need mntent.h in it's header file.

[PATCH] klibc makefile changes.

[PATCH] build tdb and libsysfs from the same makefile as udev.

This fixes problem of libsysfs and tdb getting build with different gcc options
as the rest of udev.

[PATCH] udev build tweaks to tdb's spinlock code

[PATCH] udev-add build cleanups for other libc versions.

[PATCH] tweak tdb to build within udev better.

[PATCH] make libsysfs spit debug messages to the same place as the rest of udev.

[PATCH] make libsysfs build cleanly

[PATCH] added klibc version 0.82 (cvs tree) to the udev tree.

Not hooked up to the build yet.

[PATCH] makefile fix for now.

[PATCH] hm, makefile bug with so many files... will fix later...

[PATCH] regression tests starting to be added.

[PATCH] fix LABEL bug for device files (not class files.)

[PATCH] got rid of struct device_attr

namedev.c is still a mess, that's up next after testing...

[PATCH] rename namedev.permissions and namedev.config to udev.permissions and udev.config

the namedev name didn't really make much sense anymore...

[PATCH] more warning flags to the build.

[PATCH] namedev.c comments + debug patch

and a patch for namdev.c comments + debug
o correct 'device device'
o correct permission file to config file

[PATCH] fix dbg line in namedev.c

Thanks to Kay Sievers <kay@vrfy.org> for pointing this out.

[PATCH] man page update

here a new patch for the man page:
o namdev.permissions added
o corrected a few typos
o corrected text for config line format.
  owner, group mode are only in namedev.permissions
  (don't know what i was dreaming, while i wrote this :))

[PATCH] udev: another canidate for static

Another one of those "trivial patches so everything must be good"

sleep_for_dev() isn't exported or defined in a header, so it can be
marked 'static'

[PATCH] more overrides of config info with env variables if in test mode.

[PATCH] Fix bug causing udev to sleep forever waiting for dev file to show up.

Now we will timeout properly.

[PATCH] change version to 004_bk

[PATCH] make config files, sysfs root, and udev root configurable from config variables

This will make running tests a lot simpler.

[PATCH] 004 changelog entrys

[PATCH] 004 release

[PATCH] new version of libsysfs patch

Here's the patch applying the latest libsysfs.
- adds the latest libsysfs code to udev
* new code includes dlist implementation, a generic linked list
implementation. Needed our own because LGPL
* rearranged structures
* provided more functions for accessing directory and attributes
- gets rid of ->directory->path references in namedev.c
- replaces sysfs_get_value_from_attributes with sysfs_get_classdev_attr

[PATCH] man file update

changes:
o CALLOUT method included
o required keys for every match method listed
o keywords changed to uppercase in EXAMPLE

[PATCH] major database cleanups

Now we standardise on a struct udevice to pass around, and store in the
database.  This cleaned up the database code a lot.

[PATCH] Changed test.block and test.tty to take ACTION from the command line.

[PATCH] don't sleep if 'dev' file is already present on device add.

instead sleep for up to 10 seconds waiting for it to show up.