mdev

[Vladimir Dronnikov]

>
> People reading config with these dashes will wonder "what is it?
> oh, I vaguely remeber that it means 'use dafault'...
> but what is the default mode?

If they wonder they surely _want to specify_ something.

> and which dash is mode and which dash is user:group?"

dash in the 2nd field means use default uid:gid. dash in the 3rd field
means use default mode.
dashes in both fields mean use both default uid:gid and default mode.
It can be clearly explain in mdev.conf comment.

>
> It adds code -> it will annoy Rob even more.

I'm not pushing.

---

Another issue still does exist: when a hotplug event occured, $DEVPATH
often contain the path containing [/sys]/devices/..., not
[/sys/class/...]. In such cases our heuristics on guessing $SUBSYSTEM
by simply advance $DEVPATH pointer (path += sizeof(...)) fails. We
should use smth like:

if (subsystem) { /* it means we are in hotplug event */
   path = concat(subsystem, "/", device_name);
} else {
   /* we should guess subsystem here given the path */
   ...
}

--
Vladimir