check the udevdb before assigning a new %e

A second "add" event created a new enumeration with %e. Check the
udevdb first and if a matching name is already used.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

udev_test.pl: we replace untrusted chars with '_'

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

remove special TIMEOUT handling from incoming queue

Moving events directly to the exec queue instead of the reordering
incoming queue, leaves holes in the sequence, that lead to timeouts for
all other events. Remove that part of the special handling.

(With netlink, events can't get out-of-order and the maximum timeout is 5
seconds and should not cause any trouble with the 10 sec timout for the
firmware class anyway. Events with timeouts are still prioritized for
execution, but don't bypass the incoming queue anymore.)

Many thanks to:
  Uberto Barbini <uberto@ubiland.net>

for his endless debugging and sending all the traces, that showed this
failure with his DVB device:
  UEVENT[1124474094] add@/module/stv0299
  UEVENT[1124474094] add@/module/ves1x93
  UEVENT[1124474094] add@/module/ttpci_eeprom
  UEVENT[1124474094] add@/module/saa7146
  UEVENT[1124474094] add@/module/video_buf
  UEVENT[1124474094] add@/module/saa7146_vv
  UEVENT[1124474094] add@/module/dvb_core
  UEVENT[1124474094] add@/module/dvb_ttpci
  UEVENT[1124474094] add@/bus/pci/drivers/dvb
  UEVENT[1124474094] add@/class/firmware/0000:00:14.0
  UDEV  [1124474094] add@/module/dvb_core
  UDEV  [1124474094] add@/module/saa7146_vv
  UDEV  [1124474094] add@/module/dvb_ttpci
  UDEV  [1124474094] add@/module/ves1x93
  UDEV  [1124474094] add@/module/ttpci_eeprom
  UDEV  [1124474094] add@/module/saa7146
  UDEV  [1124474094] add@/module/stv0299
  UDEV  [1124474094] add@/module/video_buf
  UDEV  [1124474094] add@/bus/pci/drivers/dvb
  UEVENT[1124474094] remove@/class/firmware/0000:00:14.0  <- event with TIMEOUT will leave a hole in the incoming
  UDEV  [1124474094] add@/class/firmware/0000:00:14.0        sequence, which will cause a wait for the alarm()
  UEVENT[1124474094] add@/class/i2c-adapter/i2c-1            that flushes the queue
  UEVENT[1124474094] add@/class/i2c-dev/i2c-1
  UDEV  [1124474094] remove@/class/firmware/0000:00:14.0  <- event also has TIMEOUT and is executed immediately
  UEVENT[1124474095] add@/class/dvb/dvb0.demux0
  UEVENT[1124474095] add@/class/dvb/dvb0.dvr0
  UEVENT[1124474095] add@/class/dvb/dvb0.video0
  UEVENT[1124474095] add@/class/dvb/dvb0.audio0
  UEVENT[1124474095] add@/class/dvb/dvb0.ca0
  UEVENT[1124474095] add@/class/dvb/dvb0.osd0
  UEVENT[1124474095] add@/class/dvb/dvb0.net0
  UEVENT[1124474095] add@/class/video4linux/video1
  UEVENT[1124474095] add@/class/dvb/dvb0.frontend0
  UDEV  [1124474099] add@/class/i2c-adapter/i2c-1         <- all others have 5 seconds delay cause of the missing event
  UDEV  [1124474099] add@/class/dvb/dvb0.ca0                 missing events
  UDEV  [1124474099] add@/class/dvb/dvb0.osd0
  UDEV  [1124474099] add@/class/video4linux/video1
  UDEV  [1124474099] add@/class/dvb/dvb0.frontend0
  UDEV  [1124474099] add@/class/dvb/dvb0.video0
  UDEV  [1124474099] add@/class/dvb/dvb0.audio0
  UDEV  [1124474099] add@/class/i2c-dev/i2c-1
  UDEV  [1124474099] add@/class/dvb/dv

My test program that simulates a similar sequence, runs without any delay
now. (With one of the next versions we will make netlink mandatory, then
we can remove the whole input queue crap with the reordering anyway.)

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

udevd: move some logging to "info" and "err"

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

vol_id: ID_LABEL_SAFE will no longer contain fancy characters

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

rename ...trailing_char() to ...trailing_chars()

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

don't reenter get_udevd_msg() if message is ignored

Check all event sources with every iteration.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

remove not needed sig_flag for state of signal_pipe

Less internal state is better. And, we don't need to call
read() twice with every signal handling.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

use WRITE_END/READ_END for the pipe index

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

start to enforce plain ascii or valid utf8

No device node or symlink can have other characters as plain
readable ascii or valid utf8. The /dev/disk/by-label/* symlinks
can no longer contain weird stuff read from untrusted sources.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add uft8 validation for safe volume label exporting

We will not support any other character encoding than plain ascii
or utf8 for volume labels. All invalid utf8 and non-ascii characters
are substituted for security reasons. No options, no fancy heuristics.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

don't store devpath in udevdb, we don't need it

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add and use name_list_cleanup() for cleaning up the string lists

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

change parameter order of udev_db_search_name()

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

print persistent data with "udevinfo -q all"

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

prepare for new HAL udevdb dump

HAL soon wants to read the whole content of the udevdatabase while
starting up. This makes the whole udev structure available to the
udevinfo "dump".

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix lookup for name in the udevdb, it should return the devpath

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

store ENV{key}="value" exported keys in the database

Just like the IMPORT values are stored in the udev database the
exported keys are stored in the udev database. The database is also
imported on a remove event before the rules are matched. So it
is possible to do:
  ACTION=="add", KERNEL=="sda", ENV{MY_DEVICE}="1234", RUN+="/bin/logger add $devpath"
  ACTION=="remove", ENV{MY_DEVICE}=="1234", RUN+="/bin/logger remove $devpath"

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

switch to '==' in raid-devfs.sh

update SUSE rules

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add pci_express to bus list

Hopefully that wait_for_sysfs crap can go away some day...

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update SUSE rules

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

path_id: remove SUSE specific PATH

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix typo in firmware_helper

The wrong string is being nullifed.

udevmonitor: cleanup on exit

We want to write the output buffer before
we exit, otherwise redirection to a file
does not work correctly.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

volume_id: don't probe for mac_partition_maps

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

udevd: keep the right order for messages without SEQNUM

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix exit code of udevinitsend and udevmonitor

Thanks to: Marco d'Itri <md@Linux.IT> for the initial patch.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add program name to logged error

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

*_id: fix zero length in set_str()

Thanks to Olaf Hering <olh@suse.de>

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

068 release

add EXTRAS documentation to the README file.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix printed udevmonitor header

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

bring std(in|out|err) fd's in a sane state

If the kernel forks us as an usermodhelper, we don't have any of
the standard fd's and the first open() will start with fd=0.
This is inherited to all forked childs and confuses later forked
helpers where we want to read from a pipe connected to the helpers
stdout/stderr.

  # ls -l /proc/$(pidof udevd)/fd
  total 6
  dr-x------  2 root root  0 2005-08-18 12:44 .
  dr-xr-xr-x  4 root root  0 2005-08-18 12:44 ..
  lrwx------  1 root root 64 2005-08-18 12:44 0 -> /dev/null
  lrwx------  1 root root 64 2005-08-18 12:44 1 -> socket:[1274617]
  lr-x------  1 root root 64 2005-08-18 12:44 2 -> pipe:[1274618]
  l-wx------  1 root root 64 2005-08-18 12:44 3 -> pipe:[1274618]
  lrwx------  1 root root 64 2005-08-18 12:44 4 -> socket:[1274619]
  lrwx------  1 root root 64 2005-08-18 12:44 5 -> socket:[1274620]

Ouch! This will obviously not redirect sterr, it will kill the pipe
we established between the parent and the child:

  devnull = open("/dev/null", O_RDWR);
  dup2(devnull, STDERR_FILENO);

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix typo in cdrom_id syslog

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

Merge rsync.kernel.org/pub/scm/linux/hotplug/udev

cdrom_id: change err() to info() to help with debugging problems

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Always open the cdrom drive in non-blocking mode in cdrom_id

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

vol_id: fix sloppy error handling

vol_id segfaults if read() fails on broken devices reporting
the wrong size.

Thanks to Erhard Schultchen for the debugging.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

ENV{TEST}=="1" compares and ENV{TEST}="1" sets the environment

This can be uses to export stuff to the event environment or
to carry a state from one rule to another, like enabling/disabling
later rules conditionally.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

move udevmonitor to /usr/sbin

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

cleanup some debug output and move to info level + unify select() loops

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

067 release

don't fail too bad, if /dev/null does not exist

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix udevstart event ordering, we want /dev/null very early

Merge rsync.kernel.org/pub/scm/linux/hotplug/udev

fix more compiler warnings ...

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

added the cdrom.h #defines directly into the cdrom_id.c file

This solves the problem with building the file against old kernel header
files.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix make install, as we don't provide a default rule set anymore

Several people complained about the "default" rules and the "default"
setup. Here we start to remove things where we can't hava a  "default".
The best examples for rules are in the distro folders, just pick the one
that matches your needs and start from there.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update SUSE rules

don't use ID_TYPE, so cdroms will be in /dev/disk/

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

066 release

tweak the gentoo rules some more.
- make the persistant rules smaller
- add usbfs-like device node support

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

changed location of gentoo helper apps to be /sbin instead of in scripts dir

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Change the gentoo rules to use cdrom_id instead of cdsymlink.sh

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

fix klibc build issue in cdrom_id.c

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

added cdrom_id to the build check

changed cdrom_id exports to be easier to understand and consistant with other _id programs.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Created cdrom_id program to make it easier to determine cdrom types

Based on the framework from ata_id by Kay.
Now we can drop the cdsymlinks.sh and .c files

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

update gentoo rule file.
This fixes the issue of cdroms not showing up in the proper group,
and them showing up in /dev/cd/ for the persistant names.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

move some logging to the info level

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add Usage: to udevmonitor and udevcontrol

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

allow logging of all output from executed tools

If USE_DEBUG=true and udev_log="debug", all output of the forked
programs to stdout and stderr is send to syslog.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update SUSE rules

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix udevinfo output

split udev_util in several files

It too big for the small tools to link against for only the log function.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update SUSE rules

Tape naming is harder than expected, go back to block devices only.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

trivial text cleanups

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

remove example rules and put the dev.d stuff into the run_directory folder

The distro rules are the best example you can get and the use of
dev.d/ is no longer recommended.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update README

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

more distro rules updates

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add firmware_helper to load firmware

From: Bill Nottingham <notting@redhat.com>

Changed reading of firmware blob to mmap and let firmware_helper
follow the setting of the log level with UDEV_LOG.

update SUSE rules

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add SUSE rules

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

update distro rules files

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

add flag for reading of precompiled rules

Rules can be precompiled and stored on disk for initramfs, to avoid
parsing the rules with every event again and again. Also the OWNER and
GROUP names are already resolved to numerical values in the compiled
rules. This flag is used for the upcoming move of the rules parsing
into udevd:
  If the real root is mounted udevd is started and parses the rules
  only once. The event processes will inherit the already parsed rules
  from the daemon, so we want to ignore any precompiled rules and
  use the real rules files and watch the filesystem for changes to
  reload the rules automatically.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

allow clean shutdown of udevd

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

switch some strlcpy's to memcpy

strlcpy counts the sourec string lengt and is therefore not suitable
to copy a defined length of characters from one string to another.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

read sysfs attribute also from parent class device

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

updated gentoo rule file to handle removable ide devices.

Thanks to Kay for the quick fix.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

add NETLINK define for the lazy distros

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

065 release

Merge gregkh@ehlo.org:/home/kay/public_html/linux/hotplug/udev-kay

Added persistent name rules for block devices to gentoo rule file.

Copied from Kay's rules from the SuSE package.

Let the fun begin...

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

update release notes for next version

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

Added horrible (but fun) path_id script to extras.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

Update gentoo rules file.

Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>

add udevmonitor, to debug netlink+udev events at the same time

This rule:
  RUN+="socket:/org/kernel/udev/monitor"

send the event to the running monitor. Running udevstart give a nice
monitor output. Also the delay between the kernel uevent and the udev
udev event can be investigated:

  pim:/home/kay/src/udev-kay # ./udevmonitor
  UEVENT[1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1
  UEVENT[1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0
  UEVENT[1122920113] add@/class/scsi_host/host12
  UEVENT[1122920113] add@/class/usb_device/usbdev2.99
  UEVENT[1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0/host12/target12:0:0/12:0:0:0
  UDEV  [1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1
  UEVENT[1122920113] add@/block/sda
  UEVENT[1122920113] add@/class/scsi_generic/sg0
  UEVENT[1122920113] add@/class/scsi_device/12:0:0:0
  UDEV  [1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0
  UDEV  [1122920113] add@/class/scsi_host/host12
  UDEV  [1122920113] add@/class/usb_device/usbdev2.99
  UDEV  [1122920113] add@/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0/host12/target12:0:0/12:0:0:0
  UDEV  [1122920113] add@/block/sda
  UDEV  [1122920113] add@/class/scsi_generic/sg0
  UDEV  [1122920113] add@/class/scsi_device/12:0:0:0
  UEVENT[1122920115] add@/block/sda/sda1
  UDEV  [1122920115] add@/block/sda/sda1

  pim:/home/kay/src/udev-kay # ./udevmonitor --env
  UDEV  [1122920185] add@/block/sda/sda1
  ACTION=add
  DEVPATH=/block/sda/sda1
  SUBSYSTEM=block
  SEQNUM=2159
  MINOR=1
  MAJOR=8
  PHYSDEVPATH=/devices/pci0000:00/0000:00:1d.1/usb2/2-1/2-1:1.0/host13/target13:0:0/13:0:0:0
  PHYSDEVBUS=scsi
  PHYSDEVDRIVER=sd
  UDEVD_EVENT=1
  UDEV_LOG=3
  ID_VENDOR=IBM
  ID_MODEL=Memory_Key
  ID_REVISION=3.04
  ID_SERIAL=IBM_Memory_Key_0218B301030027E8
  ID_TYPE=disk
  ID_BUS=usb
  ID_PATH=usb-0218B301030027E8:0:0:0
  ID_FS_USAGE=filesystem
  ID_FS_TYPE=vfat
  ID_FS_VERSION=FAT16
  ID_FS_UUID=42D8-9E02
  ID_FS_LABEL_SAFE=s_p_c
  DEVNAME=/dev/sda1

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

allow RUN to send the environment to a local socket

RUN="socket:<name>" will send the environment in the kernel uevent
format to the named destination. Using the socket instead of the program
to pass the hotplug events to the HAL daemon, cuts down the running
time of udevstart from 0.8 to 0.4 seconds on my box.

  env -i ACTION=add DEVPATH=/block/hda/hda1 strace -s10000 ./udev block
  sendto(3, "add@/block/hda/hda1\0
    ACTION=add\0DEVPATH=/block/hda/hda1\0UDEV_LOG=3\0
    ID_TYPE=disk\0ID_MODEL=HTS726060M9AT00\0ID_SERIAL=MRH401M4G6UM9B\0
    ID_REVISION=MH4OA6BA\0ID_BUS=ata\0ID_PATH=pci-0000:00:1f.1-ide-0:0\0
    ID_FS_USAGE=other\0ID_FS_TYPE=swap\0ID_FS_VERSION=2\0
    ID_FS_UUID=9352cfef-7687-47bc-a2a3-34cf136f72e1\0
    ID_FS_LABEL=ThisIsSwap\0ID_FS_LABEL_SAFE=ThisIsSwap\0
    DEVNAME=/dev/hda1\0"

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

fix GGC signed pointer warnings and switch volume_id to stdint

Solaris uses volume_id now and they fiddled around with configure scripts
to map the linux kernel int types. Adding the types locally to volume_id
breaks the klibc build, so just switch to these ugly types and forget it. :)

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

Oops, 064 release, not 063

063 release

move code to its own files

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

make SYSFS{} usable for all devices

Patch from:
  Hannes Reinecke <hare@suse.de>

usb_id: fix typo

add ID_BUS to *_id programs

create_floppy_devices: add tool to create floppy nodes based on sysfs info

volume_id: add OCFS (Oracle Cluster File System) support

add padding to rules structure

Some architectures really want well alingned structures.
Thanks to Jim Gifford <maillist@jg555.com> for help finding it.

Signed-off-by: Kay Sievers <kay.sievers@suse.de>

allow rules to have labels and skip to next label

This will allow us to have whole blocks of rules to skip
conditionally. The following section creates the node "yes":
  GOTO="TEST"
  NAME="no"
  NAME="no2", LABEL="NO"
  NAME="yes", LABEL="TEST"
  NAME="no3"

thread unknown ENV{key} match as empty value

This way we can run a rule if a certain key is not set by:
  ENV{key}="" or ENV{key}!="?*"

Signed-off-by: Kay Sievers <kay.sievers@suse.de>