1
0
mirror of git://sourceware.org/git/lvm2.git synced 2024-12-21 13:34:40 +03:00

udev: disable suffix handling

The idea in the patch 6e6d4c62b for handling -suffix as
indication of private device needs to be disabled.

Some problematic cases are currently not resolvable and some
more thinking is needed.

Once fixed, we can revert this patch.
This commit is contained in:
Zdenek Kabelac 2024-06-25 23:13:54 +02:00
parent 336cb32884
commit 21b215ee63
2 changed files with 10 additions and 2 deletions

View File

@ -3645,7 +3645,14 @@ static int _add_new_lv_to_dtree(struct dev_manager *dm, struct dm_tree *dtree,
if (!layer && lv_is_writecache_origin(lv)) if (!layer && lv_is_writecache_origin(lv))
layer = lv_layer(lv); /* "real" */ layer = lv_layer(lv); /* "real" */
if (lvlayer->visible_component) { /*
* FIXME: we would like to have -private suffixes used for device not processed by udev
* however ATM we also sometimes want to provide /dev/vg/lv symlinks to such devices
* and also be able to correctly report its status with lvs.
*
* Until problems are resolved this code path needs to be disabled.
*/
if (0 && lvlayer->visible_component) {
/* Component LV will be public, do not add any layer suffixes */ /* Component LV will be public, do not add any layer suffixes */
if (!(dlid = dm_build_dm_uuid(dm->mem, UUID_PREFIX, lv->lvid.s, NULL))) if (!(dlid = dm_build_dm_uuid(dm->mem, UUID_PREFIX, lv->lvid.s, NULL)))
return_0; return_0;

View File

@ -35,7 +35,8 @@ ENV{DM_SUBSYSTEM_UDEV_FLAG0}=="1", ENV{.DM_NOSCAN}="1", ENV{DM_UDEV_DISABLE_OTHE
ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="lvm_end" ENV{DM_UDEV_DISABLE_SUBSYSTEM_RULES_FLAG}=="1", GOTO="lvm_end"
# Ignore private devices - the ones having any suffix in DM_UUID. Both in udev rules and in systemd. # Ignore private devices - the ones having any suffix in DM_UUID. Both in udev rules and in systemd.
ENV{DM_UUID}=="LVM-?*-?*", ENV{SYSTEMD_READY}="0", GOTO="lvm_disable" ## FIXME resolve -suffix handling
## ENV{DM_UUID}=="LVM-?*-?*", ENV{SYSTEMD_READY}="0", GOTO="lvm_disable"
# Do not create symlinks for inappropriate subdevices. # Do not create symlinks for inappropriate subdevices.
ENV{DM_LV_NAME}=="pvmove?*|?*_vorigin", GOTO="lvm_disable" ENV{DM_LV_NAME}=="pvmove?*|?*_vorigin", GOTO="lvm_disable"