[PATCH] udevsend fallback

On Tue, May 11, 2004 at 04:54:44PM -0700, Greg KH wrote:
> On Tue, May 11, 2004 at 01:16:41PM +0200, Kay Sievers wrote:
> > Hi,
> > the execution of udev depends on the proper fuction of udevd, the
> > serializing daemon. If we can't connect to udevd within a 20 second we
> > give up and the request to create a node is lost. Hope this never happens,
> > but a broken udevd may prevent udev from working.
> >
> > What do you think? Should we call the udev binary directly from udevsend
> > instead of discarding the event? This way we would create the node, regardless
> > of the state of udevd. It would be 20 seconds later and maybe not in the right
> > sequence order - but the node will propably be there.
> >
> > Does it sound sane? What do you think?
>
> That sounds like a good "failsafe" thing to do.

Here we go:

Add a fallback udev call to udevsend. If udevsend is unable to send the
event to udevd, we call the udev binary instead of doing nothing and exiting.

[PATCH] add volume_id support to the udev.spec file

[PATCH] remove dbus and selinux stuff from the udev.spec file.

[PATCH] delete udev_selinux as it doesn't work properly and is the wrong way to do it...

[PATCH] Deleted the udev_dbus extra as it didn't really work properly and HAL has a real solution now.

[PATCH] udev_volume_id update

Here is a update to extras/volume_id/*

o The device is now specified by the DEVPATH in the environment,
  it's no longer needed to pass the major/minor to the callout.

o leading spaces and slashes are now removed from the returned string
  and spaces are replaced by underscore, to not to confuse udev.

o Arnd Bergmann <arnd@arndb.de> provided the code to recognize s390
  dasd disk labels. The -d switch tries to read the main block device
  instead of the partition.

[PATCH] add udev.permissions.slackware file.

From Lucas de Souza Santos <lucasdss@yahoo.com.br>

[PATCH] udev callout for reading filesystem labels

On Fri, Apr 30, 2004 at 03:29:54PM -0700, Greg KH wrote:
> On Thu, Apr 29, 2004 at 11:04:46PM +0200, Kay Sievers wrote:
> > Hi,
> > here is a small udev toy, which enables udev to name partitions by
> > its filesystem label or uuid's.
> >
> > The following udev rule:
> >
> >   KERNEL="sd*", PROGRAM="/sbin/udev_volume_id -M%M -m%m -u", SYMLINK="%c"
> >
> > creates a symlink with the uuid read from the filesystem. If no label or
> > uuid is found the program exits with nonzero and the rule will fail.
> >
> > ext2, ext3, reiserfs, xfs, jfs, vfat, msdos volume labels are supported,
> > ntfs and swap partitions can be recognized.
> >
> > It's possible to compile with klibc and the static binary takes 13kb.
>
> Very nice, I was wondering who was going to use that library to make
> such a tool.  This is even better as we can use klibc for it.

Here is a update, which supports iso9660 and udf labels.
Not very useful in the udev case, but I've added it for hal,
so we just catch up with the latest version.

[PATCH] udev callout for reading filesystem labels

here is a small udev toy, which enables udev to name partitions by
its filesystem label or uuid's.

The following udev rule:

  KERNEL="sd*", PROGRAM="/sbin/udev_volume_id -M%M -m%m -u", SYMLINK="%c"

creates a symlink with the uuid read from the filesystem. If no label or
uuid is found the program exits with nonzero and the rule will fail.

ext2, ext3, reiserfs, xfs, jfs, vfat, msdos volume labels are supported,
ntfs and swap partitions can be recognized.

It's possible to compile with klibc and the static binary takes 13kb.

[PATCH] udevstart: close open directories.

Based on a munged patch from Olaf Hering <olh@suse.de>

[PATCH] udev default config layout changes

Here we catch up, after the default config changes.

o the man page is updated to reflect the new default config

o /etc/udev/rules.d/ + permissions.d/ dirs are created now

o udev.rules is installed in /etc/udev/rules.d/50-udev.rules
  so the user can easily order the files by prepending a number.
  (RedHat has the same name in the last rpm.)

o defined directory names in the Makefile are all without slashes now,
  not the first half with and the remaining without.

o all binaries are uninstalled now

o leading slashes in config values are now removed or prepended while the
  config is parsed, so we are more robust if the usere changes something.

o replaced the macros from udev_config.c with real code, cause we can
  skip if the value matches and not useless iterate over the remaining
  fields.

o config parsing errors are logged with info() now, fixes the bug where
  we report a error with debug_parse(), even when there isn't one

[PATCH] v025 release

[PATCH] make dev.d call each directory in the directory chain of the device name, instead of just the whole name.

This lets /etc/dev.d/sound/ work better.

[PATCH] add devd_test script.

[PATCH] add more permissions based on SuSE's recommendations.

[PATCH] gcov for udev

Sorry for the late email regarding the gcov code coverage information
for udev.  For those of you who have not yet noticed, udev can now be
compiled with gcov support and provide code coverage analysis.  All
pertinent scripts and information can be found in the udev tree.  Please
refer to "README-gcov_for_udev" for detailed information on compiling
gcov into udev and obtaining code coverage analysis.  I've enclosed a
patch that updates the README-gcov_for_udev.  Also, "udev-test.pl" in
udev/test/ was expanded to test symlinks, permissions, and some lack of
node creation a little more thoroughly.  All comments and feedback would
be greatly welcomed.  Also, any extra testing would be appreciated.
Thanks,

[PATCH] fix test regressions

On Sat, Apr 17, 2004 at 03:30:29AM +0200, Kay Sievers wrote:
> On Sat, Apr 17, 2004 at 02:04:55AM +0200, Kay Sievers wrote:
> > On Fri, Apr 16, 2004 at 04:04:42PM -0700, Greg KH wrote:
> > > Oh, and if you run the latest udev_test.pl, we have a bunch more tests,
> > > including a few that fail, if you were looking for something to do :)
> >
> > Will do it. We need to change apply_format(). I tries to expand the '%%'
> > with the next iteration over the string and removes the '%'.

The tests are all successful now.
If this patch breaks something else, we simply have too few tests :)

[PATCH] added rules for tun and raw devices

[PATCH] add udev conf.d file.

[PATCH] Switch the default config to point to a directory for the rules and permission files.

[PATCH] update the Red Hat .dev files to work on other distros

[PATCH] add dbus.dev, pam_console.dev and selinux.dev files for /etc/dev.d/default/ usage

Thanks to Red Hat for them.

[PATCH] add hints for red hat users from Leann Ogasawara <ogasawara@osdl.org>

[PATCH] add scripts to run gcov for udev from Leann Ogasawara <ogasawara@osdl.org>

[PATCH] udev_selinux changes

Here is the fix for extras/seliux/*
 o install the binary in /sbin/
 o add symlink to /etc/dev.d/ with suffix .dev
 o removed the undefined udev_log variable
 o fixed compiler warnings

[PATCH] Update writing udev rules docs

Here's an update for the writing udev rules document.
- Minor corrections/clarifications
- Added info about using NAME{all_partitions}
- Added more info about udevinfo, simplifying the rule-writing process

You can ignore the diff I sent you yesterday - according to the 20040415 bk
snapshot on codemonkey.org, you haven't applied it yet. This patch
incorporates that update, and some other changes I just made.

[PATCH] udevd test script

Thanks to Yin, Hu <hu.yin@intel.com>, who made a nice perl script to test the
expected behavior of the udevd sequence number handling. The test sends
different SEQNUM sequences to udevd, while analyzing the reordering and timeout
handling of udevd.

[PATCH] devfs.sh-ide-floppy

[PATCH] Fix build process for users who have LC_ALL set to a non-english language.

Many thanks to jack_mort <mort_jack@yahoo.fr> for helping track this down.

[PATCH] Added expanded tests to the test framework from Leann Ogasawara <ogasawara@osdl.org>

[PATCH] added execelent "writing udev rules" document from Daniel Drake <dan@reactivated.net>

[PATCH] udev_dbus changes

On Sun, Apr 11, 2004 at 03:51:07AM +0200, Kay Sievers wrote:
> Here we change extras/dbus/* to
>   o install the binary in /etc/dev.d
>   o append .dev to the binary
>   o add David's copyright
>   o add the listener script to watch the sent dbus messages
>   o removed the undefined udev_log variable
>   o switch printf() to dbg()

Here we install the binary in /usr/sbin/ and symlink it to
/etc/dev.d/ with the .dev suffix.

[PATCH] added rule to put USB printers in their proper places.

Thanks to Dan Weber <dan@mirrorlynx.com> for the info.

[PATCH] added rules for CAPI devices.

Thanks to Marcel Holtmann <marcel@holtmann.org> for the information.

[PATCH] added a dev.d alsa script to help people out.

Originally by Marco d'Itri <md@Linux.IT>

[PATCH] fix devpath for netdev

Fix the devpath string for unhandled netdev's.

[PATCH] DEVNODE -> DEVNAME transition fixes

  It seems that the transition from DEVNODE to DEVNAME wasn't done
  everywhere. This broke udev_dbus for me.

  Attached patch does the transition in the places it wasn't done yet.

[PATCH] v024 release

[PATCH] tweak node unlink handling

Based on a patch from Olaf Hering we remove the node now only if the
requested dev_t is different from the already existing node, so any
run of udevstart should preserve the inode number of the node file.

syslog while the right node is already there:
  creating device node '/udev/hda'
  make_node: preserve file '/udev/hda', cause it has correct dev_t
  make_node: chmod(/udev/hda, 060600)

syslog for wrong file already there:
  creating device node '/udev/hda'
  make_node: already present file '/udev/hda' unlinked
  make_node: chmod(/udev/hda, 060600)

syslog for directory with same name already there:
  creating device node '/udev/hda'
  make_node: unlink(/udev/hda) failed with error '21'

[PATCH] switch udevd's msg_dump() to #define

Here we switch the msg_dump() to #define instead of commenting it out.

[PATCH] handle netdev in udevruler

Here we get the netdev handling for udevruler, after all information
is available now in the udev database. It also fixes the way libsysfs is
used.

[PATCH] remove compiler warning from udevd.c

[PATCH] man page cleanup

Small cleanup of the wording in the udev. man page.

[PATCH] put config info in db for netdev

Small change, to put the config informations for handled network
interfaces into the database.

[PATCH] increase udevd event timeout

I have a very slow Bluetooth adapter hotplug event call, so we need
to increase the event timeout to something larger than 6 seconds.

  Apr  1 02:54:08 pim udevsend[12119]: main: DEVPATH = '/class/tty/ttyUB0'
  Apr  1 02:54:08 pim udevsend[12119]: main: SEQNUM = '340'

  Apr  1 02:54:08 pim udevsend[12120]: main: DEVPATH = '/devices/pci0000:00/0000:00:1d.2/usb4/4-1'
  Apr  1 02:54:08 pim udevsend[12120]: main: SEQNUM = '344'

  Apr  1 02:54:09 pim udevsend[12133]: main: DEVPATH = '/devices/pci0000:00/0000:00:1d.2/usb4/4-1/4-1:1.1'
  Apr  1 02:54:09 pim udevsend[12133]: main: SEQNUM = '342'

  Apr  1 02:54:10 pim udevsend[12136]: main: DEVPATH = '/devices/pci0000:00/0000:00:1d.2/usb4/4-1/4-1:1.2'
  Apr  1 02:54:10 pim udevsend[12136]: main: SEQNUM = '343'

  Apr  1 02:54:13 pim udevsend[12141]: main: DEVPATH = '/devices/pci0000:00/0000:00:1d.2/usb4/4-1/4-1:1.0'
  Apr  1 02:54:13 pim udevsend[12141]: main: SEQNUM = '341'

[PATCH] udevstart fix

Sorry, udevstart needs a small cleanup after my recent copy/pastes.

[PATCH] only generate udev.8 on the fly, not all other man pages.

[PATCH] update bk ignore list some more.

[PATCH] switch to generate the man pages during the normal build, not during the install.

[PATCH] convert udev.8.in to use @udevdir@ macro for make install.

[PATCH] first step of making man pages dynamically generated.

Based on a an original patch from Olaf Hering <olh@suse.de>

[PATCH] add install and uninstall the etc/dev.d/net/hotplug.dev file to the Makefile

[PATCH] tweak net_test a bit.

[PATCH] put netdev handling and dev.d/ in manpages

Mention the recently added netdev handling and the dev.d/ directories
in the man pages.

[PATCH] fix some segfaults when running udevtest for network devices.

[PATCH] make a net_test test script using udevtest.

[PATCH] handle the subsytem if provided in udevtest.

this is needed to test the /etc/dev.d/ stuff properly.

[PATCH] add hotplug.dev script to handle renamed network devices.

[PATCH] add a bunch of network class devices to the test sysfs tree

[PATCH] update RFC-dev.d docs due to DEVNODE to DEVNAME change

[PATCH] DEVPATH for netdev

Here we change the DEVPATH for netdev's in the environment of the dev.d/
scripts to the name the device is renamed to. The original name doesn't
exist in the kernel after rename.

[PATCH] netdev - udevdb+dev.d changes

Here is a patch to change the netdev handling in the database and for
the dev.d/ calls. I applies on top of the udevd.patch, cause klibc has
no sysinfo().

 o netdev's are also put into our database now. I want this for the
   udevruler gui to get a list of all handled devices.
   All devices in the db are stamped with the system uptime value at
   the creation time. 'udevinfo -d' prints it.

 o the DEVPATH value is the key for udevdb, but if we rename
   a netdev, the name is replaced in the kernel, so we add
   the changed name to the db to match with the remove event.

   NOTE: The dev.d/ scripts still get the original name from the
   hotplug call. Should we replace DEVPATH with the new name too?

 o We now only add a device to the db, if we have successfully created
   the main node or successfully renamed a netdev. This is the main part
   of the patch, cause I needed to clean the retval passing trough all
   the functions used for node creation.

 o DEVNODE sounds a bit ugly for netdev's so I exported DEVNAME too.
   Can we change the name?

 o I've added a UDEV_NO_DEVD to possibly skip the script execution
   and used it in udev-test.pl.
   udevstart is the same horror now, if you have scripts with logging
   statements in dev.d/ it takes minutes to finish, can we skip the
   scripts here too?

 o The get_device_type() function is changed to be more strict, cause
   'udevinfo -a -p /block/' gets a class device for it and tries to
   print the major/minor values.

 o bugfix, the RESULT value has now a working newline removal and a test
   for this case.

[PATCH] udevd race conditions and performance, assorted cleanups - take 2

here is a patch on top of your nice improvements.
I fixed the whitespace and it hopefully fixes the stupid timestamp bug in
udevd. Some stupid OS sets the hwclock to localtime and linux changes it
to UTC while starting. If any events are pending they may be delayed by
the users time distance from UTC :) So we use the uptime seconds now.

[PATCH] udevd race conditions and performance, assorted cleanups

This patch covers a number of areas:

1) sysfs.h is fixed up to use the common dbg() macro.  This fixes the
case where DEBUG is defined but USE_LOG isn't.

2) udevstart.c is modified to include the proper headers, rather than
getting them indirectly which can break depending on Makefile flags

3) udevd.c gets some major changes:
a) I added a pipe from the signal handler.  This fixes the race
conditions that I mentioned earlier.  Basically, the point of the pipe
is to force the select() call to return immediately if a signal handler
fired before we actually started the select() call.  This then lets us
run the appropriate code based on flags set in the signal handler proper.
b) I added a number of flags to coalesce calls to common routines.  This
should make things slightly more efficient.
c) since most calls will tend to come in with a sequence number larger
than what has been received, I switched msg_queue_insert() to scan the
msg_list backwards to improve performance.

 filename="udevd.diff"

[PATCH] udevinfo patch

On Fri, Mar 26, 2004 at 06:36:32PM +0500, Ananth N Mavinakayanahalli wrote:
> On Fri, Mar 26, 2004 at 11:24:39AM +0100, Kay Sievers wrote:
> > On Fri, Mar 26, 2004 at 11:21:29AM +0500, Ananth N Mavinakayanahalli wrote:
> > > On Fri, Mar 26, 2004 at 04:47:35AM +0100, Kay Sievers wrote:
> > > > On Fri, Mar 26, 2004 at 01:26:46AM +0100, Carl-Daniel Hailfinger wrote:
> > > > > Greg KH wrote:
> > > > > > On Tue, Mar 23, 2004 at 01:51:01PM -0800, Daniel Stekloff wrote:
> > > >
> > > > No, it breaks the net device handling. I think we should change
> > > > libsysfs instead, not to return a class device for '/block', if
> > > > we want to fix it.
> > >
> > > /sys/block is considered a sysfs "class" and not a class_device. So,
> > > going by udevinfo's help, -p expects path to a class_device and _not_
> > > a class itself and hence option /sys/block with -p is not a valid query.
> > >
> > > Kay?
> >
> > Yes, it's invalid, but we shouldn't print major minor for a invalid
> > path. sysfs_open_class_device_path("/block") returns a device. If this is
> > the right behavior for libsysfs, I will change the get_device_type("/block")
> > not to return a 'b'-type.
>
> Libsysfs validates the path given to it for opening a class_device to be
> a valid directory; it does not however validate if the path is a valid
> class_device path. So, in the case of udevinfo, a 'b' type should not
> be returned if the path is just /sys/block or /sys/block/

This may prevent it.

[PATCH] Add README for chassis_id

[PATCH] fix SEGV in libsysfs/dlist.c

Hi all, Greg,

libsysfs/dlist.c: _dlist_mark_move()

is missing checks for empty lists and may (and indeed, does) crash when=20
called with empty dlists.

[PATCH] clean up chassis_id coding style.

[PATCH] Add chassis_id program to extras directory

[PATCH] clean up the OSDL document formatting a bit

[PATCH] add OSDL documentation for persistent naming

[PATCH] add netlink rules to devfs and gentoo rules files.

[PATCH] added USB device rules to rules files.

[PATCH] clean up the gentoo rules file a bit more, adding dri rules.

[PATCH] fix up udev.rules to handle oss rules better.

Patch from Jordan <lx_xero@comcast.net>

[PATCH] 023_bk mark

[PATCH] dev_d.c file sorting and cleanup

On Thu, Mar 25, 2004 at 02:52:13AM +0100, Kay Sievers wrote:
> Please have look if it still works for you, I only did a very quick
> test.

Here is a unified version, with all the functions moved to udev_lib.c.
We have a generic function now, to call a given fnct(char *) for every
file ending with a specific suffix, sorted in lexical order. We use it
to execute the dev.d/ files and read our rules.d/ files. The binary
should be a bit smaller now.

I've also changed it, to not do the dev.d/ exec for net devices.

[PATCH] apply all_partitions rule to main block device only

Rules with NAME{all_partitions}= are now applied only to the parent of a
block device. We no longer need to exclude the partitions or sg* devices.

[PATCH] small ide-devfs.sh fix

The file may not exist and errors would be spewed on the console.

[PATCH] fix udev.spec file for where udevtest should be placed.

[PATCH] v023 release

[PATCH] create the /etc/dev.d/ directories in 'make install'

[PATCH] actually have udev run files ending in .dev in the /etc/dev.d/ directory as documented.

[PATCH] hmm, handle net devices with udev?

Hmm, Arndt Bergmann sent a patch like this one a few weeks ago and
I want to bring the question back, if we want to handle net device
naming with udev.

With this patch it is actually possible to specify something like this
in udev.rules:

  KERNEL="dummy*", SYSFS{address}="00:00:00:00:00:00", SYSFS{features}="0x0", NAME="blind%n"
  KERNEL="eth*", SYSFS{address}="00:0d:60:77:30:91", NAME="private"

and you will get:

  [root@pim udev.kay]# cat /proc/net/dev
  Inter-|   Receive                                                | Transmit
   face |bytes    packets errs drop fifo frame compressed multicast|bytes   packets errs drop fifo colls carrier compressed
       lo:    1500     30    0    0    0     0          0         0    1500      30    0    0    0     0       0          0
  private:  278393   1114    0    0    0     0          0         0  153204    1468    0    0    0     0       0          0
     sit0:       0      0    0    0    0     0          0         0       0       0    0    0    0     0       0          0
   blind0:       0      0    0    0    0     0          0         0       0       0    0    0    0     0       0          0

The udevinfo program is also working:

  [root@pim udev.kay]# ./udevinfo -a -p /sys/class/net/private
    looking at class device '/sys/class/net/private':
      SYSFS{addr_len}="6"
      SYSFS{address}="00:0d:60:77:30:91"
      SYSFS{broadcast}="ff:ff:ff:ff:ff:ff"
      SYSFS{features}="0x3a9"
      SYSFS{flags}="0x1003"
      SYSFS{ifindex}="2"
      SYSFS{iflink}="2"
      SYSFS{mtu}="1500"
      SYSFS{tx_queue_len}="1000"
      SYSFS{type}="1"

  follow the class device's "device"
    looking at the device chain at '/sys/devices/pci0000:00/0000:00:1e.0/0000:02:01.0':
      BUS="pci"
      ID="0000:02:01.0"
      SYSFS{class}="0x020000"
      SYSFS{detach_state}="0"
      SYSFS{device}="0x101e"
      SYSFS{irq}="11"
      SYSFS{subsystem_device}="0x0549"
      SYSFS{subsystem_vendor}="0x1014"
      SYSFS{vendor}="0x8086"

The matching device will be renamed to the given name. The device name
will not be put into the udev database, cause the kernel renames the
device and the sysfs name disappears.

I like it, cause it plugs in nicely. We have all the naming features
and sysfs queries and walks inside of  udev. The sysfs timing races
are already solved and the management tools are working for net devices
too. nameif can only match the MAC address now. udev can match any sysfs
value of the device tree the net device is connected to.
But right, net devices do not have device nodes :)

[PATCH] added RFC-dev.d document detailing how /etc/dev.d/ works.

[PATCH] fixed up udev.spec to handle selinux stuff properly now.

[PATCH] remove USE_DBUS and USE_SELINUX flags from the README as they are no longer present.

[PATCH] remove selinux stuff from the main Makefile

[PATCH] move udev_selinux into extras/selinux

[PATCH] fix dbus build in the udev.spec file.

[PATCH] remove dbus stuff from main Makefile

Not needed now that it's in extras.

[PATCH] move udev_dbus to extras/dbus

Should build now, but might have linking problems.

[PATCH] udev_dbus can now compile properly, but linnking is another story...

[PATCH] remove udev_dbus.h from Makefile

[PATCH] first cut at standalone udev_selinux program.

Will not work, need to finish working on this on a system with selinux installed...

[PATCH] remove selinux support from udev core as it's no longer needed.

[PATCH] first cut at standalone udev_dbus program.

Will not work, need to finish working on this on a system with dbus installed...

[PATCH] add get_devnode() helper to udev_lib for udev_dbus program

[PATCH] remove dbus code from core udev code as it's no longer needed to be there.

[PATCH] add /etc/dev.d/ support for udev add and remove events.