cdrom_id: suppress ID_CDROM_MEDIA_STATE=blank for plain non-writable CDROM media

Some broken fake cdrom drives return ID_CDROM_MEDIA_STATE=blank.

  $ /lib/udev/cdrom_id -d /dev/sr1
  main: probing: '/dev/sr1'
  cd_inquiry: INQUIRY: [Nokia   ][S60             ][1.0 ]
  cd_profiles: GET CONFIGURATION: number of profiles 76
  cd_profiles: current profile 0x08
  cd_media_toc: READ TOC: len: 12
  cd_media_toc: last track 1 starts at block 0
  cd_media_info: disk type 00
  ID_CDROM=1
  ID_CDROM_MRW=1
  ID_CDROM_MRW_W=1
  ID_CDROM_MEDIA=1
  ID_CDROM_MEDIA_CD=1
  ID_CDROM_MEDIA_STATE=blank

Others work fine, but ID_CDROM_MEDIA_STATE is not needed for non-writable CDROM media:

  $ /lib/udev/cdrom_id -d /dev/sr1
  main: probing: '/dev/sr1'
  cd_inquiry: INQUIRY: [SanDisk ][U3 Cruzer Micro ][8.02]
  cd_profiles: GET CONFIGURATION: number of profiles 72
  cd_profiles: current profile 0x08
  cd_media_toc: READ TOC: len: 20
  cd_media_toc: track=1 info=0x4(data) start_block=0
  cd_media_toc: last track 1 starts at block 0
  cd_media_info: disk type 00
  ID_CDROM=1
  ID_CDROM_MRW=1
  ID_CDROM_MRW_W=1
  ID_CDROM_MEDIA=1
  ID_CDROM_MEDIA_CD=1
  ID_CDROM_MEDIA_STATE=complete
  ID_CDROM_MEDIA_SESSION_COUNT=1
  ID_CDROM_MEDIA_TRACK_COUNT=1
  ID_CDROM_MEDIA_TRACK_COUNT_DATA=1

test: check string substitutions in OWNER and GROUP

String substitutions in OWNER and GROUP keys were broken in udev 137-142.
Explicitly test for this, since such breakage will not manifest in typical
rulesets.

rules: add SUBSYSTEM match to scsi rules

On Fri, May 22, 2009 at 16:15, Alan Jenkins <alan-jenkins@tuffmail.co.uk> wrote:
> I've been looking at what is responsible for all the path lookup activity in
> coldplug.  On my debian stable system, it looks like every device gets its
> parent looked up in sysfs.  I think this is due to SUBSYSTEMS matches.
>
> I see the udev default rules are different, but it looks like they still
> test for SUBSYSTEMS on every single device. Should we add SUBSYSTEM="scsi_generic"
> to these three rules?

update NEWS

udevadm: settle - make --timeout=0 working

udevd: implement a more efficient queue file format

Directory lookups show up in profiling. The queue files are responsible
for a large proportion of file-related system calls in udev coldplug.

Instead of creating a file for each event, append their details to a
log file.  The file is periodically rebuilt (garbage-collected) to
prevent it from growing indefinitely.

This single queue file replaces both the queue directory and the
uevent_seqnum file. On desktop systems the file tends not to grow
beyond one page. So it should also save a small amount of memory in
tmpfs.

Tests on a running EeePC indicate average savings of 5% *udevd* cpu time
as measured by oprofile. __link_path_walk is reduced from 1.5% to
1.3%. It is not completely clear where the rest of the gains come from.

In tests running ~400 events, the queue file is rebuilt about 5 times.

Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk>

udevd: remove tiny bit of dead code

fix location of database files

keep the ifdef'd udevd testing/profiling hack

require key names in uppercase

Drop pretty expensive case-insensitive matching, and key names in
mixed or lowercase are not supported anyway.

remove unused GL_FORMAT from rules parser

update TODO

use openat(), unlinkat(), fstatat()

edd_id: use openat()

use more efficient string copying

udevadm: settle - fix timeout

The timeout wasn't working when settle was run as root:
  # udevadm control --stop-exec-queue
  # udevadm trigger
  # udevadm settle --timeout=1
  ... (hangs)

Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk>

remove format char string truncation syntax

OWNER/GROUP: fix if logic

The introduction of the --resolve-names=early/never code introduced a
bug to the OWNER/GROUP lookup.  Previously if the name had contained $,
lookup would have been performed later; after the patch, the key ended
up being ignored!

rule-generator: cd - skip by-path links if we create by-id links

Revert "rules: v4l do not mix vbi and video nodes"

This reverts commit 6205f1186e4980544ea425d31770358d1b2579e4.

fix possible endless loop for GOTO to non-existent LABEL

http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=526365

rules: v4l do not mix vbi and video nodes

version bump

release 142

delete libvolume_id

delete vol_id and require util-linux-ng's blkid

update TODO

add "static" back to the inline functions

<Keybuk> kay: udev git head ftbfs
<Keybuk> udev-watch.o: In function `udev_selinux_init':
<Keybuk> /../udev/udev.h:130: multiple definition of `udev_selinux_init'

rules: fix ieee1394 rules

remove asmlinkage

remove all PHYSDEVPATH handling and warning about

oops, removed ppp entry from rules got committed

handle devtmpfs nodes

UDev follows the kernel given name, and re-uses the kernel created
device node. If the kernel and spcecified udev rules disagree, the
udev specified node node is created and the kernel-created on is
deleted.

rule-generator: net - whitelist NICs that violate MAC local scheme

According to list of assigned ethernet codes [1] referred to by
IANA [2] certain global addresses do not follow the assignement
scheme and use numbers reserved for local use. Several such adapters
are quite widely used, generate rules for them.

[1] http://www.cavebear.com/archive/cavebear/Ethernet/vendor.html
[2] http://www.iana.org/assignments/ethernet-numbers

rules: remove initramfs comment

udevd: revert initial device node creation

rules: add "event*" match

build: use AC_MSG_RESULT

update TODO

accept DEVNAME from the kernel as a hint for the node name

rules: add rfcomm* to group dialout

https://bugs.launchpad.net/bugs/368109

rules: input - fix comments

rules: input - add links for USB/platform non-kbd/mouse devices

rule_generator: net - fix usb comment generation

On Sat, Apr 4, 2009 at 22:17, Omair Eshkenazi <stimpson@phys.huji.ac.il> wrote:
> I noticed that in (70-)persistent-net.rules, the comments for USB devices
> are missing the device/vendor id's. Example:
> # USB device 0x:0x (rt73usb)

drop "extern" keyword from non-static function

update NEWS

rules: create /dev/cpu/<n>/cpuid world readable

I don't see any security implications, to be actually useful,
/dev/cpu/<n>/cpuid should be world readable. The cpuid instruction
can be called from userspace anyway, so there is nothing to hide.
The device does not support any write operation, so 0444 should
suffice.

Signed-off-by: Andre Przywara <andre.przywara@amd.com>

libudev: monitor - export filter_update()

libudev: queue - clarify comments

cdrom_id: skip media tests if CDROM_DRIVE_STATUS != CDS_DISC_OK

libudev: queue - fix get_seqnum_is_finished()

On Thu, Apr 23, 2009 at 11:39, Alan Jenkins <alan-jenkins@tuffmail.co.uk> wrote:
> I'm puzzled by this function:
>
>        /* if we have not seen this seqnum, check if it is/was already queued */
>        if (seqnum < udev_queue->last_seen_udev_seqnum) {
>                udev_queue_get_udev_seqnum(udev_queue);
>                if (seqnum < udev_queue->last_seen_udev_seqnum)
>
> Shouldn't the test be (seqnum > udev_queue->last_seen_udev_seqnum) ?

libudev: monitor - add udev_monitor_filter_remove

udevadm: monitor - add <subsytem>:<devtype> support

libudev: monitor - do not filter messages with wrong magic

libudev: monitor - switch to filter_add_match_subsystem_devtype()

libudev: monitor - use simpler hash

edd_id: add cciss devices

Since cciss devices can be BIOS boot devices just as well as sd* and
hd*, the edd_id program should be run on them so that the later
/dev/disk/by-id/edd-* rules will work.

udevadm monitor - add --subsystem-match=

update TODO

udevadm: monitor - print error if we can not bind to socket

libudev: monitor - add client socket filter for subsystem value

Messages send back by the udev daemon to the netlink socket are
multiplexed by the kernel and delivered to multiple clients. The
clients can upload a socket filter to let the kernel drop messages
not belonging to a certain subsystem. This prevent needless wakeups
and message processing for users who are only interested in a
subset of available events.

Recent kernels allow untrusted users to listen to the netlink
messages.

The messages send by the udev daemon are versioned, to prevent any
custom software reading them without libudev. The message wire format
may change with any udev version update.

cdrom_id: add and use ID_CDROM_MEDIA to decide if we run vol_id

Some broken mobile phones offer a faked cdrom drive with a media
without any tracks.

udevd: at startup create /dev/null, /dev/console, /dev/kmsg

update NEWS

TODO: add packet filter

test-libudev: update monitor source

cdrom_id: add Xen cdrom support

path_id: support identification of Xen virtual block devices

update NEWS

rules: remove DVB shell script

udevd: log error for too old kernels or CONFIG_SYSFS_DEPRECATED

libudev: monitor - clarify socket handling documentation

libudev: path_encode - always return 0 if encoded string does not fit into size

version bump

release 141

libudev: monitor - ignore messages from unusual sources

For added protection, ignore any unicast message received on the
netlink socket or any multicast message on the kernel group not
received from the kernel.

Signed-off-by: Scott James Remnant <scott@ubuntu.com>

libudev: monitor - unify socket message handling

path_encode: fix max length calculation

Sebastian Krahmer wrote:
> it should reserve 4 times not 3 times len :)

avoid leaking netlink socket fd to external programs

The netlink socket is now used by udev event processes. We should take
care not to pass it to the programs they execute. This is the same way
the inotify fd was handled.

Signed-off-by: Alan Jenkins <alan-jenkins@tuffmail.co.uk>

IMPORT: 2048 -> 4096 bytes buffer

"UDEV_MONITOR_KERNEL/UDEV" -> "kernel/udev"

send monitor events back to netlink socket

Instead of of our own private monitor socket, we send the
processed event back to our netlink socket, to the multicast
group 2 -- so any number of users can listen to udev events,
just like they can listen to kernel emitted events on group 1.

usb_id: remove unused variable

libudev: GPL -> LGPL

usb_id: add ID_VENDOR_ID, ID_MODEL_ID, ID_USB_INTERFACE_NUM, ID_USB_DRIVER

$ extras/usb_id/usb_id -x /class/tty/ttyUSB1
ID_VENDOR=HUAWEI_Technology
ID_VENDOR_ENC=HUAWEI\x20Technology
ID_VENDOR_ID=12d1
ID_MODEL=HUAWEI_Mobile
ID_MODEL_ENC=HUAWEI\x20Mobile
ID_MODEL_ID=1003
ID_REVISION=0000
ID_SERIAL=HUAWEI_Technology_HUAWEI_Mobile
ID_TYPE=generic
ID_BUS=usb
ID_USB_INTERFACES=:ffffff:080650:
ID_USB_INTERFACE_NUM=01
ID_USB_DRIVER=option

rules: replace IDE driver with media match

rules: serial - support ttyACM devices

rules: remove ram* from persisten disk links blacklist

rules: exclude mtd* from persistent disk links

rules: rename ide-floppy to ide-gd

The driver's name changed in the 2.6.28 timeframe.

udevadm: monitor - clarify printed header

udevadm: test - handling trailing '/' in devpath

On Wed, Mar 18, 2009 at 16:00, Matthias Schwarzott <zzam@gentoo.org> wrote:
found out how the error occurs:
It is a difference between
A. udevadm test /sys/class/mem/null/
and
B. udevadm test /sys/class/mem/null

Case A was the case that showed the error behaviour. It seems udevadm is
confused by the trailing slash. This behaviour seems to be there since ages.

version bump

rules: fix extra quote in 50-udev-default.rules

Patch from Gerardo Exequiel Pozzi.

udevadm: trigger - add "--action" to --help

vol_id: fix language in manpage

The documentation for the --offset option wasn't in complete
sentences.

Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>

create_floppy_devices: expand manpage

Include a table of what the CMOS types are, and note that nothing
will be created unless the -t option is specified. Also clean up
the formatting and bump the date.

Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>

cdrom_id: update manpage

Include usage information on the -d/--debug flag. Also bump the date.

Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>

usb_id: add manpage

Signed-off-by: Adam Buchbinder <adam.buchbinder@gmail.com>

rules: md-raid.rules fix

md/array_state in case of partition doesn't exist, so all uevents
for partitions didn't execute any SYMLINK rules

Signed-off-by: Michal Soltys <soltys@ziu.info>