Bug 26718 - Journal displays error about systemd-udevd from 50-udev-mageia.rules Invalid value for OPTIONS key, ignoring: 'all_partitions'
Summary: Journal displays error about systemd-udevd from 50-udev-mageia.rules Invalid ...
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: Thomas Backlund
QA Contact:
URL:
Whiteboard:
Keywords:
: 26988 28345 (view as bug list)
Depends on:
Blocks:
 
Reported: 2020-06-02 22:32 CEST by papoteur
Modified: 2021-02-13 00:31 CET (History)
4 users (show)

See Also:
Source RPM: systemd
CVE:
Status comment:

Attachments
Add an attachment (proposed patch, testcase, etc.)

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
Note You need to log in before you can comment on or make changes to this bug.