[PATCH] clarify udevinfo text

Make udevinfo attribute printing note so clear,
that's nearly impossible to misunderstand it in the future.

[PATCH] better fix for NAME="foo-%c{N}" gets a truncated name

On Wed, Mar 03, 2004 at 04:56:34PM -0800, Greg KH wrote:
> On Wed, Mar 03, 2004 at 03:57:04PM -0800, Patrick Mansfield wrote:
> >
> > Here is a patch for some new tests.
>
> Applied, thanks.

Here is a small improvement, which looks much better.

Hey Pat, thanks a lot for finding the recent bug, hope this one will
not break it again :)

[PATCH] Small fix to remove extra "will" in man page

  remove extraneous word.

[PATCH] overall trivial trivial cleanup

Here I try to make the style a bit more consistant in the different
files, so that new patches just copy the 'right' one :)

Some "magic" numbers are replaced and udevtest.c is catched up with udev.

[PATCH] add tests for NAME="foo-%c{N}"

Here is a patch for some new tests.

[PATCH] fix NAME="foo-%c{N}" gets a truncated name

On Wed, Mar 03, 2004 at 02:43:34PM -0800, Patrick Mansfield wrote:
> Here is a fix and a new test for the problem Atul hit, where if we have a
> NAME based on a result of the form:
>
>  NAME="foo-%c{7}"
>
> udev truncates the name. Without any prefix (the foo- in this example),
> the rule was working OK.

Here is a fix for the fix :)

Sorry, I broke it yesterday.

[PATCH] 021_bk mark

[PATCH] fix the build for older versions of gcc

[PATCH] cleanup mult field string handling

Here I try to cleanup our various multifield iteration over the strings.
Inspired by our nice list.h we now have a macro to iterate over the string
and process the parts of it:
It makes the code more readable and we don't change the string while we
process it like the former strsep() does.

Example:

  foreach_strpart(dev->symlink, " ", pos, len) {
   if (strncmp(&dev->symlink[pos], find_name, len) != 0)
   continue;

   ...
  }

For the callout part selector %c{2} we separate now not only by space but
also newline and return characters, cause some programs may give multiline
values back. A possible RESULT match must contain wildcards for these
characters.

Also a bug in the recent udevinfo symlink query feature is fixed.

[PATCH] 021 release

[PATCH] fix udev.spec to find udevinfo now that it has moved to /usr/bin

[PATCH] install udevinfo in /usr/bin

Here we rename the former tiny $(HELPER) to $(INFO)
cause it's no longer only a helper :)
And install it in /usr/bin instead of /sbin cause any user
may want to call it and we don't need it on startup.

[PATCH] blacklist pcmcia_socket

Is this something for the blacklist?

/sys/class/pcmcia_socket/
|-- pcmcia_socket0
|   |-- device -> ../../../devices/pci0000:00/0000:00:1e.0/0000:02:00.0
|   `-- driver -> ../../../bus/pci/drivers/yenta_cardbus
`-- pcmcia_socket1
    |-- device -> ../../../devices/pci0000:00/0000:00:1e.0/0000:02:00.1
        `-- driver -> ../../../bus/pci/drivers/yenta_cardbus

[PATCH] Fix another problem with Makefile installing initscript

[PATCH] fix the Makefile to install the init script into the proper directory

grrr...robert...

[PATCH] make spec file turn off selinux support by default.

[PATCH] 020 release

[PATCH] update the TODO list as we already have a devfs config file.

[PATCH] make start_udev use udevstart binary

[PATCH] man page udevstart

[PATCH] install udevstart

[PATCH] cleanup udevstart

I just wanted to terminate the snprintf() strings, cause I can see a
overflow with closed eyes after all the audit :)

But then I changed a bit more to bring it in line with the style of the
other files. I replaced the exec_udev() function with the one from
udevd, cause we don't need to read the stdout from udev.

Please have a look if it still works for you too and not
only for usernames with 3 characters :)

[PATCH] automatically install correct initscript

On Mon, 2004-03-01 at 20:08, Robert Love wrote:

> Ack, I did not even see that!  Thanks.
>
> Let's rip that out, and always use the new built-in logic to determine
> what initscript to install.

Hm, looks like we do not need the %{lsb} and USE_LSB logic at all,
anymore.

Here is the patch, updated, removing both completely.

[PATCH] bugfix for local user

While moving the local user logic in it's own function I missed to
change the "secure" string macro. We copy only the first 3 bytes
of the username. Guess why I didn't notice it :)

[PATCH] Remove Debian permission files as the Debian maintainer doesn't seem to want to share :(

[PATCH] update the Gentoo rules files.

[PATCH] Add Red Hat rules and permissions files

Taken from udev-018-2.src.rpm from Fedora Devel.

[PATCH] add udevstart program based on a old patch from Harald Hoyer <harald@redhat.com>

This can be used instead of the start_udev script for systems
that do not have a shell, or some other problem...

[PATCH] - unlink bugfix

Fix for recent unlink patch.

[PATCH] TODO update

I promise, that I will not take every item you put in the TODO list :)

[PATCH] clarify udevinfo device walk

It seems that the long attribute list can confuse the user.
Se here we print a few words on top the attributes.

[PATCH] udevinfo symlink reverse query

Thanks to Olaf Hering <olh@suse.de> for this patch. It's possible now to
feed the -n option of udevinfo with a symlink.

I've also added a 'all' attribute, but no more text, it's all in the
included man page :)

[PATCH] unlink the file before we try to create it.

Based on the patch in the Gentoo repo.

[PATCH] fix stroul endptr use

The endptr is never NULL, so here we hopefully do the right thing.

[PATCH] update documetation for $local

On Mon, 2004-03-01 at 17:44, Greg KH wrote:
> Hm, that should be Robert's job actually, he should do something for
> real...  :)

Hey, I wrote the nifty local user detection snippet - but, I owe Kay, so
here is an updated man page.

[PATCH] multipath update

[PATCH] add $local user spport for permissions

[PATCH] udev - man page update

Here is a small change to the udev man page:

  - clarify the use of the NAME{all_partitions} syntax
    and add a example to udev.rules.example
  - mention the empty NAME field to ignore the device
  - prepare a SYMLINK field for the addition of Andrey's
    "multiple symlinks" documentation :)

[PATCH] no error on enoent

no_error_on_enoent: do not exit with an error and delete all files
when a device or directory does not exist.

[PATCH] escape dashes in man pages

man-dashes.diff: escape dashes in man pages, helps with UTF-8 locales
(by Philipp Matthias Hahn).

[PATCH] remove usage of expr in ide-devfs.sh

fix_expr: remove usage of expr in ide-devfs.sh, because it may be in
/usr/bin and not available at early boot time.

[PATCH] udev - fix debug info for multiple rule file config

On Sat, Feb 28, 2004 at 09:56:32PM +0100, Kay Sievers wrote:
> Andrey pointed out that we don't print the right filename in the debug
> output. Here is a fix for that. It applies on top of Andrey's symlink
> patch, cause we are touching the same part of the code.

The copy/paste devil catched me :)
Here is a fixed one.

[PATCH] udev - activate formt length attribute

We carried the the old callout part selector syntax for two releases
now after it was replaced by the new %c{1} syntax. So here we remove
the old syntax and use the code to possibly specify the maximum count
of chars to insert into the string. It will work with all of our format
chars.
I don't know if somebody will use it, but the code is already there :)

's%3s{vendor}' returns "IBM" now, instead of "IBM-ESXS".

Also added is a test for it and a few words in the man page.

[PATCH] 019_bk mark

[PATCH] Add symlink only rules support

[PATCH] udev - kill udevd on install

We want to kill udevd after installing a new version, cause the event
contains a magic with the version number of udev.

[PATCH] udev - safer sprintf() use

Here is for now my last patch to the string handling for a rather
theorethical case, where the node is very very very long. :)

We have accordant to strfieldcat(to, from) now a strintcat(to, i) macro,
which appends the ascii representation of a integer to a string in a
safe way.

[PATCH] 018 release

[PATCH] update udev scsi_id to scsi_id 0.4

This patch syncs the scsi_id in the udev tree to version 0.4.

[PATCH] add new TODO item about local user permissions.

[PATCH] TODO update

On Thu, Feb 12, 2004 at 05:26:37PM -0800, Greg KH wrote:
> On Fri, Feb 13, 2004 at 12:45:38AM +0100, Kay Sievers wrote:
> >
> > Here a few questions about my favorite file in the tree :)
> > - better permission handling
> >
> > What is missing here?
>
> I don't know for sure.  Just a vague feeling that the way we currently
> handle permissions is pretty lousy.  Anyone else feel this way too?

Seems that nobody cares and perhaps the recent klibc permission changes
and the multiple file config directory are enough to kill these lines? :)

[PATCH] udev - correct relative symlink

Here we remove the useless leading "./" of the linktargets.
Thanks to Olaf Hering <olh@suse.de>, who asked why we do this :)

We have now:

  /udev
  |-- camera0 -> video0
  |-- kamera0 -> video0
  `-- video0

[PATCH] Add initial SELinux support for udev

Based on a patch from Daniel J Walsh <dwalsh@redhat.com>

[PATCH] fix build for very old versions of make.

Should get rid of some more error reports of libsysfs header issues.

[PATCH] remove limit of the number of args passed to PROGRAM

If we go over our internal limit of 7, then we call out to /bin/sh
otherwise we handle it ourself without relying on a shell.

[PATCH] udev - safer string handling - part four

Mainly a cleanup of the earlier patches with a few missing pieces
and some cosmetical changes.

I've moved the udev_init_config() to very early init, otherwise we
don't get any logging for the processing of the input. What would I
do without gdb :)

Greg, it's the 7th patch in your box to apply. I will stop now and
wait for you :)

[PATCH] udev - safer string handling - part three

Here we truncate our input strings from the environment to our
defined limit. It's a bit theroretical but better check for it.

It cleans up some magic length definitions and removes the code
duplication in udev, udevtest and udevsend.

udevd needs to be killed after installation, cause the message size
is changed with this patch.
Should we do this with the 'make install', like we do with the '.udevdb'?

[PATCH] udev - safer string handling - part two

As promised, here is the next round. We provide in addition to the
already used macros:

  strfieldcpy(to, from)
  strfieldcat(to, from)

the corresponding friends, if the size of the target is not known and
must be provided by the caller:

  strnfieldcpy(to, from, maxsize)
  strnfieldcat(to, from, maxsize)

and switch nearly all possibly unsafe users of strcat(), strncat(),
strcpy() and strncpy() to these safer macros.

The last known remaining issue seems the use of sprintf() and
snprintf(). I will take on it later today or tomorrow.

[PATCH] udev - man page update

Hey it's not longer the "goal" to provide a dynamic dev directory,
we have just arrived. So I  change it to more self-confident words :)

I've also added the completly missing environment variables to the
man pages.

To stop the misuse of the PROGRAM= call paramenters, we better mention
its limitations.

[PATCH] udev - safer string handling all over the place

On Tue, Feb 24, 2004 at 11:50:52PM +0100, Kay Sievers wrote:
> Here is the first step towards a safer string handling.
> More will follow, but for now only the easy ones :)
>
> Thanks to all who pointed this out. strncat() isn't a nice function. We
> all should remember that the destination string is not terminated if the
> given lenght is shorter than the strlen of the source string.
>
> And shame on the various implementers of strfieldcat() I found in the
> unapplied patches on this list, it's not really better than strncpy()
> and hides the real problem.

Hmm, bk didn't checked in one file, maybe I edited it again as root.
Nevermind, here is the more complete version.

[PATCH] manpage update

Nice, here is the corresponding man update which also removes the
mention of the limitation of getgrname() and friends with klibc.

[PATCH] do not remove real .udev.tdb during RPM build

[PATCH] udev - allow all files in a directory as the config

I was on the train for 5 hours today and the TODO is almost empty :)
So, at least four people wanted this feature, then here is a actual
working patch.

We may specify now in udev.conf:

  udev_rules="/etc/udev/"

and udev will scan the whole directory for files ending with *.rules,
sort it in lexical order and create our rule list from all of the files.
A plain given file will still work and the same applies to the *.permissions.

I sort the files in our usual linked list, cause klibc has no scandir().

[PATCH] udev - simple klibc textual uid/gid handling

Here we get a very dumb getpwnam() and getgrnam() for klibc to
stop the confusion of not handling textual id's if klibc is used.

If used with  initrd we just need to copy the /etc/passwd and /etc/group
file and all should work well.

[PATCH] force udev to include the internal version of libsysfs and never the external one.

Should fix some more build bugs...

[PATCH] fix up libsysfs header file usage to fix bug reports from users that have sysfsutils installed already.

[PATCH] remove udevtest on 'make clean'

[PATCH] 018_bk mark

[PATCH] remove udevd priority TODO item, as it's not needed at all.

We want to be started by keventd, as that runs at a very fast priority.

[PATCH] v018 release

[PATCH] added scsi_id and some more documentation to the udev.spec file.

[PATCH] update udev.rules.gentoo with new config file format.

[PATCH] Update the Gentoo udev.rules and udev.permissions files

These are very good devfs-like rules for others to use if they want to.

[PATCH] Create a udev.rules.examples file to hold odd udev.rules

These are for examples only, not for everyone to use as they cause too
many problems for different people when they are in the main udev.rules
file.

[PATCH] symlink dm-[0-9]* rule

[PATCH] add udevd priority issue to the TODO list.

[PATCH] more HOWTO cleanups.

[PATCH] add HOWTO detailing how to use udev to manage /dev

[PATCH] udev - TODO update

Make TODO really small :)

[PATCH] udev use new libsysfs header file location

Use the new location of libsysfs header files.

[PATCH] mv libsysfs/libsysfs.h to libsysfs/sysfs/libsysfs.h to make it easier to use.

[PATCH] add start_udev init script.

[PATCH] add support for UDEV_NO_SLEEP env variable so Gentoo people will be happy.

Actually, I'm happy to, startup time is much smaller...

[PATCH] start up udevd ourselves in the init script to give it some good priorities.

[PATCH] update extras/multipath

patch follows :

        * remove the restrictive -f flag.
          Introduce a more generic "-m iopolicy" one.
        * remove useless "int with_sysfs" in env struct

[PATCH] init.d debian patch

I dualboot between 2.4.x and 2.6.x right now, and although I
want udev to start up when 2.6 is booting, I don't want it to try when
2.4.x is booting.

This is a small patch to not start up udev if sysfs is not
mounted.

[PATCH] udev - add %s{filename} to man page

Add the new %s{filename} to the udev man page.
And add the automatic start of udevd to the udevd man page.

[PATCH] update the red hat init script to handle nodes that are not present

Thanks to Gentoo for the list of these files.

[PATCH] udev - udevd/udevsend man page

Here is the missing man page for udevd/udevsend.

[PATCH] add a "old style" SYSFS_attribute test to udev-test.pl

[PATCH] udev - switch callout part selector to {attribute}

Here we change the magic callout part number selector to the new
atribute syntax. The syntax to select the second part of the callout string:

 '%2c' is now '%c{2}'

I think it's more clear and we no longer misuse the length argument.

The old syntax is still supported, but we should remove it some
time in the future.

[PATCH] udev - switch SYSFS_file to SYSFS{file}

Here we switch the configs and man pages to the new attribute syntax.
Also the 'partition trick' is mentioned in udev.8

I think it's more clear visible now, that inside the brackets are user
supplied values used and not some magic keys handled:

  'SYSFS_dev' is now 'SYSFS{dev}'

The old syntax is still supported.

[PATCH] Have udevsend report more info in debug mode.

[PATCH] Have udevd report it's version in debug mode.

[PATCH] udev add some ID tests

Patch against current udev bk to add a few ID rule tests.

[PATCH] fix up bug created for udevtest in previous partition creation patch.

[PATCH] udev - create all partitions of blockdevice

Here is the first try to create all partitons of a blockdevice, since
removable media devices may need to acces the expected partition to
revalidate the media.

It uses the attribute syntax introduced with the last %s{file} patch.
I'm using this with my multi-slot-flash-card-reader:

  SYSFS{model}="USB Storage-SMC ", NAME{all_partitions}="smartmedia"
  SYSFS{model}="USB Storage-CFC ", NAME{all_partitions}="compactflash"
  SYSFS{model}="USB Storage-MSC ", NAME{all_partitions}="memorystick"
  SYSFS{model}="USB Storage-MMC ", NAME{all_partitions}="multimedia"

and I get:

  tree /udev/
  /udev/
  |-- memorystick
  |-- memorystick1
  |-- memorystick10
  |-- memorystick11
  |-- memorystick12
  |-- memorystick13
  |-- memorystick14
  |-- memorystick15
  |-- memorystick2
  |-- memorystick3
  |-- memorystick4
  |-- memorystick5
  |-- memorystick6
  |-- memorystick7
  |-- memorystick8
  |-- memorystick9
  |-- multimedia
  |-- multimedia1
  |-- multimedia10
  |-- multimedia11
  |-- multimedia12
  |-- multimedia13
  |-- multimedia14
  |-- multimedia15
  |-- multimedia2
  |-- multimedia3
  |-- multimedia4
  |-- multimedia5
  |-- multimedia6
  |-- multimedia7
  |-- multimedia8
  |-- multimedia9
  ...

If needed, we can make the number of partions to create
adjustable with the attribute?

[PATCH] allow SYSFS{file}

On Sun, Feb 15, 2004 at 03:36:00AM +0100, Kay Sievers wrote:
>
> Since we have %s{file} it may be nice to allow SYSFS{file}.
> This patch allows:
>
>   BUS="usb", SYSFS{idProduct}="a511", NAME="video%n"
>
> compared to the current:
>
>   BUS="usb", SYSFS_idProduct="a511", NAME="video%n"
>
> The curent syntax is still supported.
> Looks a bit nicer and less hackish, I think.

Better patch with infrastructure to easily implement KEY{attribute}
for every other key. The first user is the SYSFS{file} key.
Both versions, brackets or underscore is supported for the attribute.

[PATCH] Adding '%s' format specifier to NAME and SYMLINK

On Thu, Feb 12, 2004 at 05:34:57PM -0800, Greg KH wrote:
> On Tue, Feb 10, 2004 at 09:14:20AM +0100, Hannes Reinecke wrote:
> > Hi all,
> >
> > this patch makes the format for NAME and SYMLINK a bit more flexible:
> > I've added a new format specifier '%s{<SYSFS_var>}', which allows for
> > the value of any sysfs entry found for this device to be inserted.
> > Example (for our S/390 fcp adapter):
> >
> > BUS="ccw", SYSFS_devtype="1732/03", NAME="%k" \
> > SYMLINK="zfcp-%s{hba_id}-%s{wwpn}:%s{fcp_lun}"
> >
> > I know this could also be done with an external program, but having this
> > incorporated into udev makes life easier, especially if run from
> > initramfs. Plus it makes the rules easier to follow, as the result is
> > directly visible and need not to be looked up in some external program.
> >
> > Comments etc. welcome.
>
> Oops, sorry I missed this for the 017 release.  I'll look at it tomorrow
> and get back to you.  At first glance it looks like a good thing.
>
> Oh, you forgot to update the documentation, that's important to do if
> you want this change to make it in :)

I took a part of the code and made a version that uses already implemented
attribute finding logic.

The parsing of the format length '%3x' and the '%x{attribute}' is a fuction now,
maybe there are more possible users in the future.

I've also added the test to udev-test.pl.

[PATCH] update the udev.spec to add udevtest and make some more Red Hat suggested changes.