Bug 26718

Summary:	Journal displays error about systemd-udevd from 50-udev-mageia.rules Invalid value for OPTIONS key, ignoring: 'all_partitions'
Product:	Mageia	Reporter:	papoteur <yvesbrungard>
Component:	RPM Packages	Assignee:	Thomas Backlund <tmb>
Status:	RESOLVED FIXED	QA Contact:
Severity:	normal
Priority:	Normal	CC:	bittwister2, olav, ouaurelien, thierry.vignaud
Version:	Cauldron
Target Milestone:	---
Hardware:	All
OS:	Linux
Whiteboard:
Source RPM:	systemd	CVE:
Status comment:

Description papoteur 2020-06-02 22:32:30 CEST

juin 02 21:44:57 localhost systemd-udevd[495]: /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'

juin 02 21:51:40 localhost systemd-udevd[495]: /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'

juin 02 21:52:03 localhost systemd-udevd[495]: /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'

Mageia 8-alpha1

Comment 1 papoteur 2020-06-02 22:56:42 CEST

It seems this option is removed from a long time ago:
https://github.com/systemd/systemd/commit/1f084fe5811ce251d516a8022f8c7c27d0ef1f53
"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 2 Lewis Smith 2020-06-03 20:36:33 CEST

Thank you for the report and the follow-up.

For systemd, assigning to Thomas; CC'ing Thierry who also maintains it.

Assignee: bugsquad => tmb
CC: (none) => thierry.vignaud
Source RPM: (none) => systemd

Comment 3 Aurelien Oudelet 2021-02-07 00:54:05 CET

Copy of comment by Olav Vitters 2020-07-23 13:57:52 CEST on duplicate bug.

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.
+

CC: (none) => ouaurelien

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

*** Bug 26988 has been marked as a duplicate of this bug. ***

CC: (none) => olav

Comment 5 Aurelien Oudelet 2021-02-11 00:17:17 CET

*** Bug 28345 has been marked as a duplicate of this bug. ***

CC: (none) => bittwister2

Comment 6 Thomas Backlund 2021-02-11 14:14:43 CET

fixed in svn

Comment 7 Thomas Backlund 2021-02-13 00:31:56 CET

and pushed out in systemd-246.9-5.mga8

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