doc: udevadm test *does* create nodes and links these days

doc: writing_udev_rules updated for the new command names

Revert "extras/keymap: Fix case matching for Micro-Star"

This reverts commit 66bf63c05cdc4e9b09818aa5fab0b9d319a1c91c.

Further debugging in https://launchpad.net/bugs/178860 showed that for some
weird reason the correct key codes already come out of the "Video Bus" input
device, and the previous commit would cause them to appear a second time
through the standard keyboard device.

This is a kernel bug in the end, but let's not break working things
prematurely.

extras/keymap: Fix case matching for Micro-Star

Some Micro-Star boards apparently have mixed case vendor, instead of all-caps.
Update the glob to catch all such cases.

https://launchpad.net/bugs/178860

assign errno for getgrnam_r()/getpwnam_r()

On Mon, Aug 24, 2009 at 19:50, Lennart Poettering<lennart@poettering.net> wrote:
> One little comment here: on POSIX getrnam_r() doesn't touch
> errno. Instead it returns the error value as return value.

extras/keymap: Fix hold key on Acer Aspire 6920

The Hold key locks the panel and is hardwired. It doesn't have a sensible
keycode to map to, and shouldn't be overloaded either.

extras/modem-modeswitch: eject ZTE MF6xx fake CD-ROMs

modem-modeswitch does not fully work on ZTE MF6xx modems, their fake CD-ROMs
need to be properly ejected in order for the actual modem to appear. Add udev
rule for this device (19d2:2000 in CD-ROM mode).

https://launchpad.net/bugs/281335

extras/keymap: Add Acer Aspire 6920

https://launchpad.net/bugs/407940

hid2hci: remove superfluous bmAttributes match

For the retrigger of the device on remove we were trying to match bmAttributes
of self powered which is unnecessary.

change database file names

With very deeply nested devices, We can not use a single file
name to carry an entire DEVPATH. Use <subsystem>:<sysname> as
the database filename, which should also simplify the handling
of devices moving around, as these values will not change but
still be unique.

For the name stack we use the <maj>:<min> now as the filename.

> On Tue, Aug 18, 2009 at 09:59:56AM -0400, Ric Wheeler wrote:
> The first is that udev grumbles during boot about "file name too long"
> like the following:
>
> Aug 17 06:49:58 megadeth udevd-event[20447]: unable to create db file
> '/dev/.udev/db/\x2fdevices\x2fpci0000:00\x2f0000:00:04.0\x2f0000:17:00.0\x2f0000:18:0a.0\x2f0000:1f:00.0\x2fhost11\x2fport-11:0\x2fexpander-11:0\x2fport-11:0:0\x2fexpander-11:1\x2fport-11:1:0\x2fexpander-11:2\x2fport-11:2:17\x2fexpander-11:3\x2fport-11:3:1\x2fend_device-11:3:1\x2fbsg\x2fend_device-11:3:1':
> File name too long

warn about non-readable or empty rules file

print warning for NAME="%k" - it breaks the kernel supplied DEVNAME

don't compare a non-existing function with NULL

Obviously someone forgot something here or didn't use -ansi. Either way,
index is nowhere declared so I assume the current behaviour is to check
against the index() function coming from somewhere in the POSIX headers.
The comparison doesn't make sense then.

Signed-off-by: Daniel Mierswa <impulze@impulze.org>

update NEWS

make: add comment

rule_generator: net - fix MATCHDEVID

This got lost for some reason with an earlier change.

Thanks to Marco d'Itri <md@linux.it> for noticing.

rules: rfkill has no group, so use 0644

Thanks to Marco d'Itri<md@linux.it> for noticing.

udev-acl: allow to skip ACL handling

configure.ac: version bump

make: sort Makefile.am per target/extra

release 146

make: fix issues from non-recursive conversion

udevd: block for 15 seconds after error when too old kernel is detected

The compat code will go away some day and CONFIG_SYSFS_DEPRECATED
kernels fail in too many setups now to be worth to support them.

Use LT_INIT to explicit that udev needs libtool series 2.

Fix “make distcheck” run outside of the source directory.

Pass the path to keys.txt as second parameter of check-keymaps.sh so that
it can be found in the right path.

fix spelling

Fix spelling in docbook comments, code comments, and a local variable
name.  Thanks to "ispell -h" for docbook HTML and "scspell" for source
code.

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

Fix building of documentation when doing out-of-source builds.

Since gtk-mkhtml is executed in a sub-directory of the build directory, and
make does not know of that, the $(buildir) variable will still be "." and
the $(srcdir) will not properly be found. For this reason, use the absolute
variants for the two functions, which won't be changing.

Use the keymap check during “make distcheck” rather than “check”.

Since the check-keymaps.sh script checks for validity the source directory
and the Makefile.am file, instead of running it during user-oriented “make
check”, run it during developed-oriented “make distcheck”.

An invalid keymap will abort the execution which will prevent shipping
an incomplete Makefile.am.

To properly support out-of-source builds, pass as single parameter to the
test the path to the source directory.

Merge in Makefile.am.inc into Makefile.am

This removes another file from the distribution, since we're not using it
anywhere else but the top-level Makefile.am file.

Ask gperf to use ANSI-C for generation.

This avoids an '80s C prototype which caused a warning during our build.

Add tests to the distribution; this fixes "make distcheck".

Include the correct directory for out-of-source builds.

When building in-source, the source and header files are in the same
directory, but they are not in out-of-source.

Fix another relative path for the new working directory.

Fix building of introspection library on top-level Makefile.am.

Since the library is in a subdirectory, it has to know where to look for
it.

Add silent-rule support for the gudev rules.

Make sure to use dependency/target variables.

Make sure to clean up all the built sources.

Finally, also merge gudev into the top-level Makefile.am.

The Introspection rules are not tested yet; more touch-ups have been made
for them.

Move pkg-config docs and man pages before conditionals.

Since gudev conditionally installs further pkg-config data, we have to set
it before the condition might be expanded.

Make keymap generation rules be silent (backward-compatible).

Merge keymap building in the top-level Makefile.am.

Slight adjustment around the tests and the rules for the new working
directory.

Change hook handling to be more portable.

Also merge into the top-level Makefile.am the simpler extras.

Replace the custom test-run target with the standard make check.

A little fix is needed for the udev-test.pl script (to be called with the
proper path), but this allows for the test binaries to be only built when
running the tests themselves.

Merge libudev, udev, and the unconditional extras in a single Makefile.am.

Instead of using multiple recursive Makefile.am files, use a single
Makefile.am that sets and builds all the basic suite of libraries and
binaries for udev. This reduces the number of files in the source tree, and
also reduces drastically the build time when using parallel-make.

With this setup, all the compile steps will be executed in parallel, and
just the linking stage will be (partially) serialised on the libraries
creation.

extras/keymap: Fix Bluetooth key on Acer Aspire 6920

See https://launchpad.net/bugs/407940

man: SYMLINK can be matched as well as assigned

If by-path / by-id links don't quite do what you want, this is a nice
clean way to extend the behaviour.  Real example:

SYMLINK=="serial/by-id/usb-Novatel_Wireless_Inc*CDMA*-if00-port0", \\
 SYMLINK+="cellcard"

Some users ask how to do things like this.

 - create an additional link with a shorter name
 - create a link which matches more loosely
    (omit certain path segments e.g. serial numbers)
 - change permissions on certain USB device nodes

Allow them to realize this without reading the friendly *.c files.

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

man: fix unused, inaccurate metadata

Dates aren't shown in the manpages.  So they are not really useful,
and no-one is going to remember to update them.

"<refmiscinfo class="version"></refmiscinfo>" sounds even less useful.

I leave the unused "title" and "productname" tags.  They could
theoretically be useful, and aren't hard to maintain.  We just need to
fix the "title" for udevadm.

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

extras/keymap: Add Samsung NC20

Reported by Dirk Thierbach <dthierbach@gmx.de> on hal ML.

hid2hci: install re-trigger for hid device when recovering from S3

"record_failed" -> "fail_event_on_error"

re-enable failed event tracking

It did not work for the last couple of releases.

If RUN{record_failed}+="..." is given, a non-zero execution will mark
the event as failed. Recorded failed events can be re-triggered with:
  udevadm trigger --type=failed

The failed tracking _might_ be useful for things which might not be
ready to be executed at early bootup, but a bit later when the needed
dependencies are available. In many cases though, it indicates that
something is used in a way it should not.

make: suppress enter/leaving directory messages

libudev: silent gcc warning: may be used uninitialized in this function

udev-acl: add joystick devices

User needs read-write access to joystick devices in order to use force
feedback features.

rules: exclude digitizers from joystick class

Exclude digitizers and similar devices from ID_CLASS joystick by
checking modalias for BTN_DIGI.

This was also done for linux kernel joydev interface in linux commit
d07a9cba6be5c0e947afc1014b5a62182a86f1f1.

update a few years of copyright

fix util_lookup_group to handle large groups

I have recently been getting the above message on fc11 and
I have traced it down to a bug in util_lookup_group.

As of 145 util_lookup_group reads:

gid_t util_lookup_group(struct udev *udev, const char *group)
{
char *endptr;
int buflen = sysconf(_SC_GETGR_R_SIZE_MAX);
char buf[buflen];
struct group grbuf;
struct group *gr;
gid_t gid = 0;

if (strcmp(group, "root") == 0)
return 0;
gid = strtoul(group, &endptr, 10);
if (endptr[0] == '\0')
return gid;

errno = 0;
getgrnam_r(group, &grbuf, buf, buflen, &gr);
if (gr != NULL)
return gr->gr_gid;
if (errno == 0 || errno == ENOENT || errno == ESRCH)
err(udev, "specified group '%s' unknown\n", group);
else
err(udev, "error resolving group '%s': %m\n", group);
return 0;
}

The errno value from getgrnam_r here is ERANGE which is documented as
"Insufficient buffer space supplied".

When I call get getgrnam_r with a large enough buffer everything
works.  Indicating that the problem is that sysconf is returning
a value too small.

A quick google search tells me that sysconf(_S_GETGR_R_SIZE_MAX)
is documented as:

> sysconf(_S_GETGR_R_SIZE_MAX) returns either -1 or a good
> suggested starting value for buflen.  It does not return the
> worst case possible for buflen.

In my case I have a group with about 50 users in /etc/group
and that is what triggered the problem in udev and caused
all of the udevs group lookups to fail.

The following patch which dynamically allocates the group member buffer
should fix this problem.

Signed-off-by: "Eric W. Biederman" <ebiederm@xmission.com>

enumeration: move ALSA control devices to the end of the enumerated devices of each card

Generally ALSA control devices should be the last ones to be processed
for ACL changes and similar operations because they can then be used as
indicators that ACL management finished for all device nodes of a
specific card.

This patch simple moves each controlC device behind all the pcmC devices
(and similar).

make: build internal tools against libudev-private.la

hid2hci: rewrite (and break) rules and device handling

We must never access random devices in /dev which do not belong to
the event we are handling. Hard-coding /dev/hidrawX, and looping over all
devices is absolutely not acceptable --> hook into hidraw events.

We can not relay on (rather random) properties merged into the parent
device by earlier rules --> use libudev to find the sibling device
with a matching interface.

Libusb does not fit into udev's use case. We never want want to scan
and open() all usb devices in the system, just to find the device
we are already handling the event for --> put all the stupid scanning
into a single function and prepare for a fixed libusb or drop it later.

format names are not case insensitive

path_id: fix typo in comment

rules: serial - fix path_id call

extras/keymap: Fix crash for unknown keys

The keymap table has some holes in it, which caused the interactive mode to
crash for unknown keys. In these cases, print the numeric key code instead.

extras/modem-modeswitch: Add Huawei E1550 GSM modem

What's odd is that this is a huawei modem, not an option modem, so one would
expect it to work better with usb_modeswitch and it's -H (huawei) mode - but
that's not the case, I've tested that as well.

https://launchpad.net/bugs/401655

udevadm: trigger - add --sysname-match=

  $ udevadm trigger -n -v --subsystem-match=usb --sysname-match=2-1.1*
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1.1
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1.1/2-1.1.1:1.0
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1.2
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1.2/2-1.1.2:1.0
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1.2/2-1.1.2:1.1
  /sys/devices/pci0000:00/0000:00:1d.7/usb2/2-1/2-1.1/2-1.1:1.0

path_id: make global variable static

hid2hci: remove hid structures and include kernel header

hid2hci: support to hid2hci for recovering Dell BT devices after S3

configure.ac: version bump

v4l_id: exit with 0 when --help is given

extras/keymap: Add Samsung SX22S

Reported on hal ML by Sergey Astanin <s.astanin@gmail.com>.

extras/keymap: teach findkeyboards about USB keyboards

extras/keymap: Add Fujitsu Amilo M

https://launchpad.net/bugs/48547

extras/keymap: cover more Compaq Evo models

https://launchpad.net/bugs/35382 shows that may Evo N* models share the same
keymap, so generalize the existing rule.

extras/keymap: Add HP Presario 2100

https://launchpad.net/bugs/20223

extras/keymap: add recently added keymap files to Makefile.am

extras/keymap: fix check-keymaps.sh for inline mappings

Do not complain about missing keymap files when giving scancode/keyname pairs
on the keymap command line in the rules.

keymap: inline one-line key maps

Remove key map files which have only one override. Instead, use keymap tools'
new feature of specifying scancode/keyname pairs directly at the command line.

Also add a comment to 95-keymap.rules about how to specify key mappings in the
rules.

keymap tool: support scancode/keycode pair arguments

This avoids having to create and parse an entire keymap file for cases where
just one or two keys have to be remapped.

keymap tool: improve help

Do not duplicate help strings, and add missing calling variant for
interactivity.

extras/keymap: add Compal Hel80i

https://launchpad.net/bugs/198530

extras/keymap: add Everex Stepnote XT5000T

Reported in https://launchpad.net/bugs/400921

udevd: add timestamp to --debug output

extras/keymap: add Zepto ZNote

Reported on https://launchpad.net/bugs/400252

add keymap for Clevo D410J laptop

Reported as hal-info patch by Németh Márton <nm127@freemail.hu> on hal ML.

release 145

udevd: use bool

udevd: handle SIGCHLD before the worker event message

We may need to handle SIGCHLD before the queued worker message. The last
reference, from the SIGCHLD or the worker message will clean up the worker
context. In case we receive an unexpected SIGCHLD with an error, we let
the event fail and clean up the worker context.

udevd: make sure a worker finishes event handling before exiting

Persistent network rules write out new rules files. When rules change,
we need to kill all workers to update the in-memory copy of the rules.
We need to make sure, that a worker finshes its work for all device
messages it has accepted, before it exits after a SIGTERM from the main
process.

udevadm: info - add space after R:, A:, W: on database export

<zzam> kay: udevadmin info -e prints all lines with a space after the :, but the W: line
<kay> zzam: yeah, seems so, and the R: and the A:
<zzam> yeah

udevd: detach event from worker if we kill a worker

Jul  8 09:36:41 udevd[663]: worker [5491] did not accept message, kill it
Jul  8 09:36:41 udevd[663]: worker [5491] unexpectedly returned with 0
Jul  8 09:36:41 udevd[663]: worker [5551] unexpectedly returned with 0
Jul  8 09:36:41 kernel: [  156.832086] <6>udevd[663]: segfault at 4 ip 00959fbc sp bfbe7b78 error 6 in udevd[94f000+1c000]

https://bugs.launchpad.net/ubuntu/+source/udev/+bug/396957

libudev: enumerate - sort with qsort()

On machines with many thousands of devices:
  $ time find /sys -name uevent | wc -l
  74876

  real      0m33.171s
  user      0m3.329s
  sys       0m29.719s

the current udevtrigger spends minutes sorting the device list:
  $ time /sbin/udevadm trigger --dry-run

  real      4m56.739s
  user      4m45.743s
  sys       0m7.862s

with qsort() it looks better:
  $ time udev/udevadm trigger --dry-run

  real      0m6.495s
  user      0m0.473s
  sys       0m5.923s

libudev: device - free values before updating them

configure.ac: version bump

use MIN() MAX() from param.h

README: add CONFIG_BLK_DEV_BSG

scsi_id: correct error handling in prepend_vendor_model

The callers of prepend_vendor_model both expect < 0 to be returned on
error and the index to be returned otherwise. However
prepend_vendor_model actually returns 1 on error. Fix this by correctly
returning -1.

Older kernels (before e5b3cd42: "SCSI: sanitize INQUIRY strings")
truncated the model field in sysfs (or propagated bad results from the
target) to less than the expected/required 16 characters which meant
that the SCSI id was mangled into:
        # /sbin/scsi_id -g -s /block/sdg
        S146cee20VIRTUAL-DISK
when it should have been:
        # /sbin/scsi_id -g -s /block/sdg
        SIET     VIRTUAL-DISK   146cee20

Notice how the serial number has been pasted over the vendor+model at
index 1 instead of being added at the end.

In the former case:
        # cat /sys/devices/platform/host5/session1/target5:0:0/5:0:0:1/model | od -t c -t x1
        0000000   V   I   R   T   U   A   L   -   D   I   S   K  \n
                56 49 52 54 55 41 4c 2d 44 49 53 4b 0a
But it should have been:
        # cat /sys/devices/platform/host5/session1/target5:0:0/5:0:0:1/model | od -t c -t x1
        0000000   V   I   R   T   U   A   L   -   D   I   S   K
                56 49 52 54 55 41 4c 2d 44 49 53 4b 20 20 20 20
        0000020  \n
                0a

Signed-off-by: Ian Campbell <ijc@hellion.org.uk>

release 144