[systemd-devel] [PATCH] Set default polling interval on removable devices as well

Martin Pitt martin.pitt at ubuntu.com
Mon Apr 28 03:37:40 PDT 2014


Hey Kay,

Kay Sievers [2014-04-26 19:55 +0200]:
> SUBSYSTEM=="module" is not so much about *loadable* modules, but about
> module *parameters*, which also apply to built-in modules. "block" is
> never a module.

Ah! That misled me then, thanks for clarifying.

> Is the value not applied at all, can you check the file?
> Is it only missing for some devices which might be created before the
> setting is applied?

Here I can only check in qemu with -cdrom and on my laptop (no CD
drive), and /sys/module/block/parameters/events_dfl_poll_msecs indeed
does get applied ("2000"). So the original rule works indeed.

> Are the module parameter rules included in the initramfs?

60-persistent-storage.rules (which has that rule) is in the initramfs;
or do you mean something else? In either case, as you said the rule
should also be (re)applied after initramfs in the real system after
coldplugging?

So several folks on the Debian bug still see this (last confirmation
in Feb this year), so somehow the new rule seems to help. I need to
reinvestigate this again (I was misled by block being not a module
from above), so please ignore the patch for now (if anything, the
description is bogus).

Thanks,

Martin

-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)


More information about the systemd-devel mailing list