update NEWS TODO

path_id: fix fibre channel handling

man: fix typos

volume_id: clear probing result before probing and do not probe a second time, if not needed

On Thu, Nov 20, 2008 at 14:17, Karel Zak <kzak@redhat.com> wrote:
>  I see the patch (volume_id_probe_filesystem()) and a few things come
>  to mind:
>
>  - shouldn't be the relevant parts (label, uuid, version, ...) of
>    the "struct volume_id" zeroized when you found a signature and
>    before you call the next probing function?
>
>  - it seems as overkill to use two for()s and probe two times for all
>    filesystems. What about to store the first result and re-use it?
>
>  - .. or at least never use the second for() when the fist for() found
>    nothing ;-)

volume_id: update btrfs

vol_id: if regular files are probed, use stat() for the size value

volume_id: remove unused usage types

include errno.h in sysdeps.h

scsi_id: make sure, we do not have slashes in values

ata_id: make sure, we do not have slashes in values

usb_id: replace chars in returned strings

Slashes are not funny in device names:
  |-- input
  |   |-- by-id
  |   |   `-- usb-Logitech_USB-PS
  |   |       |-- 2_Optical_Mouse-event-mouse -> ../../event2
  |   |       `-- 2_Optical_Mouse-mouse -> ../../mouse1

libudev: accept NULL whitelist in util_replace_chars()

rules: v4l - add by-id/ links for USB devices

/dev/v4l
|-- by-id
|   |-- usb-046d_09a4_C4B15020-video-index0 -> ../../video0
|   `-- usb-05a9_a511-video-index0 -> ../../video1
`-- by-path
    |-- pci-0000:00:1d.0-usb-0:1:1.0-video-index0 -> ../../video1
    `-- pci-0000:00:1d.7-usb-0:2:1.0-video-index0 -> ../../video0

rules: merge group "video" into default rules

rules: add persistent rules for memory stick block devices

commit 5a9aed145ac0ffb3e29b1c8e0f19b34e277f9117
Author: Harald Hoyer <harald@redhat.com>
Date:   Wed Nov 19 11:22:30 2008 +0100

    added persistent rules for memory stick block devices

bump version

volume_id: update btrfs magic

autogen.sh: fix -print-multi-os-directory usage

release 133

update NEWS

fix $links substitution for devices without any link

Reported-by: Michel Hermier <michel.hermier@gmail.com>

udevd: fix cleanup of /dev/.udev/uevent_seqnum

limit $attr(<symlink>) magic to well-known links only

udevadm: fix option parsing breakage with klibc

The klibc implementation of getopt_long() behaves slightly different
from the glibc one - in particular, it treats the change of the option
string argument between invocations as start of parsing a different
command line, and resets its state.  However, the udevadm code
expected getopt_long() invocations in subcommands to continue parsing
the rest of command line after initial options has been parsed at the
top level; with klibc this broke, causing all udevadm subcommands to
stop recognizing their options.

Instead of relying on the glibc behavior, reset the getopt_long()
state properly before invoking the subcommand handler: move argv to
point to the subcommand name, decrease argc appropriately, and set
optind = 0.  This also fixes a minor bug visible with glibc - without
setting optind = 0 all getopt_long() calls in subcommand handlers were
behaving as if "+" was specified as the first character of the option
string (which disables option reordering), because that state was set
by the first getopt_long() call at the top level, and was not reset
when parsing subcommand options.

Signed-off-by: Sergey Vlasov <vsu@altlinux.ru>

libudev: device - handle disk "device" link for partitions in deprecated sysfs layout

Thanks to Mikhail Kolesnik <mike@openbunker.org> for finding this.

vol_id: clarify error message

volume_id: ntfs - mark as no other fs must match

fix spelling in comment

volume_id: fat - accept empty FAT32 fsinfo signature

volume_id: always check for all filesystem types and skip conflicting results

We probe for all known filesystems to find conflicting signatures. If
we find multiple matching signatures and one of the detected filesystem
types claims that it can not co-exist with any other filesystem type,
we do not return a probing result.

We can not afford to mount a volume with the wrong filesystem code and
possibly corrupt it. Linux ssytems have the problem of dozens of possible
filesystem types, and volumes with left-over signatures from former
filesystem types. Invalid signature need to be removed from the volume
to make the filesystem detection successful.

We do not want to read that many bytes from probed floppies, skip volumes
smaller than a usual floppy disk.

volume_id: bump revision

remove len == 0 check, the index root is always '\0'

volume_id: fat - move check for msdos signature (0x55 0xaa)

Make the msdos signature (0x55 0xaa) at 510 and 511 optional when
the standard FAT magic string is present.

configure: add linux-hotplug mail address

update NEWS

rules: remove pnp shell script, acpi loads these modules properly

rules: remove DEVTYPE disk/partition

udevadm: info - unify -V and --version

set errno = ENOSYS in inotify stub

scsi_id: we don't use DEVPATH env var anymore, update man page

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

merge trie nodes, childs and root into a single array

reserve child slot 0

string index - split nodes and childs to allow and unlimited number of childs

silence "comparison between signed and unsigned"

udevd: de-duplicate strings in rules

On my Ubuntu installation this removes 15k of duplicate strings,
using a temporary index of about 25k.

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

release 132

convert debug string arrays to functions

On Fri, Nov 7, 2008 at 13:07, Matthias Schwarzott <zzam@gentoo.org> wrote:
> I managed to let udev-131 segfault at startup.
>
> I configured it like this:
> CFLAGS="-Wall -ggdb" ./configure --prefix=/usr --sysconfdir=/etc --exec-prefix=
>
> Running it in gdb shows it segfaults at udev-rules.c:831
>
> (gdb) run
> Starting program: /tmp/udev-131/udev/udevd
>
> Program received signal SIGSEGV, Segmentation fault.
> 0x0804ea06 in get_key (udev=0x9175008, line=0xafcdc8f0, key=0xafcdc5d8,
> op=0xafcdc5d0, value=0xafcdc5d4)
>    at udev-rules.c:831
> 831             dbg(udev, "%s '%s'-'%s'\n", operation_str[*op], *key, *value);

If compiled without optimization, the dbg() macro dereferences variables
which are not available. Convert the string array to a function, which just
returns NULL if compiled without DEBUG.

fix size_t compiler warning on 32 bit platforms

release 131

udevadm: settle - exit if udevd exits

cleanup /dev/.udev/queue on startup and exit

rules: md - add mdadm 3 device naming

fix indentation

udevd: shrink struct token to 12 bytes

$links should be relative

unify string replacement

usb_id: MassStorage SubClass 6 is "scsi" not "disk"

do not import the "uevent" file when we only read the db to get old symlinks

fix cleanup of possible left-over symlinks

fix $links substitution

info() PROGRAM and IMPORT execution

volume_id: btrfs update

fix list handling in enumerate and rules file sorting

firmware.sh: record missing files in /dev/.udev/firmware-missing/

move debugging strings inside #ifdef DEBUG

add "devel" and "install" switches to autogen.sh

move some info() to dbg()

fix udev_node_update_old_links() logic

No need to check if links that still belong to us, need to be updated.

skip entire rule containing device naming keys, if no device can be named

If rules set NAME, SYMLINK, OWNER, GROUP, MODE, events without a device node
and remove events will skip the entire rule. The old parser did the same.

log rules file and line number when NAME, SYMLINK, OWNER, GROUP, MODE, RUN is applied

udevd: be more careful when matching against parents

I'm worried about what will happen with things like

KERNELS=="*" # pointless rule
KERNELS=="doesnt-match" # another pointless rule

Since TK_RULE < TK_M_PARENTS_MAX, we will try to match all three tokens
against parents of the current device.  I can't think of a bad case,
but it's not exactly good either.

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

udevd: fix termination of rule execution

LAST_RULE was broken, and I broke TK_END by making it do the same.
It used a "break" which exited the switch statement, but not the loop!

==2953== Invalid read of size 4
==2953==    at 0x4081EE: dump_token (udev-rules.c:859)
==2953==    by 0x40BADB: udev_rules_apply_to_event (udev-rules.c:1849)
==2953==    by 0x403F17: udev_event_execute_rules (udev-event.c:554)
==2953==    by 0x418626: main (test-udev.c:100)
==2953==  Address 0x55ab1f8 is 0 bytes after a block of size 80 alloc'd
==2953==    at 0x4C23082: realloc (vg_replace_malloc.c:429)
==2953==    by 0x40B13B: udev_rules_new (udev-rules.c:1670)
==2953==    by 0x418536: main (test-udev.c:84)
...

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

write trace log to stderr

fstab_import: add "change" event match

edd_id: add "change" event match

rules: remove "add" match from usb device node rule

rules: remove SCSI timeouts

On Thu, Oct 30, 2008 at 03:55, Tejun Heo <tj@kernel.org> wrote:
The appropriate default timeout differs depending on the transport and
the type of the attached device, so the above two rules harm more than
help.  The affect of the above two rules weren't visible for some
reason but with recent block layer timeout update, they actually work
and cause problems.

ATTR{}== always fails if the attribute does not exist

udevd: merge exec and run queue to minimize devpath string compares

udevd: simplify rules execution loop

cur can't become NULL, and the check for TK_END
can be folded into the switch statement.

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

update NEWS

kerneldoc comment fixes

s/ressources/resources/

Also reduce commas per sentence and add a possessive apostrophe.

libudev: device - allocate envp array only once

test: add RUN+="socket: ..." to a test to run monitor code

libudev: device - fill envp array while composing monitor buffer

Thanks to Alan Jenkins, for the idea.

fix $attr{[<subsystem>/<sysname>]<attribute>} substitution

do not init string arrays, just clear first byte

match_attr() - copy attr value only when needed

udevd: avoid implicit memset in match_attr()

Initializing a char array to "" is equivalent to a memset()
call - which is exactly what it gets compiled to.

Fixing this one callsite reduced memset() _user_ cpu cycles
from 2-4% to 0.05% on the EeePC.

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

udevd: use a tighter loop for compare_devpath()

This crops up in my threaded udevd profiles from time to time.
It's not consistent - probably due to variations in the number
of concurrent events - but it can hit 4% user time and higher.

The change halves the user time spent in compare_devpath().

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

fix handling of string_escape option

match KEY="A|B" without temporary string copy

remove debug printf

fix "unused" warnings

libudev: monitor - cache result of monitor send buffer

libudev: monitor - replace far too expensive snprintf() with strlcpy()

special-case "?*" match to skip fnmatch()

determine at rule parse time if we need to call fnmatch()

This cuts down the large rule set's 120.000 calls to fnmatch() to
51.000, and we can just call strcmp for the simple matches.

distinguish "match" from "assign" by (op < OP_MATCH_MAX)

cache uid/gid during rule parsing

This cuts down the number of parsing /etc/group from ~700 to 11,
with some large rule files installed.