Bug 26988 - /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ignoring: 'all_partitions'
Summary: /usr/lib/udev/rules.d/50-udev-mageia.rules:17 Invalid value for OPTIONS key, ...
Status: RESOLVED DUPLICATE of bug 26718
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: Cauldron
Hardware: All Linux
Priority: Normal minor
Target Milestone: ---
Assignee: All Packagers
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-07-22 19:56 CEST by Olav Vitters
Modified: 2021-02-07 00:55 CET (History)
2 users (show)

See Also:
Source RPM: systemd-245.6-3.mga8.src.rpm
CVE:
Status comment:

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

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