Bug 26988

Summary:	/usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'
Product:	Mageia	Reporter:	Olav Vitters <olav>
Component:	RPM Packages	Assignee:	All Packagers <pkg-bugs>
Status:	RESOLVED DUPLICATE	QA Contact:
Severity:	minor
Priority:	Normal	CC:	ouaurelien, smelror
Version:	Cauldron
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	systemd-245.6-3.mga8.src.rpm	CVE:
Status comment:

Description Olav Vitters 2020-07-22 19:56:28 CEST

Description of problem:
journalctl shows the following error message on startup:
> /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'

From that file:
> # workaround for devices which do not report media changes (adapted from Debian)
> SUBSYSTEMS=="ide", KERNEL=="hd[a-z]", ATTR{removable}=="1", \
>         ENV{ID_MODEL}=="IOMEGA_ZIP*", OPTIONS+="all_partitions", GROUP="floppy"

Version-Release number of selected component (if applicable):
systemd-245.6-3.mga8.src.rpm

How reproducible:
Start the computer and look at journalctl

Steps to Reproduce:
1. start your computer on Cauldron
2. journalctl -b 0
3. see the message appear as a warning

Comment 1 Aurelien Oudelet 2020-07-23 09:36:26 CEST

See this warning level message too on mga8a1.

CC: (none) => ouaurelien

Comment 2 Olav Vitters 2020-07-23 13:57:52 CEST

commit 1f084fe5811ce251d516a8022f8c7c27d0ef1f53
Author: Kay Sievers <kay@vrfy.org>
Date:   Tue Apr 13 08:52:48 2010 +0200

    remove "all_partitions" option

    We can not predict the major/minor of non-existing devices:
      $ grep . /sys/class/block/sd*/dev
      /sys/class/block/sda1/dev:259:524288
      /sys/class/block/sda2/dev:259:262144
      /sys/class/block/sda3/dev:259:786432
      /sys/class/block/sda4/dev:259:131072
      /sys/class/block/sda/dev:259:0
      /sys/class/block/sdb/dev:259:655360
      /sys/class/block/sdc/dev:259:393216

    If this functionality is still needed for some broken hardware, it needs to be
    solved with a tool not part of the udev package. Because such option is unreliable
    and unsafe to use.

diff --git a/NEWS b/NEWS
index c5fbbe165d..c422eaa8e3 100644
--- a/NEWS
+++ b/NEWS
@@ -2,9 +2,14 @@ udev 152
 ========
 Bugfixes.

-"udevadm trigger" default to "change" events now, instead of "add"
+"udevadm trigger" defaults to "change" events now, instead of "add"
 events.

+The option "all_partitons" was removed from udev. This should not be
+needed for usual hardware. Udev can not safely make assumptions
+about non-exixting partition major/minor numbers, and therefore no
+longer provide such an unreliable and unsafe option.
+

Comment 3 Lewis Smith 2020-07-25 20:29:25 CEST

Thanks for the report & details, Olav.

Assigning this globally as systemd has lost its regular maintainer; CC'ing Stig who has dealt with it recently.

CC: (none) => smelror
Assignee: bugsquad => pkg-bugs

Comment 4 Aurelien Oudelet 2021-02-07 00:55:14 CET

Duplicate. Comment 2 copies there.

*** This bug has been marked as a duplicate of bug 26718 ***

Status: NEW => RESOLVED
Resolution: (none) => DUPLICATE