https://bugzilla.suse.com/show_bug.cgi?id=1220303 https://bugzilla.suse.com/show_bug.cgi?id=1220303#c2 Jan Engelhardt changed: What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(jengelh@inai.de) | --- Comment #2 from Jan Engelhardt ---

Why do you expect that after installing the package, the module is automatically loaded?

That is *not* what I wrote in the "Expected behavior" section. -- You are receiving this mail because: You are the assignee for the bug.

https://bugzilla.suse.com/show_bug.cgi?id=1220303 https://bugzilla.suse.com/show_bug.cgi?id=1220303#c4 Martin Wilck changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(jengelh@inai.de) --- Comment #4 from Martin Wilck --- I created https://github.com/openSUSE/suse-module-tools/pull/100, please take a look. I tested it with vhba-kmp successfully. But there's a catch: This works for KMP installation (should also work for "inkmp", not tested), but when KMPs are removed, the previously created static device nodes remain in place, which isn't optimal. This is because "kmod static nodes --format=tmpfiles" creates entries like this: c! /dev/vhba_ctl 0600 - - - 10:170 but it never creates "remove" entries for already existing static nodes that aren't listed in modules.devname any more. I suppose doing so might actually be dangerous. I don't think that there is a safe way to remove such nodes on KMP removal. Note also that according to tmpfiles.d(5), the exclamation mark means "this line is only safe to execute during boot, and can break a running system". kmod always adds this "!", which may seem over-cautious, but is in line with tmpfiles.d(5), which recommends doing this for static device nodes. If we add this functionality, we will be deliberately ignoring these recommendations. -- You are receiving this mail because: You are the assignee for the bug.

https://bugzilla.suse.com/show_bug.cgi?id=1220303 https://bugzilla.suse.com/show_bug.cgi?id=1220303#c6 Martin Wilck changed: What |Removed |Added ---------------------------------------------------------------------------- Flags| |needinfo?(msuchanek@suse.co | |m) --- Comment #6 from Martin Wilck --- (In reply to Jan Engelhardt from comment #5)

...

I don't think that there is a safe way to remove such nodes on KMP removal.

The kernel module stays loaded even if you remove the KMP; therefore the node should stay too. Moreover, some kernel modules may not be rmmodable right now due to being in use.

Right. But the device node stays in place even if the module had never been loaded.

I can't find a direct wording in tmpfiles.d(5) for c! specifically (just r! and ! in general).

"c, c+ [...] It is recommended to suffix this entry with an exclamation mark to only create static device nodes at boot, as udev will not manage static device nodes that are created at runtime." It comes down to basically the same thing as your quote below.

kmod commit 27eceb2e4ecbf2e31940b67cee682a6935f61e6c has some helpful remarks:

>creating static nodes later on will likely not >behave as expected. In particular, recreating >the static nodes at run-time will reset any >permissions udev may have applied to the nodes >at boot.

I suppose that we won't do much harm by creating new static nodes when KMPs are added. Unfortunately "systemctl restart kmod-static-nodes.service systemd-tmpfiles-setup-dev.service" will also re-create all previously existing nodes, which may actually be harmful. Michal, what's your take on this? -- You are receiving this mail because: You are the assignee for the bug.

https://bugzilla.suse.com/show_bug.cgi?id=1220303 https://bugzilla.suse.com/show_bug.cgi?id=1220303#c8 --- Comment #8 from Takashi Iwai --- The purpose is other way round, those static nodes are rather to do auto-loading the module, but yeah, they are mostly legacy infrastructure. -- You are receiving this mail because: You are the assignee for the bug.

https://bugzilla.suse.com/show_bug.cgi?id=1220303 https://bugzilla.suse.com/show_bug.cgi?id=1220303#c10 --- Comment #10 from Martin Wilck --- I don't think we can follow the approach in https://github.com/openSUSE/suse-module-tools/pull/100 (which is basically what Jan proposed in commment 0). What we can try at KMP install time is to check the output of "kmod static-nodes" for new entries, and create them without touching existing ones. I see this as a feature request though. I don't think I'll have time to work on this soon. -- You are receiving this mail because: You are the assignee for the bug.